From 5c1068488ee3a0204bf1155b0a4bd45b89fddae4 Mon Sep 17 00:00:00 2001 From: elajkat Date: Thu, 15 Apr 2021 17:45:34 +0200 Subject: [PATCH] Add waiter for test_live_migration_with_trunk Add waiter for parent_port status check Change-Id: Ia2b292d0e8bca016b8c81b2a41c35536a8a959bd Closes-Bug: #1924258 --- tempest/api/compute/admin/test_live_migration.py | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/tempest/api/compute/admin/test_live_migration.py b/tempest/api/compute/admin/test_live_migration.py index 73b795fdcc..c91b557784 100644 --- a/tempest/api/compute/admin/test_live_migration.py +++ b/tempest/api/compute/admin/test_live_migration.py @@ -263,8 +263,10 @@ class LiveMigrationTest(LiveMigrationTestBase): test_utils.call_until_true( self._is_port_status_active, CONF.validation.connect_timeout, 5, subport['id'])) - parent = self.ports_client.show_port(parent['id'])['port'] - self.assertEqual('ACTIVE', parent['status']) + self.assertTrue( + test_utils.call_until_true( + self._is_port_status_active, CONF.validation.connect_timeout, + 5, parent['id'])) subport = self.ports_client.show_port(subport['id'])['port'] if not CONF.compute_feature_enabled.can_migrate_between_any_hosts: @@ -280,8 +282,10 @@ class LiveMigrationTest(LiveMigrationTestBase): test_utils.call_until_true( self._is_port_status_active, CONF.validation.connect_timeout, 5, subport['id'])) - parent = self.ports_client.show_port(parent['id'])['port'] - self.assertEqual('ACTIVE', parent['status']) + self.assertTrue( + test_utils.call_until_true( + self._is_port_status_active, CONF.validation.connect_timeout, + 5, parent['id'])) class LiveMigrationRemoteConsolesV26Test(LiveMigrationTestBase):