)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"3acb4cc0c6d49a25cf8a55cf63a2eff2c744c872","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make _create_loginable_secgroup public"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As tempest.scenario.manager was announced stable interface in Tempest 27.0.0[1] it can be now reused in plugins."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"_create_loginable_secgroup method isn\u0027t ready to be deleted yet, because its rulesets differs from tempest\u0027s. This method will be public now, so replaced methods will use this method over the tempest\u0027s one. Tempest doesn\u0027t allow to modify rulesets, but patch that will allow this is already proposed[2]. Some time after merging the mentioned tempest patch this plugin\u0027s implementation will be replaced."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Etherpad concerning this effort:"},{"line_number":14,"context_line":"https://etherpad.opendev.org/p/tempest-scenario-manager-cleanup"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"d4db2297_8925e2e6","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":402},"updated":"2022-07-13 11:42:12.000000000","message":"note: please wrap at 72 characters","commit_id":"22fde6a5882e189db52a5cf3ed63e185ee929955"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"17dd2d5f21052f179f6f8f9da435f9ffb7506f44","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make _create_loginable_secgroup public"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As tempest.scenario.manager was announced stable interface in Tempest 27.0.0[1] it can be now reused in plugins."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"_create_loginable_secgroup method isn\u0027t ready to be deleted yet, because its rulesets differs from tempest\u0027s. This method will be public now, so replaced methods will use this method over the tempest\u0027s one. Tempest doesn\u0027t allow to modify rulesets, but patch that will allow this is already proposed[2]. Some time after merging the mentioned tempest patch this plugin\u0027s implementation will be replaced."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Etherpad concerning this effort:"},{"line_number":14,"context_line":"https://etherpad.opendev.org/p/tempest-scenario-manager-cleanup"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"cc02d387_c0382df8","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":402},"in_reply_to":"d4db2297_8925e2e6","updated":"2022-07-13 19:25:26.000000000","message":"++","commit_id":"22fde6a5882e189db52a5cf3ed63e185ee929955"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"4403925ae6ed4c718ec38b37691ed87672d79779","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8ca6e354_828ce7d6","updated":"2022-06-08 12:24:44.000000000","message":"recheck\n\nlogs are old","commit_id":"e07afa9cfee32f1f2f02f949f3942588218476f5"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"449e8b1d5a700385de0bbc82688e345c731c2d39","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"840efee9_40a4b58a","updated":"2022-07-13 19:34:57.000000000","message":"LGTM, thank you","commit_id":"22fde6a5882e189db52a5cf3ed63e185ee929955"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"cfcb5c03a8b5ab00c95495acb15324beb7ac9041","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"58f43150_7da964e1","updated":"2022-07-13 19:25:32.000000000","message":"Looking good, thanks","commit_id":"22fde6a5882e189db52a5cf3ed63e185ee929955"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"3acb4cc0c6d49a25cf8a55cf63a2eff2c744c872","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"4fcaecd0_b6377257","updated":"2022-07-13 11:42:12.000000000","message":"recheck\n\nGeneric scenario test must pass ","commit_id":"22fde6a5882e189db52a5cf3ed63e185ee929955"}]}
