diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2019-11-08 13:12:31 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2019-11-08 15:12:51 +0100 |
commit | 0e3e974bffa4e9842127451072eab1646ac39a1a (patch) | |
tree | 6532f35dad72c45bd796f4799ef04d17aab6cc8b | |
parent | bbdcdc12b2240e00f6ab8ff105e954629412b234 (diff) | |
download | lvm2-0e3e974bffa4e9842127451072eab1646ac39a1a.tar.gz |
tests: skip unneeded status check
If 'remove' was succesful - we can break loop immediatelly.
-rw-r--r-- | test/shell/pvmove-restart.sh | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/test/shell/pvmove-restart.sh b/test/shell/pvmove-restart.sh index dcf4c3792..6a5ba589f 100644 --- a/test/shell/pvmove-restart.sh +++ b/test/shell/pvmove-restart.sh @@ -51,12 +51,11 @@ wait # First take down $lv1 then it's pvmove0 j=0 for i in $lv1 pvmove0 pvmove0_mimage_0 pvmove0_mimage_1 ; do - while dmsetup status "$vg-$i"; do - dmsetup remove "$vg-$i" || { - j=$(( j + 1 )) - test $j -le 100 || die "Cannot take down devices." - sleep .1; - } + while dmsetup status "$vg-$i" ; do + dmsetup remove "$vg-$i" && break + j=$(( j + 1 )) + test $j -le 100 || die "Cannot take down devices." + sleep .1; done done dmsetup table | grep $PREFIX |