From a159f162c52fb9d12da5bb05ad512e610e67cdee Mon Sep 17 00:00:00 2001 From: David Zafman Date: Thu, 7 Jun 2018 18:03:28 -0700 Subject: [PATCH] test: osd-scrub-snaps.sh: After snapshot removal wait for snaptrim to complete Due to deliberate corruptions snaptrim_error means snaptrim is done Signed-off-by: David Zafman --- qa/standalone/scrub/osd-scrub-snaps.sh | 40 +++++++++++++++++++++++--- 1 file changed, 36 insertions(+), 4 deletions(-) diff --git a/qa/standalone/scrub/osd-scrub-snaps.sh b/qa/standalone/scrub/osd-scrub-snaps.sh index e4856bdb8a2..0b7c70e0d13 100755 --- a/qa/standalone/scrub/osd-scrub-snaps.sh +++ b/qa/standalone/scrub/osd-scrub-snaps.sh @@ -688,12 +688,28 @@ EOF pids+="$(cat $pidfile) " done + ERRORS=0 + for i in `seq 1 7` do rados -p $poolname rmsnap snap$i done - - ERRORS=0 + sleep 5 + local -i loop=0 + while ceph pg dump pgs | grep -q snaptrim; + do + if ceph pg dump pgs | grep -q snaptrim_error; + then + break + fi + sleep 2 + loop+=1 + if (( $loop >= 10 )) ; then + ERRORS=$(expr $ERRORS + 1) + break + fi + done + ceph pg dump pgs for pid in $pids do @@ -1136,6 +1152,8 @@ fi pids+="$(cat $pidfile) " done + ERRORS=0 + # When removing snapshots with a corrupt replica, it crashes. # See http://tracker.ceph.com/issues/23875 if [ $which = "primary" ]; @@ -1144,9 +1162,23 @@ fi do rados -p $poolname rmsnap snap$i done + sleep 5 + local -i loop=0 + while ceph pg dump pgs | grep -q snaptrim; + do + if ceph pg dump pgs | grep -q snaptrim_error; + then + break + fi + sleep 2 + loop+=1 + if (( $loop >= 10 )) ; then + ERRORS=$(expr $ERRORS + 1) + break + fi + done fi - - ERRORS=0 + ceph pg dump pgs for pid in $pids do -- 2.39.5