)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"88b3b3476ff2df3b1b31e63d1054ad9754cf083b","unresolved":true,"context_lines":[{"line_number":10,"context_line":"fail at destination, the possible casue can be fail to get"},{"line_number":11,"context_line":"instance network info or  block device info"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Conflicts:"},{"line_number":14,"context_line":"\tnova/tests/functional/integrated_helpers.py"},{"line_number":15,"context_line":"\tnova/tests/functional/regressions/test_bug_1628606.py"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"NOTE(auniyal): conflicts are due to the following changes that are not"},{"line_number":18,"context_line":"in Ussuri"},{"line_number":19,"context_line":"\t* Ie0852a89fc9423a92baa7c29a8806c0628cae220"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Related-Bug: #1628606"},{"line_number":22,"context_line":"Change-Id: I48dbe0aae8a3943fdde69cda1bd663d70ea0eb19"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"03ff7e8a_edfd241b","line":19,"range":{"start_line":13,"start_character":1,"end_line":19,"end_character":44},"updated":"2022-11-24 08:56:23.000000000","message":"this is no longer valid, you now depend on https://review.opendev.org/c/openstack/nova/+/865382/","commit_id":"3ad070947e97b157c0f42da04de0a43863e39334"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"f84d4fea1f609f90703459ccda06150ec8f8f176","unresolved":false,"context_lines":[{"line_number":10,"context_line":"fail at destination, the possible casue can be fail to get"},{"line_number":11,"context_line":"instance network info or  block device info"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Conflicts:"},{"line_number":14,"context_line":"\tnova/tests/functional/integrated_helpers.py"},{"line_number":15,"context_line":"\tnova/tests/functional/regressions/test_bug_1628606.py"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"NOTE(auniyal): conflicts are due to the following changes that are not"},{"line_number":18,"context_line":"in Ussuri"},{"line_number":19,"context_line":"\t* Ie0852a89fc9423a92baa7c29a8806c0628cae220"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Related-Bug: #1628606"},{"line_number":22,"context_line":"Change-Id: I48dbe0aae8a3943fdde69cda1bd663d70ea0eb19"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"2fa78f11_0ff72431","line":19,"range":{"start_line":13,"start_character":1,"end_line":19,"end_character":44},"in_reply_to":"03ff7e8a_edfd241b","updated":"2022-11-24 12:21:00.000000000","message":"Done","commit_id":"3ad070947e97b157c0f42da04de0a43863e39334"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"bcaa26ab5a0bb853a993c6abb15478bd35a6efbc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"bd1e19cf_42d92dd0","updated":"2022-11-15 14:05:06.000000000","message":"at least the pep8 fail seems legit","commit_id":"9ffce2897ae9498bde9d31916f98ed1e1ed9aa30"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0c855fe67102f48263f82fd3265d86af0f75485a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"8fd58c83_e02b7155","updated":"2022-11-29 16:55:47.000000000","message":"+1 because the ussuri version is still pending","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4d18cda9c65dee930ab71e853506d516c53ce98a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"a3e7fa07_2b8791cd","updated":"2022-12-08 09:16:01.000000000","message":"Soft -1 just because of the commit msg miss for the use of the mock package and the unnecessary comment in the code itself.","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"ac1f68ef9523ed661dad2fa4fcb77937dba44e69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"2620121e_e9ada91f","updated":"2023-01-25 04:21:01.000000000","message":"recheck failed because parent commit failed to merge","commit_id":"3969228eebac197fc9bbb700c5d6999e06dd71c5"}],"nova/tests/functional/integrated_helpers.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e921653b7de14f57bf1684b3f3664c2441250c97","unresolved":true,"context_lines":[{"line_number":381,"context_line":"        self.api.post_server_action("},{"line_number":382,"context_line":"            server[\u0027id\u0027],"},{"line_number":383,"context_line":"            {\u0027os-migrateLive\u0027: {\u0027host\u0027: None,"},{"line_number":384,"context_line":"                                \u0027block_migration\u0027: \u0027true\u0027,"},{"line_number":385,"context_line":"                                \u0027disk_over_commit\u0027: \u0027true\u0027}})"},{"line_number":386,"context_line":"        self._wait_for_state_change(self.api, server, server_expected_state)"},{"line_number":387,"context_line":"        self._wait_for_migration_status(server, [migration_expected_state])"}],"source_content_type":"text/x-python","patch_set":7,"id":"7d63710d_6ca520dc","line":384,"range":{"start_line":384,"start_character":52,"end_line":384,"end_character":56},"updated":"2022-11-23 10:46:45.000000000","message":"no we should keep this at auto \n\nit was intoduced in mitaka  when disk_over_commit was removed \nand passing true is considered deprecated\n\nhttps://docs.openstack.org/nova/latest/reference/api-microversion-history.html#maximum-in-mitaka","commit_id":"03805caf7a0c41ca02cb46a365de5a29fbe64682"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"f84d4fea1f609f90703459ccda06150ec8f8f176","unresolved":false,"context_lines":[{"line_number":381,"context_line":"        self.api.post_server_action("},{"line_number":382,"context_line":"            server[\u0027id\u0027],"},{"line_number":383,"context_line":"            {\u0027os-migrateLive\u0027: {\u0027host\u0027: None,"},{"line_number":384,"context_line":"                                \u0027block_migration\u0027: \u0027true\u0027,"},{"line_number":385,"context_line":"                                \u0027disk_over_commit\u0027: \u0027true\u0027}})"},{"line_number":386,"context_line":"        self._wait_for_state_change(self.api, server, server_expected_state)"},{"line_number":387,"context_line":"        self._wait_for_migration_status(server, [migration_expected_state])"}],"source_content_type":"text/x-python","patch_set":7,"id":"818277c9_e97b3c3a","line":384,"range":{"start_line":384,"start_character":52,"end_line":384,"end_character":56},"in_reply_to":"7d63710d_6ca520dc","updated":"2022-11-24 12:21:00.000000000","message":"Ack","commit_id":"03805caf7a0c41ca02cb46a365de5a29fbe64682"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"88b3b3476ff2df3b1b31e63d1054ad9754cf083b","unresolved":true,"context_lines":[{"line_number":382,"context_line":"            server[\u0027id\u0027],"},{"line_number":383,"context_line":"            {\u0027os-migrateLive\u0027: {\u0027host\u0027: None,"},{"line_number":384,"context_line":"                                \u0027block_migration\u0027: \u0027auto\u0027}})"},{"line_number":385,"context_line":"        self._wait_for_state_change(self.api, server, server_expected_state)"},{"line_number":386,"context_line":"        self._wait_for_migration_status(server, [migration_expected_state])"},{"line_number":387,"context_line":"        return server"},{"line_number":388,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"80cc2091_f221c72c","line":385,"updated":"2022-11-24 08:56:23.000000000","message":"the logic is different from https://review.opendev.org/c/openstack/nova/+/864006/2/nova/tests/functional/integrated_helpers.py\n\nI don\u0027t get why you changed it if your git cherry-pick hadn\u0027t got you any merge conflict.\n\nPlease keep the exact same and just do a git cherry-pick -x \u003cussuri_sha1\u003e.","commit_id":"3ad070947e97b157c0f42da04de0a43863e39334"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"f84d4fea1f609f90703459ccda06150ec8f8f176","unresolved":false,"context_lines":[{"line_number":382,"context_line":"            server[\u0027id\u0027],"},{"line_number":383,"context_line":"            {\u0027os-migrateLive\u0027: {\u0027host\u0027: None,"},{"line_number":384,"context_line":"                                \u0027block_migration\u0027: \u0027auto\u0027}})"},{"line_number":385,"context_line":"        self._wait_for_state_change(self.api, server, server_expected_state)"},{"line_number":386,"context_line":"        self._wait_for_migration_status(server, [migration_expected_state])"},{"line_number":387,"context_line":"        return server"},{"line_number":388,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"361a6951_8efe25de","line":385,"in_reply_to":"80cc2091_f221c72c","updated":"2022-11-24 12:21:00.000000000","message":"Done","commit_id":"3ad070947e97b157c0f42da04de0a43863e39334"}],"nova/tests/functional/regressions/test_bug_1628606.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d3d39eed36b13372ba3ea52471d7d6780decd847","unresolved":true,"context_lines":[{"line_number":43,"context_line":"        self.api \u003d api_fixture.api"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"        # self.api \u003d api_fixture.admin_api"},{"line_number":46,"context_line":"        # self.api.microversion \u003d \u0027latest\u0027"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        self.src \u003d self._start_compute(host\u003d\u0027host1\u0027)"},{"line_number":49,"context_line":"        self.dest \u003d self._start_compute(host\u003d\u0027host2\u0027)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1dec5f38_f232afd4","line":46,"range":{"start_line":46,"start_character":9,"end_line":46,"end_character":42},"updated":"2022-11-23 12:02:00.000000000","message":"you forgot to uncomment this\n\nthat is why you reverted to passing block_migration","commit_id":"cb6fed53e25cb5403ff32311f402c4d915a050ac"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"f84d4fea1f609f90703459ccda06150ec8f8f176","unresolved":false,"context_lines":[{"line_number":43,"context_line":"        self.api \u003d api_fixture.api"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"        # self.api \u003d api_fixture.admin_api"},{"line_number":46,"context_line":"        # self.api.microversion \u003d \u0027latest\u0027"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        self.src \u003d self._start_compute(host\u003d\u0027host1\u0027)"},{"line_number":49,"context_line":"        self.dest \u003d self._start_compute(host\u003d\u0027host2\u0027)"}],"source_content_type":"text/x-python","patch_set":9,"id":"3ab5ec59_50e688d3","line":46,"range":{"start_line":46,"start_character":9,"end_line":46,"end_character":42},"in_reply_to":"1dec5f38_f232afd4","updated":"2022-11-24 12:21:00.000000000","message":"yes, setting api microversion fixed this","commit_id":"cb6fed53e25cb5403ff32311f402c4d915a050ac"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0c855fe67102f48263f82fd3265d86af0f75485a","unresolved":true,"context_lines":[{"line_number":10,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"# under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"import mock"},{"line_number":14,"context_line":"from nova import test"},{"line_number":15,"context_line":"from nova.tests import fixtures as nova_fixtures"},{"line_number":16,"context_line":"from nova.tests.functional.api import client"}],"source_content_type":"text/x-python","patch_set":15,"id":"55cf018b_7b35e461","line":13,"updated":"2022-11-29 16:55:47.000000000","message":"your change the import order.\nbut that proably fine as long as pep8 is happy.","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2e1bfeec84a2ca7be8646db96f459e85b4936df8","unresolved":false,"context_lines":[{"line_number":10,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"# under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"import mock"},{"line_number":14,"context_line":"from nova import test"},{"line_number":15,"context_line":"from nova.tests import fixtures as nova_fixtures"},{"line_number":16,"context_line":"from nova.tests.functional.api import client"}],"source_content_type":"text/x-python","patch_set":15,"id":"73ce6ddc_54a2b2b5","line":13,"in_reply_to":"17fe187c_db8f00c7","updated":"2023-01-24 11:06:45.000000000","message":"Ack","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4d18cda9c65dee930ab71e853506d516c53ce98a","unresolved":true,"context_lines":[{"line_number":10,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"# under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"import mock"},{"line_number":14,"context_line":"from nova import test"},{"line_number":15,"context_line":"from nova.tests import fixtures as nova_fixtures"},{"line_number":16,"context_line":"from nova.tests.functional.api import client"}],"source_content_type":"text/x-python","patch_set":15,"id":"17fe187c_db8f00c7","line":13,"in_reply_to":"55cf018b_7b35e461","updated":"2022-12-08 09:16:01.000000000","message":"that\u0027s because unittest only added the mock module post-train, right ? (so we need to import the whole mock package)\n\nAs Train supports py2, then unittest in py2 doesn\u0027t ship it. You could document it in the commit msg as this is a backport-specific modification.","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0c855fe67102f48263f82fd3265d86af0f75485a","unresolved":true,"context_lines":[{"line_number":25,"context_line":"    \"\"\""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def setUp(self):"},{"line_number":28,"context_line":"        super(PostLiveMigrationFail, self).setUp()"},{"line_number":29,"context_line":"        self.useFixture(nova_fixtures.NeutronFixture(self))"},{"line_number":30,"context_line":"        fake.stub_out_image_service(self)"},{"line_number":31,"context_line":"        self.useFixture(func_fixtures.PlacementFixture())"}],"source_content_type":"text/x-python","patch_set":15,"id":"013e391a_f9376f4a","line":28,"range":{"start_line":28,"start_character":7,"end_line":28,"end_character":50},"updated":"2022-11-29 16:55:47.000000000","message":"this is just the python 2 compatiable supper call so that is expected","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4d18cda9c65dee930ab71e853506d516c53ce98a","unresolved":true,"context_lines":[{"line_number":25,"context_line":"    \"\"\""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def setUp(self):"},{"line_number":28,"context_line":"        super(PostLiveMigrationFail, self).setUp()"},{"line_number":29,"context_line":"        self.useFixture(nova_fixtures.NeutronFixture(self))"},{"line_number":30,"context_line":"        fake.stub_out_image_service(self)"},{"line_number":31,"context_line":"        self.useFixture(func_fixtures.PlacementFixture())"}],"source_content_type":"text/x-python","patch_set":15,"id":"ef70ddba_439c4f9f","line":28,"range":{"start_line":28,"start_character":7,"end_line":28,"end_character":50},"in_reply_to":"013e391a_f9376f4a","updated":"2022-12-08 09:16:01.000000000","message":"correct","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2e1bfeec84a2ca7be8646db96f459e85b4936df8","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    \"\"\""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def setUp(self):"},{"line_number":28,"context_line":"        super(PostLiveMigrationFail, self).setUp()"},{"line_number":29,"context_line":"        self.useFixture(nova_fixtures.NeutronFixture(self))"},{"line_number":30,"context_line":"        fake.stub_out_image_service(self)"},{"line_number":31,"context_line":"        self.useFixture(func_fixtures.PlacementFixture())"}],"source_content_type":"text/x-python","patch_set":15,"id":"d07f6044_76ef97d5","line":28,"range":{"start_line":28,"start_character":7,"end_line":28,"end_character":50},"in_reply_to":"ef70ddba_439c4f9f","updated":"2023-01-24 11:06:45.000000000","message":"Ack","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0c855fe67102f48263f82fd3265d86af0f75485a","unresolved":true,"context_lines":[{"line_number":37,"context_line":"        api_fixture \u003d self.useFixture(nova_fixtures.OSAPIFixture("},{"line_number":38,"context_line":"            api_version\u003d\u0027v2.1\u0027))"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        # The admin API is used to get the server details to verify the"},{"line_number":41,"context_line":"        # host on which the server was built."},{"line_number":42,"context_line":"        self.admin_api \u003d api_fixture.admin_api"},{"line_number":43,"context_line":"        self.api \u003d api_fixture.api"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"0920e349_dc2d045d","line":41,"range":{"start_line":40,"start_character":5,"end_line":41,"end_character":45},"updated":"2022-11-29 16:55:47.000000000","message":"ok so you added this comment","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4d18cda9c65dee930ab71e853506d516c53ce98a","unresolved":true,"context_lines":[{"line_number":37,"context_line":"        api_fixture \u003d self.useFixture(nova_fixtures.OSAPIFixture("},{"line_number":38,"context_line":"            api_version\u003d\u0027v2.1\u0027))"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        # The admin API is used to get the server details to verify the"},{"line_number":41,"context_line":"        # host on which the server was built."},{"line_number":42,"context_line":"        self.admin_api \u003d api_fixture.admin_api"},{"line_number":43,"context_line":"        self.api \u003d api_fixture.api"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"12d3502b_ff36e2af","line":41,"range":{"start_line":40,"start_character":5,"end_line":41,"end_character":45},"in_reply_to":"0920e349_dc2d045d","updated":"2022-12-08 09:16:01.000000000","message":"yup, I don\u0027t see why we need it here","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2e1bfeec84a2ca7be8646db96f459e85b4936df8","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        api_fixture \u003d self.useFixture(nova_fixtures.OSAPIFixture("},{"line_number":38,"context_line":"            api_version\u003d\u0027v2.1\u0027))"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        # The admin API is used to get the server details to verify the"},{"line_number":41,"context_line":"        # host on which the server was built."},{"line_number":42,"context_line":"        self.admin_api \u003d api_fixture.admin_api"},{"line_number":43,"context_line":"        self.api \u003d api_fixture.api"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"aa3595db_1e219a27","line":41,"range":{"start_line":40,"start_character":5,"end_line":41,"end_character":45},"in_reply_to":"12d3502b_ff36e2af","updated":"2023-01-24 11:06:45.000000000","message":"Done","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0c855fe67102f48263f82fd3265d86af0f75485a","unresolved":true,"context_lines":[{"line_number":42,"context_line":"        self.admin_api \u003d api_fixture.admin_api"},{"line_number":43,"context_line":"        self.api \u003d api_fixture.api"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"        # retrieve image already so we can set api.microversion as latest"},{"line_number":46,"context_line":"        self.image_id \u003d self.api.get_images()[0][\u0027id\u0027]"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        self.admin_api.microversion \u003d \u0027latest\u0027"},{"line_number":49,"context_line":"        self.api.microversion \u003d \u0027latest\u0027"}],"source_content_type":"text/x-python","patch_set":15,"id":"1d82952d_b41ffc9b","line":46,"range":{"start_line":45,"start_character":8,"end_line":46,"end_character":54},"updated":"2022-11-29 16:55:47.000000000","message":"and looked up the image id","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4d18cda9c65dee930ab71e853506d516c53ce98a","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        self.admin_api \u003d api_fixture.admin_api"},{"line_number":43,"context_line":"        self.api \u003d api_fixture.api"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"        # retrieve image already so we can set api.microversion as latest"},{"line_number":46,"context_line":"        self.image_id \u003d self.api.get_images()[0][\u0027id\u0027]"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        self.admin_api.microversion \u003d \u0027latest\u0027"},{"line_number":49,"context_line":"        self.api.microversion \u003d \u0027latest\u0027"}],"source_content_type":"text/x-python","patch_set":15,"id":"1a8796dd_e2c5a464","line":46,"range":{"start_line":45,"start_character":8,"end_line":46,"end_character":54},"in_reply_to":"1d82952d_b41ffc9b","updated":"2022-12-08 09:16:01.000000000","message":"this is needed because of the use of _build_minimal_create_server_request() which requires an image_id as a param. This is explained in the commit msg, so I\u0027m good with this backport-specific modification.","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0c855fe67102f48263f82fd3265d86af0f75485a","unresolved":true,"context_lines":[{"line_number":55,"context_line":"        \u0027nova.compute.manager.ComputeManager\u0027"},{"line_number":56,"context_line":"        \u0027._post_live_migration_remove_source_vol_connections\u0027)"},{"line_number":57,"context_line":"    def test_post_live_migration(self, mock_migration):"},{"line_number":58,"context_line":"        server \u003d self.admin_api.post_server("},{"line_number":59,"context_line":"            dict(server\u003dself._build_minimal_create_server_request("},{"line_number":60,"context_line":"                self.api, \u0027test_post_live_migration\u0027, self.image_id,"},{"line_number":61,"context_line":"                networks\u003d\u0027none\u0027)))"},{"line_number":62,"context_line":"        server \u003d self._wait_for_state_change(self.admin_api, server, \u0027ACTIVE\u0027)"},{"line_number":63,"context_line":"        self.assertEqual(self.src.host, server[\u0027OS-EXT-SRV-ATTR:host\u0027])"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        error \u003d client.OpenStackApiException("}],"source_content_type":"text/x-python","patch_set":15,"id":"95173933_ee3b969b","line":62,"range":{"start_line":58,"start_character":7,"end_line":62,"end_character":78},"updated":"2022-11-29 16:55:47.000000000","message":"this is the replacement for the lack fo _create_server\n\nand this is equivalent so ok","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4d18cda9c65dee930ab71e853506d516c53ce98a","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        \u0027nova.compute.manager.ComputeManager\u0027"},{"line_number":56,"context_line":"        \u0027._post_live_migration_remove_source_vol_connections\u0027)"},{"line_number":57,"context_line":"    def test_post_live_migration(self, mock_migration):"},{"line_number":58,"context_line":"        server \u003d self.admin_api.post_server("},{"line_number":59,"context_line":"            dict(server\u003dself._build_minimal_create_server_request("},{"line_number":60,"context_line":"                self.api, \u0027test_post_live_migration\u0027, self.image_id,"},{"line_number":61,"context_line":"                networks\u003d\u0027none\u0027)))"},{"line_number":62,"context_line":"        server \u003d self._wait_for_state_change(self.admin_api, server, \u0027ACTIVE\u0027)"},{"line_number":63,"context_line":"        self.assertEqual(self.src.host, server[\u0027OS-EXT-SRV-ATTR:host\u0027])"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        error \u003d client.OpenStackApiException("}],"source_content_type":"text/x-python","patch_set":15,"id":"c80c7748_c4b08ddc","line":62,"range":{"start_line":58,"start_character":7,"end_line":62,"end_character":78},"in_reply_to":"95173933_ee3b969b","updated":"2022-12-08 09:16:01.000000000","message":"Yup, explained in the commit msg.","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0c855fe67102f48263f82fd3265d86af0f75485a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":15,"id":"ca337178_3f5fa3e5","line":75,"updated":"2022-11-29 16:55:47.000000000","message":"and the rest is the same","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4d18cda9c65dee930ab71e853506d516c53ce98a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"0a821050_eddd54aa","line":75,"in_reply_to":"ca337178_3f5fa3e5","updated":"2022-12-08 09:16:01.000000000","message":"Ack","commit_id":"c2de41f617109e396acfcc50aa44d6ca7fcf40da"}]}