From df41ea7467db3b40776030865896af0102129283 Mon Sep 17 00:00:00 2001 From: Kamoltat Date: Mon, 8 Feb 2021 15:45:06 +0000 Subject: [PATCH] qa/tasks/mgr/test_progress: fix wait_until_equal Octopus ceph_test_case doesn't have period arg so remove that in wait_until_equal. Also increase time to wait for complete events by using RECOVERY_PERIOD instead of EVENT_CREATION_PERIOD Not needed in masters because only octopus and nautilus doesn't have a period argument in qa/tasks/mgr/test_progress.py wait_until_equals() function Fixes: https://tracker.ceph.com/issues/48824 Signed-off-by: Kamoltat --- qa/tasks/mgr/test_progress.py | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/qa/tasks/mgr/test_progress.py b/qa/tasks/mgr/test_progress.py index 9ba549e6b7e3e..f309515bdf2ab 100644 --- a/qa/tasks/mgr/test_progress.py +++ b/qa/tasks/mgr/test_progress.py @@ -188,8 +188,7 @@ class TestProgress(MgrTestCase): # Wait for a progress event to pop up self.wait_until_equal(lambda: self._osd_in_out_events_count('out'), 1, - timeout=self.EVENT_CREATION_PERIOD*2, - period=1) + timeout=self.EVENT_CREATION_PERIOD*2) ev = self._get_osd_in_out_events('out')[0] log.info(json.dumps(ev, indent=1)) self.assertIn("Rebalancing after osd.0 marked out", ev['message']) @@ -204,13 +203,12 @@ class TestProgress(MgrTestCase): # First Event should complete promptly self.wait_until_true(lambda: self._is_complete(initial_event['id']), - timeout=self.EVENT_CREATION_PERIOD) + timeout=self.RECOVERY_PERIOD) try: # Wait for progress event marked in to pop up self.wait_until_equal(lambda: self._osd_in_out_events_count('in'), 1, - timeout=self.EVENT_CREATION_PERIOD*2, - period=1) + timeout=self.EVENT_CREATION_PERIOD*2) except RuntimeError as ex: if not "Timed out after" in str(ex): raise ex @@ -283,7 +281,7 @@ class TestProgress(MgrTestCase): # Event should complete promptly self.wait_until_true(lambda: self._is_complete(ev['id']), - timeout=self.EVENT_CREATION_PERIOD) + timeout=self.RECOVERY_PERIOD) self.assertTrue(self._is_quiet()) def test_osd_came_back(self): @@ -390,8 +388,7 @@ class TestProgress(MgrTestCase): # Wait for a progress event to pop up self.wait_until_equal(lambda: self._osd_in_out_events_count('out'), 1, - timeout=self.EVENT_CREATION_PERIOD*2, - period=1) + timeout=self.RECOVERY_PERIOD) ev1 = self._get_osd_in_out_events('out')[0] -- 2.39.5