diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2019-11-08 13:12:31 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2019-11-11 22:44:25 +0100 |
commit | ba313ac84a7bc27ce69d225dec1bfc80bcb20988 (patch) | |
tree | 2b81db422fbe61e765c715d4f18b5aad8f0126e8 | |
parent | 1cc13335998558b84750fc2ba25c8b9f7b053746 (diff) | |
download | lvm2-ba313ac84a7bc27ce69d225dec1bfc80bcb20988.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 115ce1499..16691878a 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 |