)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"a430bf4a309aafc7d072a99c5be864aa76813f28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6ea9b62c_c7942fca","updated":"2026-09-26 12:00:03.000000000","message":"Thanks for the patch! I added some small remarks","commit_id":"f38716fc3fb4603096cfb2e5aa5949b2f80d0e1e"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"b9c9760724bae48ce3629b705f670248bb4dcb19","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c364fd2b_1560a783","updated":"2026-09-29 09:23:15.000000000","message":"thanks, looks ok","commit_id":"aa698252292dc79eb0b2481e429312ac2ef05801"}],"neutron/services/logapi/drivers/ovn/driver.py":[{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"a430bf4a309aafc7d072a99c5be864aa76813f28","unresolved":true,"context_lines":[{"line_number":450,"context_line":"            # to check which ACL is that UUID referring to, using match as"},{"line_number":451,"context_line":"            # differentiating value."},{"line_number":452,"context_line":"            for acl in neutron_acls:"},{"line_number":453,"context_line":"                if acl[\u0027port_group\u0027] !\u003d pg[\u0027name\u0027]:"},{"line_number":454,"context_line":"                    continue"},{"line_number":455,"context_line":"                acl_visits +\u003d 1"},{"line_number":456,"context_line":"                # skip acls used by a different network log"},{"line_number":457,"context_line":"                n_acl_name \u003d acl[\u0027name\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"5af69e67_b30681e2","line":454,"range":{"start_line":453,"start_character":1,"end_line":454,"end_character":28},"updated":"2026-09-26 12:00:03.000000000","message":"Please put this after the acl_visits counter is increased","commit_id":"f38716fc3fb4603096cfb2e5aa5949b2f80d0e1e"},{"author":{"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},"change_message_id":"5a5d17a7ec998e9911894f04f854266fa4e1f481","unresolved":false,"context_lines":[{"line_number":451,"context_line":"            # differentiating value."},{"line_number":452,"context_line":"            for acl in neutron_acls:"},{"line_number":453,"context_line":"                acl_visits +\u003d 1"},{"line_number":454,"context_line":"                if acl[\u0027port_group\u0027] !\u003d pg[\u0027name\u0027]:"},{"line_number":455,"context_line":"                    continue"},{"line_number":456,"context_line":"                # skip acls used by a different network log"},{"line_number":457,"context_line":"                n_acl_name \u003d acl[\u0027name\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"2015ccda_b5e0ca7b","line":454,"in_reply_to":"5af69e67_b30681e2","updated":"2026-09-26 14:02:54.000000000","message":"Done in patch set 2.","commit_id":"f6193c59cdf71495604f0b09ec59c2f3afc1cc7f"}],"neutron/tests/unit/services/logapi/drivers/ovn/test_driver.py":[{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"a430bf4a309aafc7d072a99c5be864aa76813f28","unresolved":true,"context_lines":[{"line_number":386,"context_line":"        get_sg.return_value \u003d ("},{"line_number":387,"context_line":"            fake_resources.FakeSecurityGroup.create_one_security_group("},{"line_number":388,"context_line":"                attrs\u003d{\u0027stateful\u0027: True}))"},{"line_number":389,"context_line":"        n_acls \u003d ["},{"line_number":390,"context_line":"            {\u0027port_group\u0027: pg[\u0027name\u0027],"},{"line_number":391,"context_line":"             \u0027priority\u0027: 1002,"},{"line_number":392,"context_line":"             \u0027action\u0027: ovn_const.ACL_ACTION_ALLOW_RELATED,"},{"line_number":393,"context_line":"             \u0027log\u0027: False,"},{"line_number":394,"context_line":"             \u0027name\u0027: \u0027\u0027,"},{"line_number":395,"context_line":"             \u0027severity\u0027: \u0027\u0027,"},{"line_number":396,"context_line":"             \u0027direction\u0027: \u0027from-lport\u0027,"},{"line_number":397,"context_line":"             \u0027match\u0027: f\u0027inport \u003d\u003d @{pg[\"name\"]} \u0026\u0026 ip4\u0027},"},{"line_number":398,"context_line":"            {\u0027port_group\u0027: other_pg[\u0027name\u0027],"},{"line_number":399,"context_line":"             \u0027priority\u0027: 1002,"},{"line_number":400,"context_line":"             \u0027action\u0027: ovn_const.ACL_ACTION_ALLOW_RELATED,"},{"line_number":401,"context_line":"             \u0027log\u0027: False,"},{"line_number":402,"context_line":"             \u0027name\u0027: \u0027\u0027,"},{"line_number":403,"context_line":"             \u0027severity\u0027: \u0027\u0027,"},{"line_number":404,"context_line":"             \u0027direction\u0027: \u0027from-lport\u0027,"},{"line_number":405,"context_line":"             \u0027match\u0027: f\u0027inport \u003d\u003d @{other_pg[\"name\"]} \u0026\u0026 ip4\u0027},"},{"line_number":406,"context_line":"        ]"},{"line_number":407,"context_line":"        other_acl \u003d n_acls[1].copy()"},{"line_number":408,"context_line":"        sg_id \u003d pg[\u0027external_ids\u0027][ovn_const.OVN_SG_EXT_ID_KEY]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1d77a308_acf94b8a","line":405,"range":{"start_line":389,"start_character":0,"end_line":405,"end_character":63},"updated":"2026-09-26 12:00:03.000000000","message":"nit: This will pass but for correctness, you can also add meter which is a property for ACLs too","commit_id":"f38716fc3fb4603096cfb2e5aa5949b2f80d0e1e"},{"author":{"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},"change_message_id":"5a5d17a7ec998e9911894f04f854266fa4e1f481","unresolved":false,"context_lines":[{"line_number":402,"context_line":"             \u0027log\u0027: False,"},{"line_number":403,"context_line":"             \u0027name\u0027: \u0027\u0027,"},{"line_number":404,"context_line":"             \u0027meter\u0027: [],"},{"line_number":405,"context_line":"             \u0027severity\u0027: \u0027\u0027,"},{"line_number":406,"context_line":"             \u0027direction\u0027: \u0027from-lport\u0027,"},{"line_number":407,"context_line":"             \u0027match\u0027: f\u0027inport \u003d\u003d @{other_pg[\"name\"]} \u0026\u0026 ip4\u0027},"},{"line_number":408,"context_line":"        ]"}],"source_content_type":"text/x-python","patch_set":2,"id":"f6938b18_ed763807","line":405,"in_reply_to":"1d77a308_acf94b8a","updated":"2026-09-26 14:02:54.000000000","message":"Done in patch set 2.","commit_id":"f6193c59cdf71495604f0b09ec59c2f3afc1cc7f"}]}