From: Sridhar Seshasayee Date: Tue, 16 Mar 2021 19:48:40 +0000 (+0530) Subject: qa/tasks: Add additional wait_for_clean() check in lost_unfound tasks. X-Git-Tag: v16.2.0~88^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F40221%2Fhead;p=ceph.git qa/tasks: Add additional wait_for_clean() check in lost_unfound tasks. At the end of the lost_unfound tests add an additional wait_for_clean() check to ensure that recoveries get enough time to complete before proceeding and avoid failures down the line. For e.g. failure like "Scrubbing terminated -- not all pgs were active and clean." is because recoveries on the PGs did not get sufficient time to complete even though they were bound to eventually complete. Fixes: https://tracker.ceph.com/issues/49844 Signed-off-by: Sridhar Seshasayee (cherry picked from commit 88df47230b5ad85e95b0be2eca6f5763914b175c) --- diff --git a/qa/tasks/ec_lost_unfound.py b/qa/tasks/ec_lost_unfound.py index 71068692d0a4..57a9364eceb1 100644 --- a/qa/tasks/ec_lost_unfound.py +++ b/qa/tasks/ec_lost_unfound.py @@ -156,3 +156,4 @@ def task(ctx, config): manager.wait_till_osd_is_up(1) manager.wait_for_clean() run.wait(procs) + manager.wait_for_clean() diff --git a/qa/tasks/lost_unfound.py b/qa/tasks/lost_unfound.py index ec101b2041c3..5a9142a70a1b 100644 --- a/qa/tasks/lost_unfound.py +++ b/qa/tasks/lost_unfound.py @@ -177,3 +177,4 @@ def task(ctx, config): manager.wait_till_osd_is_up(1) manager.wait_for_clean() run.wait(procs) + manager.wait_for_clean() diff --git a/qa/tasks/rep_lost_unfound_delete.py b/qa/tasks/rep_lost_unfound_delete.py index 893f76ea195a..8e99ade27c16 100644 --- a/qa/tasks/rep_lost_unfound_delete.py +++ b/qa/tasks/rep_lost_unfound_delete.py @@ -175,4 +175,5 @@ def task(ctx, config): manager.wait_till_osd_is_up(1) manager.wait_for_clean() run.wait(procs) + manager.wait_for_clean()