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 |