)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a0f3d4bc5b88b15feec9da93c52002a764daba63","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7e928961_d1910f31","updated":"2026-08-05 02:53:41.000000000","message":"recheck bug 2160254, change lgtm just a question on \u0027fault\u0027 to be populated during wait","commit_id":"ab50f021ac9144fabb45394163dedf170afda55a"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"610c6aaa9ba4f3914035a14c1a0b1b94a8cc5a2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e96486e1_03f78264","updated":"2026-07-30 08:56:18.000000000","message":"recheck tox cover has another instability in test_revert_resize_dest_compute_graceful_shutdown\n```\n\nTraceback (most recent call last):\n  File \"/home/zuul/src/opendev.org/openstack/nova/nova/tests/functional/test_graceful_shutdown.py\", line 438, in test_revert_resize_dest_compute_graceful_shutdown\n    self.api.post_server_action(server[\u0027id\u0027], {\u0027migrate\u0027: None})\n  File \"/home/zuul/src/opendev.org/openstack/nova/nova/tests/functional/api/client.py\", line 273, in post_server_action\n    return self.api_post(\n           ^^^^^^^^^^^^^^\n  File \"/home/zuul/src/opendev.org/openstack/nova/nova/tests/functional/api/client.py\", line 215, in api_post\n    return APIResponse(self.api_request(relative_uri, **kwargs))\n                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n  File \"/home/zuul/src/opendev.org/openstack/nova/nova/tests/functional/api/client.py\", line 191, in api_request\n    raise OpenStackApiException(\nnova.tests.functional.api.client.OpenStackApiException: Unexpected status code: {\"conflictingRequest\": {\"code\": 409, \"message\": \"Service is unavailable at this time.\"}}\n\n```","commit_id":"ab50f021ac9144fabb45394163dedf170afda55a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a0f3d4bc5b88b15feec9da93c52002a764daba63","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2e1716a1_7e5df5fd","in_reply_to":"e96486e1_03f78264","updated":"2026-08-05 02:53:41.000000000","message":"yeah, I am seeing these failure in other changes also (bug#2160254), let me fix that tomorrow.","commit_id":"ab50f021ac9144fabb45394163dedf170afda55a"}],"nova/tests/functional/test_cross_cell_migrate.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a0f3d4bc5b88b15feec9da93c52002a764daba63","unresolved":true,"context_lines":[{"line_number":1109,"context_line":""},{"line_number":1110,"context_line":"        # The migration should be in \u0027error\u0027 status."},{"line_number":1111,"context_line":"        self._wait_for_migration_status(server, [\u0027error\u0027])"},{"line_number":1112,"context_line":"        server \u003d self.api.get_server(server[\u0027id\u0027])"},{"line_number":1113,"context_line":"        # Assert a fault was recorded."},{"line_number":1114,"context_line":"        self.assertIn(\u0027fault\u0027, server)"},{"line_number":1115,"context_line":"        self.assertIn(\u0027Connection to the hypervisor is broken\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"71b21020_f334a1e6","line":1112,"range":{"start_line":1112,"start_character":7,"end_line":1112,"end_character":50},"updated":"2026-08-05 02:53:41.000000000","message":"this is called in _wait_for_migration_status()[1]. any reason it did not populated the fault in server response during _wait_for_migration_status() itself? I commented in the previous change but I feel like _wait_for_migration_status() should populate/wait for fault to be in server response if \u0027error\u0027 state is asked to wait.\n\n[1] https://github.com/openstack/nova/blob/1a4601ef3a120d898feda81dc1db4de3d99df7f7/nova/tests/functional/integrated_helpers.py#L131","commit_id":"ab50f021ac9144fabb45394163dedf170afda55a"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"558e4e1d589bc711e635758436db603d4c29959c","unresolved":true,"context_lines":[{"line_number":1109,"context_line":""},{"line_number":1110,"context_line":"        # The migration should be in \u0027error\u0027 status."},{"line_number":1111,"context_line":"        self._wait_for_migration_status(server, [\u0027error\u0027])"},{"line_number":1112,"context_line":"        server \u003d self.api.get_server(server[\u0027id\u0027])"},{"line_number":1113,"context_line":"        # Assert a fault was recorded."},{"line_number":1114,"context_line":"        self.assertIn(\u0027fault\u0027, server)"},{"line_number":1115,"context_line":"        self.assertIn(\u0027Connection to the hypervisor is broken\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7366c894_06d9f112","line":1112,"range":{"start_line":1112,"start_character":7,"end_line":1112,"end_character":50},"in_reply_to":"71b21020_f334a1e6","updated":"2026-08-14 15:04:35.000000000","message":"fixed in the parent commit. But note that get_server is not called in _wait_for_migration_status","commit_id":"ab50f021ac9144fabb45394163dedf170afda55a"}]}