)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"13662593f053d9091071315ced991896a200d216","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6d28d57c_91b30a74","updated":"2026-09-14 20:08:24.000000000","message":"This misclassifies a plain VIF port too: normal ports have LSP.type \u003d\u003d \u0027\u0027, and \u0027\u0027 in \u0027external\u0027 is True. So _parent_port_handled_by_ovn() returns False for every ordinary trunk parent port, not just external ones — this disables trunk management broadly rather than fixing the baremetal-specific case. Should be:\n\n```\n    if not pport or pport.type \u003d\u003d ovn_const.LSP_TYPE_EXTERNAL:\n        return False\n```","commit_id":"dee2316fdeef58d6510c85fb4578087df8468fb2"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"2345705b6fd8c0029dc5acf5640e4370ba92dc30","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cce2eddf_923c9e84","updated":"2026-09-14 20:07:17.000000000","message":"Well so much for trying to write myself a CLI for writing gerrit comments...","commit_id":"dee2316fdeef58d6510c85fb4578087df8468fb2"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"897726d1e43e63c12f30e636014be52657260bde","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"13a2bfc1_f104006f","in_reply_to":"6d28d57c_91b30a74","updated":"2026-09-15 12:45:00.000000000","message":"That\u0027s an error that should have been catch in the tests. That means the tests do not cover that.\n\nThanks for the review.","commit_id":"dee2316fdeef58d6510c85fb4578087df8468fb2"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"3b3715211f2c168488c6d4857a940abf84cd0d78","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0327f59b_f891b39e","in_reply_to":"cce2eddf_923c9e84","updated":"2026-09-15 22:27:38.000000000","message":"Done","commit_id":"dee2316fdeef58d6510c85fb4578087df8468fb2"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"1120c7606a200b06cfbc8422fa2b8fc4ea5658b2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"53377ae1_09763741","updated":"2026-09-15 22:27:29.000000000","message":"Will this be backported?","commit_id":"4b19e7edb2dac83d4a411ff3129caa308c98b5c7"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c6937f7362dae1a22c5ea04036af28365828f8c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7cd26b9c_70096524","updated":"2026-09-16 07:23:16.000000000","message":"looks ok, thanks","commit_id":"4b19e7edb2dac83d4a411ff3129caa308c98b5c7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"38460f9e21cd01df71fc64175c6e7b1ea281e380","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8f058599_e3e862bb","in_reply_to":"53377ae1_09763741","updated":"2026-09-16 07:17:44.000000000","message":"Yes, this is indeed an issue affecting all stable branches. Once merged, I\u0027ll propose all backports.","commit_id":"4b19e7edb2dac83d4a411ff3129caa308c98b5c7"}],"neutron/services/trunk/drivers/ovn/trunk_driver.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c6937f7362dae1a22c5ea04036af28365828f8c6","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        LOG.debug(\"Done unsetting parent for subport %s\", subport.port_id)"},{"line_number":159,"context_line":"        return db_port"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"    def _parent_port_handled_by_ovn(self, pport_id):"},{"line_number":162,"context_line":"        \"\"\"Check if the parent port is managed by OVN."},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"        This method determines whether OVN is responsible for managing"}],"source_content_type":"text/x-python","patch_set":2,"id":"2606e94b_e1c0d378","line":161,"range":{"start_line":161,"start_character":8,"end_line":161,"end_character":35},"updated":"2026-09-16 07:23:16.000000000","message":"nt: is_parent_port_handled...","commit_id":"4b19e7edb2dac83d4a411ff3129caa308c98b5c7"}],"neutron/tests/unit/services/trunk/drivers/ovn/test_trunk_driver.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"44f02f9db6b6cd342049bef4a2bdb19123273f22","unresolved":false,"context_lines":[{"line_number":450,"context_line":"        are managed out-of-band. OVN should not handle trunking for them."},{"line_number":451,"context_line":"        \"\"\""},{"line_number":452,"context_line":"        lsp_row \u003d mock.Mock()"},{"line_number":453,"context_line":"        lsp_row.type \u003d ovn_const.LSP_TYPE_EXTERNAL"},{"line_number":454,"context_line":"        self.plugin_driver.nb_ovn.lookup.return_value \u003d lsp_row"},{"line_number":455,"context_line":"        result \u003d self.handler._parent_port_handled_by_ovn(\u0027test-port-id\u0027)"},{"line_number":456,"context_line":"        self.assertFalse(result)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f1dcc00_667f5646","line":453,"updated":"2026-09-14 08:46:07.000000000","message":"pep8: F821 undefined name \u0027ovn_const\u0027","commit_id":"dee2316fdeef58d6510c85fb4578087df8468fb2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"44f02f9db6b6cd342049bef4a2bdb19123273f22","unresolved":false,"context_lines":[{"line_number":472,"context_line":"        the trunk, as L2 forwarding happens outside OVN."},{"line_number":473,"context_line":"        \"\"\""},{"line_number":474,"context_line":"        lsp_row \u003d mock.Mock()"},{"line_number":475,"context_line":"        lsp_row.type \u003d ovn_const.LSP_TYPE_EXTERNAL"},{"line_number":476,"context_line":"        self.plugin_driver.nb_ovn.lookup.return_value \u003d lsp_row"},{"line_number":477,"context_line":"        fake_payload \u003d self._fake_trunk_event_payload()"},{"line_number":478,"context_line":"        self.handler.trunk_created("}],"source_content_type":"text/x-python","patch_set":1,"id":"ea7261a3_83841665","line":475,"updated":"2026-09-14 08:46:07.000000000","message":"pep8: F821 undefined name \u0027ovn_const\u0027","commit_id":"dee2316fdeef58d6510c85fb4578087df8468fb2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"44f02f9db6b6cd342049bef4a2bdb19123273f22","unresolved":false,"context_lines":[{"line_number":489,"context_line":"        is external."},{"line_number":490,"context_line":"        \"\"\""},{"line_number":491,"context_line":"        lsp_row \u003d mock.Mock()"},{"line_number":492,"context_line":"        lsp_row.type \u003d ovn_const.LSP_TYPE_EXTERNAL"},{"line_number":493,"context_line":"        self.plugin_driver.nb_ovn.lookup.return_value \u003d lsp_row"},{"line_number":494,"context_line":"        fake_payload \u003d self._fake_subport_event_payload()"},{"line_number":495,"context_line":"        self.handler.subports_added("}],"source_content_type":"text/x-python","patch_set":1,"id":"230b4d04_678b87f9","line":492,"updated":"2026-09-14 08:46:07.000000000","message":"pep8: F821 undefined name \u0027ovn_const\u0027","commit_id":"dee2316fdeef58d6510c85fb4578087df8468fb2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"44f02f9db6b6cd342049bef4a2bdb19123273f22","unresolved":false,"context_lines":[{"line_number":506,"context_line":"        port is external."},{"line_number":507,"context_line":"        \"\"\""},{"line_number":508,"context_line":"        lsp_row \u003d mock.Mock()"},{"line_number":509,"context_line":"        lsp_row.type \u003d ovn_const.LSP_TYPE_EXTERNAL"},{"line_number":510,"context_line":"        self.plugin_driver.nb_ovn.lookup.return_value \u003d lsp_row"},{"line_number":511,"context_line":"        fake_payload \u003d self._fake_subport_event_payload()"},{"line_number":512,"context_line":"        self.handler.subports_deleted("}],"source_content_type":"text/x-python","patch_set":1,"id":"85df5b10_dcb28b59","line":509,"updated":"2026-09-14 08:46:07.000000000","message":"pep8: F821 undefined name \u0027ovn_const\u0027","commit_id":"dee2316fdeef58d6510c85fb4578087df8468fb2"}]}