)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"fe5a0d6b798700f6945cf09d413eb87012ed10ba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"76c94c3c_96c2c9f1","updated":"2022-11-15 14:05:27.000000000","message":"parent patch has test failures","commit_id":"926ed0fbc5e8113136575ea8290cf1e4353e441a"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"f07abdb2d8ad0192abb345ba1beb034710ce5a3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"d4e8e51a_5ff50ee5","updated":"2022-11-24 16:40:44.000000000","message":"The nova-multi-cell issue is independent from this patch. But the unites test failures are relevant here.","commit_id":"e4e9476cdf07b12af149096608cf1930a560d77e"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"e9bce2376948f3a09273ba7671d7ab582b9a614f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"73c9fe39_e94f8103","updated":"2022-11-27 07:27:52.000000000","message":"recheck nova multi cell","commit_id":"39782a9a4eb99b7e958fd868c7f356a5c82e131a"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"0681e53d196b243e253c5867b509cdc04011e56c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"858e9ab0_e8868f0e","updated":"2022-11-25 10:33:28.000000000","message":"recheck nova multi cell failed for tempest test volume swap\n","commit_id":"39782a9a4eb99b7e958fd868c7f356a5c82e131a"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"63054c51e22b15e1fbd6f1ba2acaf0c238837e3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"dc687223_9b03f998","updated":"2022-11-25 15:23:03.000000000","message":"recheck nova multi cell failed for tempest test volume swap\n","commit_id":"39782a9a4eb99b7e958fd868c7f356a5c82e131a"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"35bb613ae35c5fd78450cdbe7c3780d4fcb6506f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"e20df75b_ef51828e","updated":"2022-11-25 08:47:18.000000000","message":"recheck nova tempest test for volume swap","commit_id":"39782a9a4eb99b7e958fd868c7f356a5c82e131a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"532db782e05b8eb69a952c78488a7f5b7311f7b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"10280e81_104234cc","updated":"2022-11-29 17:36:39.000000000","message":"+1 since the ussuri version is still pending but this looks ok\n\nyou called out the change in the commit and over all the code looks mostly the same so this is a pretty clean backport.","commit_id":"5c1df0baa9435a9bab27eca0f04c08b48e3b452c"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"938525c20842aaee0c279cbed0c18a5b2e1e0025","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"27c7ab11_c2583734","updated":"2022-11-29 04:46:02.000000000","message":"recheck nova tempest test volume swap failed (unreleated)","commit_id":"5c1df0baa9435a9bab27eca0f04c08b48e3b452c"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"5b1a0d676e5b2c97bc9991fc38d8c0d62f30794e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"8608153e_090fc86d","updated":"2023-01-16 09:59:02.000000000","message":"Thanks for having modified the commit msg.","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3e8f1957468944cc86e145017e3dfba80ef57390","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"95c22b47_220ca1cd","updated":"2023-01-24 11:08:51.000000000","message":"ok this should be good to proceed","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"c70c6470c8df194f611ccbcd87d6902952c32e6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"d3880b40_f648e148","updated":"2023-03-07 13:19:37.000000000","message":"recheck - Volume failed to become in-use","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"2e8afa41511690d447d2e7e1aae3bfaa04c39180","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"f23c26cb_ca46752c","updated":"2023-03-14 14:47:35.000000000","message":"recheck - volume e22daf2e-403f-4e19-a021-4ad5c3b96d57 failed to reach in-use status (current available) within the required time (196 s).","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"456ad618dac6b7ea413ac71a6922f8f7ef0e7f8c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"a52ac89f_5466eda5","updated":"2023-01-25 04:22:22.000000000","message":"recheck failed because parent commit failed to merge","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"cf50f0ac899439f934d84ebf4fcf2d26aba3d7ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"fa3603fc_3bd7eb3f","updated":"2023-03-05 10:03:45.000000000","message":"recheck known gate issue - test_volume_swap failed ","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"56a036ef5044ef9dac003a06b2e0cf662ed77fe2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"f8575662_2eff63c7","updated":"2023-03-03 05:32:09.000000000","message":"recheck known test_volume_swap failed with timeout","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"cbc0084014a68469cd6907fc24c2e4fc8258f242","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"b54a5983_1aa27da3","updated":"2023-04-03 07:12:51.000000000","message":"recheck live migration failed (unrelated)","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"a9412e99e4bee98a697bd48c2361a4632140601b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"3d8d9d53_318b6a49","updated":"2023-03-02 10:50:30.000000000","message":"recheck related patch merged","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"7b10be3534d1695a25b4b2b4181c553ead85c708","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"5c71638b_7e403b91","updated":"2023-03-05 14:37:32.000000000","message":"recheck timeout","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"d9d8e5cc93b8e86ec5efa4ff6b76598b8a0d7495","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"8d110676_be7ea405","updated":"2023-03-12 09:36:43.000000000","message":"recheck unrelated failed","commit_id":"ec31d4d22e4163a37f2a63f387b2614189d77ff9"}],"nova/tests/functional/regressions/test_bug_1628606.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"dd7ed2b9b15916553c78aaeeab38347aa9fcdfaa","unresolved":true,"context_lines":[{"line_number":70,"context_line":"            server, migration_expected_state\u003d\u0027error\u0027,"},{"line_number":71,"context_line":"            server_expected_state\u003d\u0027ERROR\u0027)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        self.assertEqual(self.dest.host, server[\u0027OS-EXT-SRV-ATTR:host\u0027])"}],"source_content_type":"text/x-python","patch_set":12,"id":"93d5b149_559d5728","line":73,"updated":"2022-11-24 06:46:57.000000000","message":"as expected control went through the second exception of _post_live_migration_update_host.\n\nso value of instance.host was also updated to destination before raising the final exception.\nstill test failed, because in integrated_helpers.InstanceHelperMixin._live_migrate control is not waiting for migrate_state to be set as error.","commit_id":"2c3ac5fda8a98109353680e18cd3c3664ac00854"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"b2f791ecaa32b4157fe2c0092f74806489ccbed8","unresolved":false,"context_lines":[{"line_number":70,"context_line":"            server, migration_expected_state\u003d\u0027error\u0027,"},{"line_number":71,"context_line":"            server_expected_state\u003d\u0027ERROR\u0027)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        self.assertEqual(self.dest.host, server[\u0027OS-EXT-SRV-ATTR:host\u0027])"}],"source_content_type":"text/x-python","patch_set":12,"id":"62dfe20c_cda69073","line":73,"in_reply_to":"93d5b149_559d5728","updated":"2022-11-24 14:08:03.000000000","message":"resolved","commit_id":"2c3ac5fda8a98109353680e18cd3c3664ac00854"}],"nova/tests/unit/compute/test_compute_mgr.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"dd7ed2b9b15916553c78aaeeab38347aa9fcdfaa","unresolved":true,"context_lines":[{"line_number":9381,"context_line":"                self.assertEqual(instance.host, dest_host)"},{"line_number":9382,"context_line":""},{"line_number":9383,"context_line":"        _test_post_live_migration()"},{"line_number":9384,"context_line":""},{"line_number":9385,"context_line":"    def test_post_live_migration_cinder_pre_344_api(self):"},{"line_number":9386,"context_line":"        # Because live migration has"},{"line_number":9387,"context_line":"        # succeeded,_post_live_migration_remove_source_vol_connections()"}],"source_content_type":"text/x-python","patch_set":12,"id":"b306507b_d388d7e1","line":9384,"updated":"2022-11-24 06:46:57.000000000","message":"raise Exception(\u0027This test uses methods that set internal oslo_db \u0027\n    Exception: This test uses methods that set internal oslo_db state, but it does not claim to use the database. This will conflict with the setup of tests that do use the database and cause failures later.\n    \n\nCaptured pythonlogging:\n~~~~~~~~~~~~~~~~~~~~~~~\n    2022-11-24 06:27:38,529 INFO [nova.virt.driver] Loading compute driver \u0027fake.SmallFakeDriver\u0027","commit_id":"2c3ac5fda8a98109353680e18cd3c3664ac00854"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"f59e906bafad5383b32afb93412ec7b63d275727","unresolved":true,"context_lines":[{"line_number":9381,"context_line":"                self.assertEqual(instance.host, dest_host)"},{"line_number":9382,"context_line":""},{"line_number":9383,"context_line":"        _test_post_live_migration()"},{"line_number":9384,"context_line":""},{"line_number":9385,"context_line":"    def test_post_live_migration_cinder_pre_344_api(self):"},{"line_number":9386,"context_line":"        # Because live migration has"},{"line_number":9387,"context_line":"        # succeeded,_post_live_migration_remove_source_vol_connections()"}],"source_content_type":"text/x-python","patch_set":12,"id":"a877dc86_d59ef168","line":9384,"in_reply_to":"73bf325e_f218a4ee","updated":"2022-11-24 18:35:16.000000000","message":"This test runs without any DB so every call that would go the the DB is poisioned to stop the test cleanly. \n\nFile \"/home/zuul/src/opendev.org/openstack/nova/nova/objects/instance.py\", line 969, in _load_migration_context\n    self._context, self.uuid))\n  File \"/home/zuul/src/opendev.org/openstack/nova/.tox/py37/lib/python3.7/site-packages/oslo_versionedobjects/base.py\", line 184, in wrapper\n    result \u003d fn(cls, context, *args, **kwargs)\n  File \"/home/zuul/src/opendev.org/openstack/nova/nova/objects/migration_context.py\", line 85, in get_by_instance_uuid\n    context, instance_uuid, columns\u003d[\u0027migration_context\u0027])\n  File \"/home/zuul/src/opendev.org/openstack/nova/nova/db/api.py\", line 969, in instance_extra_get_by_instance_uuid\n  \nThis part of the stack trace shows that the code tries to load migration_context from the DB. Nova has this lazy loading behavior where if a field of an object is not defined when it is accessed then nova tries to load it from the DB. In this test you don\u0027t have DB to load from, so you need to make sure that the instance object has the migration_context field defined. In ussuri, the fake_instance.fake_instance_obj() call returns an instance that has the migration_context field defined, in train it does not. This is why your test fails.\n\nI suggest to add\n  inst.migration_context \u003d None\nto fake_instance.fake_instance_obj the same way is that function has it in ussuri. That will make your test pass.","commit_id":"2c3ac5fda8a98109353680e18cd3c3664ac00854"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"35bb613ae35c5fd78450cdbe7c3780d4fcb6506f","unresolved":true,"context_lines":[{"line_number":9381,"context_line":"                self.assertEqual(instance.host, dest_host)"},{"line_number":9382,"context_line":""},{"line_number":9383,"context_line":"        _test_post_live_migration()"},{"line_number":9384,"context_line":""},{"line_number":9385,"context_line":"    def test_post_live_migration_cinder_pre_344_api(self):"},{"line_number":9386,"context_line":"        # Because live migration has"},{"line_number":9387,"context_line":"        # succeeded,_post_live_migration_remove_source_vol_connections()"}],"source_content_type":"text/x-python","patch_set":12,"id":"ff3890ce_de31f99a","line":9384,"in_reply_to":"a877dc86_d59ef168","updated":"2022-11-25 08:47:18.000000000","message":"Thanks Gibi","commit_id":"2c3ac5fda8a98109353680e18cd3c3664ac00854"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"b2f791ecaa32b4157fe2c0092f74806489ccbed8","unresolved":true,"context_lines":[{"line_number":9381,"context_line":"                self.assertEqual(instance.host, dest_host)"},{"line_number":9382,"context_line":""},{"line_number":9383,"context_line":"        _test_post_live_migration()"},{"line_number":9384,"context_line":""},{"line_number":9385,"context_line":"    def test_post_live_migration_cinder_pre_344_api(self):"},{"line_number":9386,"context_line":"        # Because live migration has"},{"line_number":9387,"context_line":"        # succeeded,_post_live_migration_remove_source_vol_connections()"}],"source_content_type":"text/x-python","patch_set":12,"id":"73bf325e_f218a4ee","line":9384,"in_reply_to":"b306507b_d388d7e1","updated":"2022-11-24 14:08:03.000000000","message":"unit tc failed\nhttps://zuul.opendev.org/t/openstack/build/fe5bf43b09cb42b4a0ebfb3104905a1c","commit_id":"2c3ac5fda8a98109353680e18cd3c3664ac00854"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"532db782e05b8eb69a952c78488a7f5b7311f7b9","unresolved":false,"context_lines":[{"line_number":9381,"context_line":"                self.assertEqual(instance.host, dest_host)"},{"line_number":9382,"context_line":""},{"line_number":9383,"context_line":"        _test_post_live_migration()"},{"line_number":9384,"context_line":""},{"line_number":9385,"context_line":"    def test_post_live_migration_cinder_pre_344_api(self):"},{"line_number":9386,"context_line":"        # Because live migration has"},{"line_number":9387,"context_line":"        # succeeded,_post_live_migration_remove_source_vol_connections()"}],"source_content_type":"text/x-python","patch_set":12,"id":"c3243e61_6b99f033","line":9384,"in_reply_to":"ff3890ce_de31f99a","updated":"2022-11-29 17:36:39.000000000","message":"Done","commit_id":"2c3ac5fda8a98109353680e18cd3c3664ac00854"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"532db782e05b8eb69a952c78488a7f5b7311f7b9","unresolved":true,"context_lines":[{"line_number":9370,"context_line":"            instance \u003d fake_instance.fake_instance_obj(self.context,"},{"line_number":9371,"context_line":"                                                        node\u003d\u0027src\u0027,"},{"line_number":9372,"context_line":"                                                        uuid\u003duuids.instance)"},{"line_number":9373,"context_line":"            instance.migration_context \u003d None"},{"line_number":9374,"context_line":"            with mock.patch.object(self.compute, \"_post_live_migration\""},{"line_number":9375,"context_line":"                    ) as plm, mock.patch.object(instance, \"save\") as save:"},{"line_number":9376,"context_line":"                error \u003d ValueError(\"some failure\")"}],"source_content_type":"text/x-python","patch_set":18,"id":"95f40cdb_550a45b9","line":9373,"range":{"start_line":9373,"start_character":11,"end_line":9373,"end_character":45},"updated":"2022-11-29 17:36:39.000000000","message":"ok so this is the main delta to make the test pass.","commit_id":"5c1df0baa9435a9bab27eca0f04c08b48e3b452c"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"cbc0084014a68469cd6907fc24c2e4fc8258f242","unresolved":false,"context_lines":[{"line_number":9370,"context_line":"            instance \u003d fake_instance.fake_instance_obj(self.context,"},{"line_number":9371,"context_line":"                                                        node\u003d\u0027src\u0027,"},{"line_number":9372,"context_line":"                                                        uuid\u003duuids.instance)"},{"line_number":9373,"context_line":"            instance.migration_context \u003d None"},{"line_number":9374,"context_line":"            with mock.patch.object(self.compute, \"_post_live_migration\""},{"line_number":9375,"context_line":"                    ) as plm, mock.patch.object(instance, \"save\") as save:"},{"line_number":9376,"context_line":"                error \u003d ValueError(\"some failure\")"}],"source_content_type":"text/x-python","patch_set":18,"id":"77b62d5f_1d59533c","line":9373,"range":{"start_line":9373,"start_character":11,"end_line":9373,"end_character":45},"in_reply_to":"95f40cdb_550a45b9","updated":"2023-04-03 07:12:51.000000000","message":"Done","commit_id":"5c1df0baa9435a9bab27eca0f04c08b48e3b452c"}]}