)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"69ce80f7a7c8c419925c4602f35893fd26bd0619","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"40427a19_92052933","updated":"2025-12-02 19:13:30.000000000","message":"Should also add a unit test, I *think* it should be easy enough looking at the existing ones.","commit_id":"5213d07f23cf5bcb50afe5827738f1646fcd2889"}],"neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"bc7a18503e05411e224ad4d69113f5ae60653ff4","unresolved":true,"context_lines":[{"line_number":2756,"context_line":"            self.dvr_agent.reset_dvr_flows("},{"line_number":2757,"context_line":"                self.int_br, self.tun_br, self.phys_brs,"},{"line_number":2758,"context_line":"                self.patch_int_ofport, self.patch_tun_ofport)"},{"line_number":2759,"context_line":"        self.ext_manager.initialize("},{"line_number":2760,"context_line":"            self.connection, ovs_const.EXTENSION_DRIVER_TYPE, self.agent_api)"},{"line_number":2761,"context_line":"        # notify that OVS has restarted"},{"line_number":2762,"context_line":"        registry.publish("},{"line_number":2763,"context_line":"            callback_resources.AGENT,"}],"source_content_type":"text/x-python","patch_set":1,"id":"dd84ac20_5b3474e0","line":2760,"range":{"start_line":2759,"start_character":3,"end_line":2760,"end_character":77},"updated":"2025-11-19 08:24:13.000000000","message":"good catch, thanks","commit_id":"5213d07f23cf5bcb50afe5827738f1646fcd2889"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"inactive contributor"},"change_message_id":"78bc27c3abaa89772dfba340ae6977791d506814","unresolved":false,"context_lines":[{"line_number":2756,"context_line":"            self.dvr_agent.reset_dvr_flows("},{"line_number":2757,"context_line":"                self.int_br, self.tun_br, self.phys_brs,"},{"line_number":2758,"context_line":"                self.patch_int_ofport, self.patch_tun_ofport)"},{"line_number":2759,"context_line":"        self.ext_manager.initialize("},{"line_number":2760,"context_line":"            self.connection, ovs_const.EXTENSION_DRIVER_TYPE, self.agent_api)"},{"line_number":2761,"context_line":"        # notify that OVS has restarted"},{"line_number":2762,"context_line":"        registry.publish("},{"line_number":2763,"context_line":"            callback_resources.AGENT,"}],"source_content_type":"text/x-python","patch_set":1,"id":"6d5f1ac1_1a4de599","line":2760,"range":{"start_line":2759,"start_character":3,"end_line":2760,"end_character":77},"in_reply_to":"dd84ac20_5b3474e0","updated":"2025-12-03 13:06:44.000000000","message":"Acknowledged","commit_id":"5213d07f23cf5bcb50afe5827738f1646fcd2889"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4c3a0277ca27be93ad86955d680fdec76e8a8058","unresolved":true,"context_lines":[{"line_number":2756,"context_line":"            self.dvr_agent.reset_dvr_flows("},{"line_number":2757,"context_line":"                self.int_br, self.tun_br, self.phys_brs,"},{"line_number":2758,"context_line":"                self.patch_int_ofport, self.patch_tun_ofport)"},{"line_number":2759,"context_line":"        self.ext_manager.initialize("},{"line_number":2760,"context_line":"            self.connection, ovs_const.EXTENSION_DRIVER_TYPE, self.agent_api)"},{"line_number":2761,"context_line":"        # notify that OVS has restarted"},{"line_number":2762,"context_line":"        registry.publish("}],"source_content_type":"text/x-python","patch_set":2,"id":"e615d62e_7b44f1bb","line":2759,"range":{"start_line":2759,"start_character":8,"end_line":2759,"end_character":36},"updated":"2025-12-24 10:22:39.000000000","message":"The extension initialization does more than creating the needed OVN flows. Most of the extension also initialize the RPC clients, some of them load new drivers, instantiate local variables, etc. Do we really want to do this again?\n\nFor example, the DHCP extension is loading a os-ken app. If we call `initialize`, that will create another instance.\n\nThe metadata extension is instantiating a HAProxy process. That will create another instance.\n\nIf the issue is the missing flows, we maybe should think about adding a `OVSL2AgentExtension`, derived from `L2AgentExtension`, that implements a `install_flows` method.","commit_id":"22f395ebc2e155dbd7dc3ab5205f2c47da4dae97"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"inactive contributor"},"change_message_id":"70090e8810a2b83a38422e2de008a46cc93fba88","unresolved":true,"context_lines":[{"line_number":2756,"context_line":"            self.dvr_agent.reset_dvr_flows("},{"line_number":2757,"context_line":"                self.int_br, self.tun_br, self.phys_brs,"},{"line_number":2758,"context_line":"                self.patch_int_ofport, self.patch_tun_ofport)"},{"line_number":2759,"context_line":"        self.ext_manager.initialize("},{"line_number":2760,"context_line":"            self.connection, ovs_const.EXTENSION_DRIVER_TYPE, self.agent_api)"},{"line_number":2761,"context_line":"        # notify that OVS has restarted"},{"line_number":2762,"context_line":"        registry.publish("}],"source_content_type":"text/x-python","patch_set":2,"id":"f2ddbb5f_f74c32ce","line":2759,"range":{"start_line":2759,"start_character":8,"end_line":2759,"end_character":36},"in_reply_to":"e615d62e_7b44f1bb","updated":"2026-06-09 14:09:32.000000000","message":"It took me quite some time to get back to your comment, but I uploaded an alternative fix attempt to this bug:\n\nhttps://review.opendev.org/q/topic:%22bug/2131666%22\n\nWill be abandoning this one in favor of that.","commit_id":"22f395ebc2e155dbd7dc3ab5205f2c47da4dae97"}]}