From f205cb8af29e0cd1fc133691b1bde2f067790177 Mon Sep 17 00:00:00 2001 From: Corey Bryant Date: Thu, 28 Sep 2023 12:52:05 -0400 Subject: [PATCH] Add tenacity retries around ring sync check This is still seems to have a race even after ensuring the units are settled. --- zaza/openstack/charm_tests/swift/tests.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/zaza/openstack/charm_tests/swift/tests.py b/zaza/openstack/charm_tests/swift/tests.py index 605b275c1..3b862042a 100644 --- a/zaza/openstack/charm_tests/swift/tests.py +++ b/zaza/openstack/charm_tests/swift/tests.py @@ -132,9 +132,14 @@ def test_904_set_weight_action_and_validate_rebalance(self): zaza.model.wait_for_agent_status() zaza.model.block_until_all_units_idle() - self.assertTrue( - swift_utils.is_ring_synced('swift-proxy', 'object', - self.TEST_EXPECTED_RING_HOSTS)) + for attempt in tenacity.Retrying( + wait=tenacity.wait_fixed(2), + retry=tenacity.retry_if_exception_type(AssertionError), + reraise=True, + stop=tenacity.stop_after_attempt(10)): + self.assertTrue( + swift_utils.is_ring_synced('swift-proxy', 'object', + self.TEST_EXPECTED_RING_HOSTS)) def test_905_remove_device_action_and_validate_rebalance(self): """Remove device from object ring."""