)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"b5dd87cf7764661e43b035c542b36b5a99191ca3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"91fa2651_f15a131b","updated":"2023-10-04 10:58:29.000000000","message":"looks ok","commit_id":"226220ac3c9f651ea4d5b564f7cbe1acd3a272a8"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"2e0bbd5ff572f53e0255795841ef12f5de31c83d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f03599bc_0e5eaf75","updated":"2023-10-07 08:37:26.000000000","message":"recheck timeout","commit_id":"226220ac3c9f651ea4d5b564f7cbe1acd3a272a8"}],"neutron/services/logapi/drivers/ovn/driver.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6ada886429741fe150b693e2ed3b52e76d218485","unresolved":true,"context_lines":[{"line_number":157,"context_line":"        acl_changes, acl_visits \u003d 0, 0"},{"line_number":158,"context_line":"        for pg in pgs:"},{"line_number":159,"context_line":"            meter_name \u003d self.meter_name"},{"line_number":160,"context_line":"            if ovn_const.OVN_DROP_PORT_GROUP_NAME not in pg[\"name\"]:"},{"line_number":161,"context_line":"                stateful \u003d (sg_obj.SecurityGroup"},{"line_number":162,"context_line":"                            .get_sg_by_id(context, pg[\"name\"]"},{"line_number":163,"context_line":"                                          .replace(\u0027pg_\u0027, \u0027\u0027, 1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"274ad368_82b81632","line":160,"range":{"start_line":160,"start_character":50,"end_line":160,"end_character":56},"updated":"2023-09-14 15:10:24.000000000","message":"why is there \"not in\" operator and not \"!\u003d\" ?","commit_id":"3d45228d5b826b52a15af7309be673af3aa78bf9"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"90859be11ba72b19edc97343989dad55030b0f40","unresolved":false,"context_lines":[{"line_number":157,"context_line":"        acl_changes, acl_visits \u003d 0, 0"},{"line_number":158,"context_line":"        for pg in pgs:"},{"line_number":159,"context_line":"            meter_name \u003d self.meter_name"},{"line_number":160,"context_line":"            if ovn_const.OVN_DROP_PORT_GROUP_NAME not in pg[\"name\"]:"},{"line_number":161,"context_line":"                stateful \u003d (sg_obj.SecurityGroup"},{"line_number":162,"context_line":"                            .get_sg_by_id(context, pg[\"name\"]"},{"line_number":163,"context_line":"                                          .replace(\u0027pg_\u0027, \u0027\u0027, 1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"81a43b0f_76ec11be","line":160,"range":{"start_line":160,"start_character":50,"end_line":160,"end_character":56},"in_reply_to":"15dacee8_6f4d4060","updated":"2023-09-27 11:16:56.000000000","message":"Done, thanks!","commit_id":"3d45228d5b826b52a15af7309be673af3aa78bf9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f228c18c01c364db6dffc79fb06db7fab4bc7c68","unresolved":true,"context_lines":[{"line_number":157,"context_line":"        acl_changes, acl_visits \u003d 0, 0"},{"line_number":158,"context_line":"        for pg in pgs:"},{"line_number":159,"context_line":"            meter_name \u003d self.meter_name"},{"line_number":160,"context_line":"            if ovn_const.OVN_DROP_PORT_GROUP_NAME not in pg[\"name\"]:"},{"line_number":161,"context_line":"                stateful \u003d (sg_obj.SecurityGroup"},{"line_number":162,"context_line":"                            .get_sg_by_id(context, pg[\"name\"]"},{"line_number":163,"context_line":"                                          .replace(\u0027pg_\u0027, \u0027\u0027, 1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"15dacee8_6f4d4060","line":160,"range":{"start_line":160,"start_character":50,"end_line":160,"end_character":56},"in_reply_to":"274ad368_82b81632","updated":"2023-09-21 08:42:02.000000000","message":"Agree with this comment. This is working by accident. This is comparing two strings, checking if the pg name is \"neutron_pg_drop\".\n\nPlease fix this in master and 2023.2 (new stable branch) and change it here.","commit_id":"3d45228d5b826b52a15af7309be673af3aa78bf9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ed0feb4ea14fb0eeee6942b036bcbfea390e6f22","unresolved":false,"context_lines":[{"line_number":157,"context_line":"        acl_changes, acl_visits \u003d 0, 0"},{"line_number":158,"context_line":"        for pg in pgs:"},{"line_number":159,"context_line":"            meter_name \u003d self.meter_name"},{"line_number":160,"context_line":"            if ovn_const.OVN_DROP_PORT_GROUP_NAME not in pg[\"name\"]:"},{"line_number":161,"context_line":"                stateful \u003d (sg_obj.SecurityGroup"},{"line_number":162,"context_line":"                            .get_sg_by_id(context, pg[\"name\"]"},{"line_number":163,"context_line":"                                          .replace(\u0027pg_\u0027, \u0027\u0027, 1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5c24f14c_fbf083f7","line":160,"range":{"start_line":160,"start_character":50,"end_line":160,"end_character":56},"in_reply_to":"81a43b0f_76ec11be","updated":"2023-10-04 10:28:27.000000000","message":"Change cherry-picked in https://review.opendev.org/c/openstack/neutron/+/896614","commit_id":"3d45228d5b826b52a15af7309be673af3aa78bf9"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6ada886429741fe150b693e2ed3b52e76d218485","unresolved":true,"context_lines":[{"line_number":159,"context_line":"            meter_name \u003d self.meter_name"},{"line_number":160,"context_line":"            if ovn_const.OVN_DROP_PORT_GROUP_NAME not in pg[\"name\"]:"},{"line_number":161,"context_line":"                stateful \u003d (sg_obj.SecurityGroup"},{"line_number":162,"context_line":"                            .get_sg_by_id(context, pg[\"name\"]"},{"line_number":163,"context_line":"                                          .replace(\u0027pg_\u0027, \u0027\u0027, 1)"},{"line_number":164,"context_line":"                                          .replace(\u0027_\u0027, \u0027-\u0027)).stateful)"},{"line_number":165,"context_line":"                if not stateful:"},{"line_number":166,"context_line":"                    meter_name \u003d meter_name + (\"_stateless\")"},{"line_number":167,"context_line":"            for acl_uuid in pg[\"acls\"]:"}],"source_content_type":"text/x-python","patch_set":2,"id":"e069465f_4b317a94","line":164,"range":{"start_line":162,"start_character":51,"end_line":164,"end_character":61},"updated":"2023-09-14 15:10:24.000000000","message":"Is the external_ids:security-group not available?","commit_id":"3d45228d5b826b52a15af7309be673af3aa78bf9"}]}
