)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"3230b395825df88a1b131fbaf972dd97e9ae2d56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4d606dcc_08ee3ef8","updated":"2022-10-18 08:02:27.000000000","message":"LGTM","commit_id":"512b2c83b567eaecdfa2d96ecc1facc3723b4f55"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"8c99202b9865233a88cd0fc45178a8f6283b7c2b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"95850b62_a62a7fe9","updated":"2022-10-25 16:57:50.000000000","message":"recheck ovn-octavia-provider-tempest-release immutable loadbalancer","commit_id":"512b2c83b567eaecdfa2d96ecc1facc3723b4f55"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"4980d9faf8f6513249e1d4bcfd83ed52d3c8a3c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cd3ebd28_917c0fda","updated":"2022-10-26 05:42:09.000000000","message":"recheck ovn-octavia-provider-tempest-release immutable loadbalancer","commit_id":"512b2c83b567eaecdfa2d96ecc1facc3723b4f55"}],"ovn_octavia_provider/tests/unit/test_helper.py":[{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"1a4e251fc168491ae323ff2b5884828e4a2f1ed9","unresolved":true,"context_lines":[{"line_number":2219,"context_line":"            \u0027networks\u0027: [\"10.0.10.1/24\"]})"},{"line_number":2220,"context_line":"        self.router.ports.append(p2)"},{"line_number":2221,"context_line":"        self.router.ports.append(p1)"},{"line_number":2222,"context_line":"        res \u003d self.helper._find_ls_for_lr(self.router, 4)"},{"line_number":2223,"context_line":"        self.assertListEqual([\u0027neutron-foo1\u0027], res)"},{"line_number":2224,"context_line":""},{"line_number":2225,"context_line":"    def test__find_ls_for_lr_different_ip_version(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"f09681fc_11c31957","line":2222,"updated":"2022-10-17 09:01:32.000000000","message":"nit: You can use n_const.IP_VERSION_4 or n_const.IP_VERSION_6 for those values, they are available in neutron-lib","commit_id":"40e655041b977731d8764a26c7b02db04ac839e7"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"f7c2225774f6af4ac1344266c114e344bb7bb38e","unresolved":false,"context_lines":[{"line_number":2219,"context_line":"            \u0027networks\u0027: [\"10.0.10.1/24\"]})"},{"line_number":2220,"context_line":"        self.router.ports.append(p2)"},{"line_number":2221,"context_line":"        self.router.ports.append(p1)"},{"line_number":2222,"context_line":"        res \u003d self.helper._find_ls_for_lr(self.router, 4)"},{"line_number":2223,"context_line":"        self.assertListEqual([\u0027neutron-foo1\u0027], res)"},{"line_number":2224,"context_line":""},{"line_number":2225,"context_line":"    def test__find_ls_for_lr_different_ip_version(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"8306a85b_e7684b2b","line":2222,"in_reply_to":"f09681fc_11c31957","updated":"2022-10-18 06:29:29.000000000","message":"Done","commit_id":"40e655041b977731d8764a26c7b02db04ac839e7"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"1a4e251fc168491ae323ff2b5884828e4a2f1ed9","unresolved":true,"context_lines":[{"line_number":2236,"context_line":"        self.router.ports.append(p2)"},{"line_number":2237,"context_line":"        self.router.ports.append(p1)"},{"line_number":2238,"context_line":"        res \u003d self.helper._find_ls_for_lr(self.router, 4)"},{"line_number":2239,"context_line":"        self.assertListEqual([\u0027neutron-foo1\u0027], res)"},{"line_number":2240,"context_line":""},{"line_number":2241,"context_line":"    def test__find_ls_for_lr_gw_port(self):"},{"line_number":2242,"context_line":"        p1 \u003d fakes.FakeOVNPort.create_one_port(attrs\u003d{"}],"source_content_type":"text/x-python","patch_set":2,"id":"66f0a9e7_b1d45fca","line":2239,"updated":"2022-10-17 09:01:32.000000000","message":"You could add this portion of code to have the new changes to function _find_ls_for_lr 100% coverage:\n\nres \u003d self.helper._find_ls_for_lr(self.router, 6)\nself.assertListEqual([\u0027neutron-foo2\u0027], res)","commit_id":"40e655041b977731d8764a26c7b02db04ac839e7"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"f7c2225774f6af4ac1344266c114e344bb7bb38e","unresolved":false,"context_lines":[{"line_number":2236,"context_line":"        self.router.ports.append(p2)"},{"line_number":2237,"context_line":"        self.router.ports.append(p1)"},{"line_number":2238,"context_line":"        res \u003d self.helper._find_ls_for_lr(self.router, 4)"},{"line_number":2239,"context_line":"        self.assertListEqual([\u0027neutron-foo1\u0027], res)"},{"line_number":2240,"context_line":""},{"line_number":2241,"context_line":"    def test__find_ls_for_lr_gw_port(self):"},{"line_number":2242,"context_line":"        p1 \u003d fakes.FakeOVNPort.create_one_port(attrs\u003d{"}],"source_content_type":"text/x-python","patch_set":2,"id":"ce2c9c1d_6724c20f","line":2239,"in_reply_to":"66f0a9e7_b1d45fca","updated":"2022-10-18 06:29:29.000000000","message":"Done","commit_id":"40e655041b977731d8764a26c7b02db04ac839e7"}]}