)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b606183e73afb43ff0dfa0eb3af0e299b8dafd18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"05cdad6b_ef8c996e","updated":"2026-06-12 14:09:43.000000000","message":"Hi Seyeong, thanks for the patch.","commit_id":"bece66b7c56abe0ca28bbbb6ae67c0ed5499c7ee"},{"author":{"_account_id":5112,"name":"Seyeong Kim","email":"seyeong.kim@canonical.com","username":"xtrusia"},"change_message_id":"c1d865cc9117de478350437624f8d155a188512a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0a8941a1_1efff85a","updated":"2026-06-15 23:15:49.000000000","message":"recheck","commit_id":"92181b39e793e917483b9a9536bc6925961060cb"}],"neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b606183e73afb43ff0dfa0eb3af0e299b8dafd18","unresolved":true,"context_lines":[{"line_number":518,"context_line":"                    self.available_local_vlans.remove(local_vlan)"},{"line_number":519,"context_line":"                    # Restore the br-tun flood output ports"},{"line_number":520,"context_line":"                    # See LP #1978088"},{"line_number":521,"context_line":"                    tun_ofports \u003d None"},{"line_number":522,"context_line":"                    if self.tun_br is not None:"},{"line_number":523,"context_line":"                        tun_ofports \u003d self.tun_br.get_flood_to_tun_ofports("},{"line_number":524,"context_line":"                            local_vlan)"}],"source_content_type":"text/x-python","patch_set":1,"id":"904f5c35_9fccbe4b","line":521,"updated":"2026-06-12 14:09:43.000000000","message":"get_flood_to_tun_ofports() will return an empty set if it finds no actions, this should be the same.","commit_id":"bece66b7c56abe0ca28bbbb6ae67c0ed5499c7ee"},{"author":{"_account_id":5112,"name":"Seyeong Kim","email":"seyeong.kim@canonical.com","username":"xtrusia"},"change_message_id":"ca65e5075406e07debe52f96e62038dc70bc5bc3","unresolved":false,"context_lines":[{"line_number":518,"context_line":"                    self.available_local_vlans.remove(local_vlan)"},{"line_number":519,"context_line":"                    # Restore the br-tun flood output ports"},{"line_number":520,"context_line":"                    # See LP #1978088"},{"line_number":521,"context_line":"                    tun_ofports \u003d None"},{"line_number":522,"context_line":"                    if self.tun_br is not None:"},{"line_number":523,"context_line":"                        tun_ofports \u003d self.tun_br.get_flood_to_tun_ofports("},{"line_number":524,"context_line":"                            local_vlan)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a3a58cf8_a59ca255","line":521,"in_reply_to":"904f5c35_9fccbe4b","updated":"2026-06-15 01:05:17.000000000","message":"Thanks for the review! I modified it.","commit_id":"bece66b7c56abe0ca28bbbb6ae67c0ed5499c7ee"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b606183e73afb43ff0dfa0eb3af0e299b8dafd18","unresolved":true,"context_lines":[{"line_number":519,"context_line":"                    # Restore the br-tun flood output ports"},{"line_number":520,"context_line":"                    # See LP #1978088"},{"line_number":521,"context_line":"                    tun_ofports \u003d None"},{"line_number":522,"context_line":"                    if self.tun_br is not None:"},{"line_number":523,"context_line":"                        tun_ofports \u003d self.tun_br.get_flood_to_tun_ofports("},{"line_number":524,"context_line":"                            local_vlan)"},{"line_number":525,"context_line":"                    self._local_vlan_hints[key] \u003d {"}],"source_content_type":"text/x-python","patch_set":1,"id":"d8b64007_0fade7e2","line":522,"updated":"2026-06-12 14:09:43.000000000","message":"So the other places here that use self.tun_br.* are all under this check:\n\n    if self.enable_tunneling:\n\nI think maybe this should be the same.","commit_id":"bece66b7c56abe0ca28bbbb6ae67c0ed5499c7ee"},{"author":{"_account_id":5112,"name":"Seyeong Kim","email":"seyeong.kim@canonical.com","username":"xtrusia"},"change_message_id":"ca65e5075406e07debe52f96e62038dc70bc5bc3","unresolved":false,"context_lines":[{"line_number":519,"context_line":"                    # Restore the br-tun flood output ports"},{"line_number":520,"context_line":"                    # See LP #1978088"},{"line_number":521,"context_line":"                    tun_ofports \u003d None"},{"line_number":522,"context_line":"                    if self.tun_br is not None:"},{"line_number":523,"context_line":"                        tun_ofports \u003d self.tun_br.get_flood_to_tun_ofports("},{"line_number":524,"context_line":"                            local_vlan)"},{"line_number":525,"context_line":"                    self._local_vlan_hints[key] \u003d {"}],"source_content_type":"text/x-python","patch_set":1,"id":"cc8f2cb8_3aafbbce","line":522,"in_reply_to":"d8b64007_0fade7e2","updated":"2026-06-15 01:05:17.000000000","message":"Thanks!","commit_id":"bece66b7c56abe0ca28bbbb6ae67c0ed5499c7ee"}],"neutron/tests/unit/plugins/ml2/drivers/openvswitch/agent/test_ovs_neutron_agent.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"3b642983d899762f34e11892c7bf9d0b7121ee2d","unresolved":false,"context_lines":[{"line_number":335,"context_line":"                                        \u0027get_flood_to_tun_ofports\u0027,"},{"line_number":336,"context_line":"                                        return_value\u003dtun_ofports)"},{"line_number":337,"context_line":"            expected_tun_ofports \u003d tun_ofports"},{"line_number":338,"context_line":"        with mock.patch.object(self.agent.int_br,"},{"line_number":339,"context_line":"                               \u0027get_ports_attributes\u0027,"},{"line_number":340,"context_line":"                               side_effect\u003d[get_interfaces,"},{"line_number":341,"context_line":"                                   get_ports]) as gpa, tun_ctx:"}],"source_content_type":"text/x-python","patch_set":2,"id":"929ef196_4204be69","line":338,"updated":"2026-06-15 02:45:06.000000000","message":"pep8: W0124: Following \"as\" with another context manager looks like a tuple. (confusing-with-statement)","commit_id":"27e1f707006c56436b812410ceba253fecbb88c9"}]}