)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"dfaf524d10cfaca4e32971872cd3ff940d68c4c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6516c9a2_8920129e","updated":"2024-02-26 11:44:51.000000000","message":"Thanks!","commit_id":"981df944168367a8a0753d8e6fb21f9d21a56f88"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/api.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ad0e787fb2637f75ad077f93987c98b581f2d2e2","unresolved":true,"context_lines":[{"line_number":92,"context_line":"        \"\"\""},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"    @abc.abstractmethod"},{"line_number":95,"context_line":"    def add_lrouter_port(self, name, lrouter, may_exist\u003dTrue,"},{"line_number":96,"context_line":"                         **columns):"},{"line_number":97,"context_line":"        \"\"\"Create a command to add an OVN lrouter port"},{"line_number":98,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fe6d3d3f_c5f91611","line":95,"updated":"2024-02-21 23:52:14.000000000","message":"I guess this and delete_lrouter_port could be done as well? In a separate patch of course. And probably others as well.","commit_id":"41d500aa2d3a6ac5918acf8b40f3aa8ad4624f44"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f1c3bd3d05d3f622b6df704563cd193127cb04e0","unresolved":false,"context_lines":[{"line_number":92,"context_line":"        \"\"\""},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"    @abc.abstractmethod"},{"line_number":95,"context_line":"    def add_lrouter_port(self, name, lrouter, may_exist\u003dTrue,"},{"line_number":96,"context_line":"                         **columns):"},{"line_number":97,"context_line":"        \"\"\"Create a command to add an OVN lrouter port"},{"line_number":98,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"195c1994_a003bc85","line":95,"in_reply_to":"fe6d3d3f_c5f91611","updated":"2024-02-22 09:24:38.000000000","message":"The ovsdbapp LrpAddCommand is more complex than the one implemented in Neutron. But yes, we should delete any duplicated code and use ovsdbapp instead, if possible.","commit_id":"41d500aa2d3a6ac5918acf8b40f3aa8ad4624f44"}],"neutron/tests/unit/services/ovn_l3/test_plugin.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ad0e787fb2637f75ad077f93987c98b581f2d2e2","unresolved":true,"context_lines":[{"line_number":664,"context_line":"                                            self, payload)"},{"line_number":665,"context_line":""},{"line_number":666,"context_line":"        self.l3_inst._nb_ovn.delete_lrouter.assert_called_once_with("},{"line_number":667,"context_line":"            \u0027neutron-router-id\u0027)"},{"line_number":668,"context_line":""},{"line_number":669,"context_line":"    @mock.patch(\u0027neutron.plugins.ml2.drivers.ovn.mech_driver.ovsdb.ovn_client\u0027"},{"line_number":670,"context_line":"                \u0027.OVNClient._get_router_ports\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"cb910a9b_a535f89c","line":667,"updated":"2024-02-21 23:52:14.000000000","message":"Needs something similar to above.","commit_id":"41d500aa2d3a6ac5918acf8b40f3aa8ad4624f44"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f1c3bd3d05d3f622b6df704563cd193127cb04e0","unresolved":false,"context_lines":[{"line_number":664,"context_line":"                                            self, payload)"},{"line_number":665,"context_line":""},{"line_number":666,"context_line":"        self.l3_inst._nb_ovn.delete_lrouter.assert_called_once_with("},{"line_number":667,"context_line":"            \u0027neutron-router-id\u0027)"},{"line_number":668,"context_line":""},{"line_number":669,"context_line":"    @mock.patch(\u0027neutron.plugins.ml2.drivers.ovn.mech_driver.ovsdb.ovn_client\u0027"},{"line_number":670,"context_line":"                \u0027.OVNClient._get_router_ports\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"83a8aa58_b6112ffa","line":667,"in_reply_to":"cb910a9b_a535f89c","updated":"2024-02-22 09:24:38.000000000","message":"Done","commit_id":"41d500aa2d3a6ac5918acf8b40f3aa8ad4624f44"}]}