)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e84e549080640199e7654368aad1ea971adfaca7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"95b0151d_63d0ed97","updated":"2024-07-26 19:24:13.000000000","message":"Code looks ok but we need some testing","commit_id":"62b06816c66acb1137c43a2590e38f6468e3f3d1"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e84e549080640199e7654368aad1ea971adfaca7","unresolved":true,"context_lines":[{"line_number":586,"context_line":"        ports \u003d self.driver._plugin.get_ports("},{"line_number":587,"context_line":"            self.admin_context,"},{"line_number":588,"context_line":"            filters\u003d{\u0027id\u0027: [row.name]})"},{"line_number":589,"context_line":"        if len(ports) \u003e 0:"},{"line_number":590,"context_line":"            self.l3_plugin._ovn_client.update_router_port("},{"line_number":591,"context_line":"                self.admin_context,"},{"line_number":592,"context_line":"                ports[0])"}],"source_content_type":"text/x-python","patch_set":1,"id":"f0764aa0_403268c3","line":589,"range":{"start_line":589,"start_character":11,"end_line":589,"end_character":25},"updated":"2024-07-26 19:24:13.000000000","message":"nit: ``if ports``","commit_id":"62b06816c66acb1137c43a2590e38f6468e3f3d1"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5e766dfc53fa33051dc13803d66d1f66e5f4ee84","unresolved":true,"context_lines":[{"line_number":586,"context_line":"        ports \u003d self.driver._plugin.get_ports("},{"line_number":587,"context_line":"            self.admin_context,"},{"line_number":588,"context_line":"            filters\u003d{\u0027id\u0027: [row.name]})"},{"line_number":589,"context_line":"        if len(ports) \u003e 0:"},{"line_number":590,"context_line":"            self.l3_plugin._ovn_client.update_router_port("},{"line_number":591,"context_line":"                self.admin_context,"},{"line_number":592,"context_line":"                ports[0])"}],"source_content_type":"text/x-python","patch_set":1,"id":"fb9f2b43_d8741a8a","line":589,"range":{"start_line":589,"start_character":11,"end_line":589,"end_character":25},"in_reply_to":"f0764aa0_403268c3","updated":"2024-07-28 20:16:43.000000000","message":"Or is a try/except n_exc.PortNotFound simpler? Assuming it\u0027s the get_port() throwing the exception? That\u0027s used elsewhere in the OVN mech driver.","commit_id":"62b06816c66acb1137c43a2590e38f6468e3f3d1"}]}