[lvm-devel] main - tests: this test has race in it depending on kernel

Zdenek Kabelac zkabelac at sourceware.org
Tue Mar 23 20:36:07 UTC 2021


Gitweb:        https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=8df0a32abb840815c5d210cbdbd1483d483ee729
Commit:        8df0a32abb840815c5d210cbdbd1483d483ee729
Parent:        dd1f5f9a24fc906de7e6dd064ecdd8fdfa9eebda
Author:        Zdenek Kabelac <zkabelac at redhat.com>
AuthorDate:    Tue Mar 23 21:32:51 2021 +0100
Committer:     Zdenek Kabelac <zkabelac at redhat.com>
CommitterDate: Tue Mar 23 21:32:51 2021 +0100

tests: this test has race in it depending on kernel

Some kernel seems to keep 'lvextend' busy so long,
that actual resize already happens.

So ATM use 'should'  until something better is invented.
---
 test/shell/lvextend-raid.sh | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/test/shell/lvextend-raid.sh b/test/shell/lvextend-raid.sh
index 9370bf295..08b34d00c 100644
--- a/test/shell/lvextend-raid.sh
+++ b/test/shell/lvextend-raid.sh
@@ -63,7 +63,8 @@ aux delayzero_dev "$dev2"  0 50 "${sector}:"
 lvextend -y -L+${lvext}M $vg/$lv1
 if [ $PROGRESS -eq 1 ]
 then
-not check lv_field $vg/$lv1 sync_percent "100.00"
+# Even with delayed devices wre are catching races here.
+should not check lv_field $vg/$lv1 sync_percent "100.00"
 check lv_field $vg/$lv1 size "$(($lvsz)).00m" $vg/$lv1
 fi
 aux wait_for_sync $vg $lv1




More information about the lvm-devel mailing list