shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-21 13:34:40 +03:00
Code Releases Activity

tests: ignore incosistent raid status

Browse Source 
Just like lvm command ignores  0/xxxx report from judging the status.
Avoid using infinite loop and limit report checking to 100 checks.
If it would need more - something is not right.
This commit is contained in:
Zdenek Kabelac 2021-03-24 12:08:40 +01:00
parent afd43a75f2
commit 9684e82cc4
1 changed files with 31 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

51
test/shell/lvconvert-raid-status-validation.sh
View File

@ -36,25 +36,28 @@ vgcreate $SHARED -s 2m "$vg" "${DEVICES[@]}"
########################################### ###########################################
# Upconverted RAID1 should never have all 'a's in status output # Upconverted RAID1 should never have all 'a's in status output
########################################### ###########################################
aux delay_dev "$dev2" 0 50 aux delay_dev "$dev2" 0 20
lvcreate -aey -l 2 -n $lv1 $vg "$dev1" lvcreate -aey -l 2 -n $lv1 $vg "$dev1"
lvconvert --type raid1 -y -m 1 $vg/$lv1 "$dev2" lvconvert --type raid1 -y -m 1 $vg/$lv1 "$dev2"
while ! check in_sync $vg $lv1; do for i in {100..0}; do
a=( $(dmsetup status $vg-$lv1) ) || die "Unable to get status of $vg/$lv1" check in_sync $vg $lv1 && break
a=( $(dmsetup status $vg-$lv1) ) || die "Unable to get status of $vg/$lv1"
b=( $(echo "${a[6]}" | sed s:/:' ':) ) b=( $(echo "${a[6]}" | sed s:/:' ':) )
if [ "${b[0]}" -ne "${b[1]}" ]; then if [ "${b[0]}" -ne "${b[1]}" ]; then
# First, 'check in_sync' should only need to check the ratio # First, 'check in_sync' should only need to check the ratio
# If we are here, it is probably doing more than that. # If we are here, it is probably doing more than that.
# If not in-sync, then we should only ever see "Aa" # If not in-sync, then we should only ever see "Aa"
[ "${a[5]}" == "Aa" ] # Ignore until raid starts to report consistent data
[ "${b[0]}" = "0" ] || [ "${a[5]}" == "Aa" ]
else else
[ "${a[5]}" != "aa" ] [ "${a[5]}" != "aa" ]
should [ "${a[5]}" == "AA" ] # RHBZ 1507719 should [ "${a[5]}" == "AA" ] # RHBZ 1507719
fi fi
sleep .1 sleep .1
done done
aux enable_dev "$dev2" aux enable_dev "$dev2"
lvremove -ff $vg lvremove -ff $vg
test "$i" -gt 0 || die "Unable to get in sync $vg/$lv1"
########################################### ###########################################
# Upconverted RAID1 should not be at 100% right after upconvert # Upconverted RAID1 should not be at 100% right after upconvert
@ -71,13 +74,15 @@ lvremove -ff $vg
########################################### ###########################################
# Catch anything suspicious with linear -> RAID1 upconvert # Catch anything suspicious with linear -> RAID1 upconvert
########################################### ###########################################
aux delay_dev "$dev2" 0 50 aux delay_dev "$dev2" 0 20
lvcreate -aey -l 2 -n $lv1 $vg "$dev1" lvcreate -aey -l 2 -n $lv1 $vg "$dev1"
lvconvert --type raid1 -y -m 1 $vg/$lv1 "$dev2" lvconvert --type raid1 -y -m 1 $vg/$lv1 "$dev2"
while true; do for i in {100..0}; do
a=( $(dmsetup status $vg-$lv1) ) || die "Unable to get status of $vg/$lv1" a=( $(dmsetup status $vg-$lv1) ) || die "Unable to get status of $vg/$lv1"
b=( $(echo "${a[6]}" | sed s:/:' ':) ) b=( $(echo "${a[6]}" | sed s:/:' ':) )
if [ "${b[0]}" -ne "${b[1]}" ]; then if [ "${b[0]}" -eq "0" ]; then
: # Ignore until raid starts to report consistent data
elif [ "${b[0]}" -ne "${b[1]}" ]; then
# If the sync operation ("recover" in this case) is not # If the sync operation ("recover" in this case) is not
# finished, then it better be as follows: # finished, then it better be as follows:
[ "${a[5]}" = "Aa" ] [ "${a[5]}" = "Aa" ]
@ -96,7 +101,7 @@ while true; do
should [ "${a[7]}" = "idle" ] # RHBZ 1507719 should [ "${a[7]}" = "idle" ] # RHBZ 1507719
break break
fi fi
sleep .1 sleep .1
done done
aux enable_dev "$dev2" aux enable_dev "$dev2"
lvremove -ff $vg lvremove -ff $vg
@ -104,14 +109,16 @@ lvremove -ff $vg
########################################### ###########################################
# Catch anything suspicious with RAID1 2-way -> 3-way upconvert # Catch anything suspicious with RAID1 2-way -> 3-way upconvert
########################################### ###########################################
aux delay_dev "$dev3" 0 50 aux delay_dev "$dev3" 0 20
lvcreate --type raid1 -m 1 -aey -l 2 -n $lv1 $vg "$dev1" "$dev2" lvcreate --type raid1 -m 1 -aey -l 2 -n $lv1 $vg "$dev1" "$dev2"
aux wait_for_sync $vg $lv1 aux wait_for_sync $vg $lv1
lvconvert -y -m +1 $vg/$lv1 "$dev3" lvconvert -y -m +1 $vg/$lv1 "$dev3"
while true; do for i in {100..0}; do
a=( $(dmsetup status $vg-$lv1) ) || die "Unable to get status of $vg/$lv1" a=( $(dmsetup status $vg-$lv1) ) || die "Unable to get status of $vg/$lv1"
b=( $(echo "${a[6]}" | sed s:/:' ':) ) b=( $(echo "${a[6]}" | sed s:/:' ':) )
if [ "${b[0]}" -ne "${b[1]}" ]; then if [ "${b[0]}" -eq "0" ]; then
: # Ignore until raid starts to report consistent data
elif [ "${b[0]}" -ne "${b[1]}" ]; then
# If the sync operation ("recover" in this case) is not # If the sync operation ("recover" in this case) is not
# finished, then it better be as follows: # finished, then it better be as follows:
[ "${a[5]}" = "AAa" ] [ "${a[5]}" = "AAa" ]
@ -125,20 +132,23 @@ while true; do
should [ "${a[7]}" = "idle" ] # RHBZ 1507719 should [ "${a[7]}" = "idle" ] # RHBZ 1507719
break break
fi fi
sleep .1 sleep .1
done done
aux enable_dev "$dev3" aux enable_dev "$dev3"
lvremove -ff $vg lvremove -ff $vg
test "$i" -gt 0 || die "Unable to get in sync $vg/$lv1"
########################################### ###########################################
# Catch anything suspicious with RAID1 initial resync # Catch anything suspicious with RAID1 initial resync
########################################### ###########################################
aux delay_dev "$dev2" 0 50 aux delay_dev "$dev2" 0 20
lvcreate --type raid1 -m 1 -aey -l 2 -n $lv1 $vg "$dev1" "$dev2" lvcreate --type raid1 -m 1 -aey -l 2 -n $lv1 $vg "$dev1" "$dev2"
while true; do for i in {100..0}; do
a=( $(dmsetup status $vg-$lv1) ) || die "Unable to get status of $vg/$lv1" a=( $(dmsetup status $vg-$lv1) ) || die "Unable to get status of $vg/$lv1"
b=( $(echo "${a[6]}" | sed s:/:' ':) ) b=( $(echo "${a[6]}" | sed s:/:' ':) )
if [ "${b[0]}" -ne "${b[1]}" ]; then if [ "${b[0]}" -eq "0" ]; then
: # Ignore until raid starts to report consistent data
elif [ "${b[0]}" -ne "${b[1]}" ]; then
# If the sync operation ("resync" in this case) is not # If the sync operation ("resync" in this case) is not
# finished, then it better be as follows: # finished, then it better be as follows:
[ "${a[5]}" = "aa" ] [ "${a[5]}" = "aa" ]
@ -153,9 +163,10 @@ while true; do
should [ "${a[7]}" = "idle" ] # RHBZ 1507719 should [ "${a[7]}" = "idle" ] # RHBZ 1507719
break break
fi fi
sleep .1 sleep .1
done done
aux enable_dev "$dev2" aux enable_dev "$dev2"
lvremove -ff $vg lvremove -ff $vg
test "$i" -gt 0 || die "Unable to get in sync $vg/$lv1"
vgremove -ff $vg vgremove -ff $vg