qa: use safe_while() to avoid racy checks in test_subvolume_metrics_lifecycle()

Signed-off-by: Venky Shankar <vshankar@redhat.com>

diff --git a/qa/tasks/cephfs/test_subvolume.py b/qa/tasks/cephfs/test_subvolume.py
index 08680615af153b23e87f12e653695d98da86ddee..bbb7f0db9b9f3cdeb4d8e8ce87df0ebc5e2f3962 100644 (file)
--- a/qa/tasks/cephfs/test_subvolume.py
+++ b/qa/tasks/cephfs/test_subvolume.py
@@ -411,12 +411,12 @@ class TestSubvolumeMetrics(CephFSTestCase):
         self.assertEqual(counters_after_delete["quota_bytes"], quota_bytes,
                          "Quota should remain unchanged after deletions")
 
-        # wait for metrics to expire after inactivity
-        sleep(30)
-
         # verify that metrics are not present anymore
-        subvolume_metrics = self.get_subvolume_metrics()
-        self.assertFalse(subvolume_metrics, "Subvolume metrics should be gone after inactivity window")
+        with safe_while(sleep=1, tries=60, action='wait for empty subvolume metrics after inactivity window') as proceed:
+            while proceed():
+                subvolume_metrics = self.get_subvolume_metrics()
+                if not subvolume_metrics:
+                    break
 
     def test_subvolume_quota_resize_update(self):
         """