)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"2d97545c7b083c9c000d7e13d55b539b738f76cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cb93659e_4a1aa67f","updated":"2022-06-23 12:33:58.000000000","message":"I have to go through the revert itself (quick look it looks good) but this should have a launchpad bug for tracking this - stable/train only issue xxx","commit_id":"51e2547051f4d688ec48bfcb9fcb4fbc16cbe2e8"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"da71eec7cec429e38470183ef84874d64931a533","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"62d8b872_b2983a3a","updated":"2022-06-23 12:42:22.000000000","message":"One note in tests, rest looks good comapred to original commit","commit_id":"51e2547051f4d688ec48bfcb9fcb4fbc16cbe2e8"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"b2aa12ecfaf68214fc0245358bb24d8eb06972eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"12323dd7_e9bd69b8","updated":"2022-06-23 11:54:17.000000000","message":"recheck networking-ovn-octavia-v2-dsvm-scenario timeout","commit_id":"51e2547051f4d688ec48bfcb9fcb4fbc16cbe2e8"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"ce95c1f907b40c7a13ee3f4d56ee52ea6fd3f00e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a6a803ef_0ca27fe8","updated":"2022-06-24 07:12:42.000000000","message":"LGTM now!","commit_id":"49bab190161eb85174476978447f782c2d156a6d"}],"networking_ovn/tests/unit/octavia/test_ovn_driver.py":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"da71eec7cec429e38470183ef84874d64931a533","unresolved":true,"context_lines":[{"line_number":509,"context_line":"        self.ref_member.subnet_id \u003d None"},{"line_number":510,"context_line":"        self.driver.member_batch_update([self.ref_member])"},{"line_number":511,"context_line":""},{"line_number":512,"context_line":"    def test_member_batch_update_missing_subnet_id_get_from_pool(self):"},{"line_number":513,"context_line":"        self.driver._ovn_helper._get_subnet_from_pool.return_value \u003d ("},{"line_number":514,"context_line":"            self.ref_member.subnet_id)"},{"line_number":515,"context_line":"        self.ref_member.subnet_id \u003d None"},{"line_number":516,"context_line":"        self.driver.member_batch_update([self.ref_member])"},{"line_number":517,"context_line":""},{"line_number":518,"context_line":"    def test_member_update_failure(self):"},{"line_number":519,"context_line":"        self.assertRaises(exceptions.UnsupportedOptionError,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f611ef2_bf0ddff6","line":516,"range":{"start_line":512,"start_character":0,"end_line":516,"end_character":58},"updated":"2022-06-23 12:42:22.000000000","message":"Duplicate of previous test","commit_id":"51e2547051f4d688ec48bfcb9fcb4fbc16cbe2e8"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"3c07df893cba73b6fc98d676da4931411eb95d57","unresolved":false,"context_lines":[{"line_number":509,"context_line":"        self.ref_member.subnet_id \u003d None"},{"line_number":510,"context_line":"        self.driver.member_batch_update([self.ref_member])"},{"line_number":511,"context_line":""},{"line_number":512,"context_line":"    def test_member_batch_update_missing_subnet_id_get_from_pool(self):"},{"line_number":513,"context_line":"        self.driver._ovn_helper._get_subnet_from_pool.return_value \u003d ("},{"line_number":514,"context_line":"            self.ref_member.subnet_id)"},{"line_number":515,"context_line":"        self.ref_member.subnet_id \u003d None"},{"line_number":516,"context_line":"        self.driver.member_batch_update([self.ref_member])"},{"line_number":517,"context_line":""},{"line_number":518,"context_line":"    def test_member_update_failure(self):"},{"line_number":519,"context_line":"        self.assertRaises(exceptions.UnsupportedOptionError,"}],"source_content_type":"text/x-python","patch_set":3,"id":"bac02e44_161df59d","line":516,"range":{"start_line":512,"start_character":0,"end_line":516,"end_character":58},"in_reply_to":"5f611ef2_bf0ddff6","updated":"2022-06-23 14:33:52.000000000","message":"Ack","commit_id":"51e2547051f4d688ec48bfcb9fcb4fbc16cbe2e8"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"ce95c1f907b40c7a13ee3f4d56ee52ea6fd3f00e","unresolved":true,"context_lines":[{"line_number":464,"context_line":"        self.driver.member_batch_update([self.ref_member, self.update_member])"},{"line_number":465,"context_line":"        self.assertEqual(self.mock_add_request.call_count, 3)"},{"line_number":466,"context_line":""},{"line_number":467,"context_line":"    def test_member_batch_update_no_members_exception(self):"},{"line_number":468,"context_line":"        self.assertRaises(exceptions.UnsupportedOptionError,"},{"line_number":469,"context_line":"                          self.driver.member_batch_update, [])"},{"line_number":470,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"83f2aeea_34c3fb55","line":467,"range":{"start_line":467,"start_character":8,"end_line":467,"end_character":53},"updated":"2022-06-24 07:12:42.000000000","message":"As discussed with froyo on IRC, this is the \"original\" test with this name that was removed in first patch. A later test was added with similar name (L476) so here we have this rename for the restored test, and keeping the newer test","commit_id":"49bab190161eb85174476978447f782c2d156a6d"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"8b6a346521fd72fd8a44660bcb9dfc7a22781ae8","unresolved":true,"context_lines":[{"line_number":464,"context_line":"        self.driver.member_batch_update([self.ref_member, self.update_member])"},{"line_number":465,"context_line":"        self.assertEqual(self.mock_add_request.call_count, 3)"},{"line_number":466,"context_line":""},{"line_number":467,"context_line":"    def test_member_batch_update_no_members_exception(self):"},{"line_number":468,"context_line":"        self.assertRaises(exceptions.UnsupportedOptionError,"},{"line_number":469,"context_line":"                          self.driver.member_batch_update, [])"},{"line_number":470,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"391dc02f_40810253","line":467,"range":{"start_line":467,"start_character":8,"end_line":467,"end_character":53},"in_reply_to":"83f2aeea_34c3fb55","updated":"2022-06-24 09:08:01.000000000","message":"yeah, thx for the clarification here 😊","commit_id":"49bab190161eb85174476978447f782c2d156a6d"}]}
