Skip to content

Commit

Permalink
tests: increase sleeps from 1s to 2s
Browse files Browse the repository at this point in the history
The issue here is that some of the tests sporadically fail due to things
being still processed. Default 1s delays proven not to be sufficient for
newly created CI, as tests tend to ocassionally fail.

This patch increases default 1s sleep to 2s, to hopefully get rid of
sporadical fails.

Signed-off-by: Mateusz Kusiak <[email protected]>
  • Loading branch information
dancesWithMachines authored and mtkaczyk committed Dec 13, 2024
1 parent d4be82f commit 999c963
Show file tree
Hide file tree
Showing 26 changed files with 47 additions and 47 deletions.
4 changes: 2 additions & 2 deletions clustermd_tests/01r10_Grow_bitmap-switch
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ check all nobitmap
mdadm --grow $md0 --bitmap=internal
[ $? -eq '0' ] ||
die "$NODE1: change bitmap 'none' to 'internal' failed."
sleep 1
sleep 2
mdadm -X $dev0 $dev1 &> /dev/null
[ $? -eq '0' ] ||
die "$NODE1: create 'internal' bitmap failed."
Expand All @@ -37,7 +37,7 @@ mdadm --grow $md0 --bitmap=clustered
[ $? -eq '0' ] ||
die "$NODE1: change bitmap 'none' to 'clustered' failed."
ssh $NODE2 mdadm -A $md0 $dev0 $dev1
sleep 1
sleep 2
for ip in $NODES
do
ssh $ip "mdadm -X $dev0 $dev1 | grep -q 'Cluster name'" ||
Expand Down
2 changes: 1 addition & 1 deletion clustermd_tests/func.sh
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ check_dlm()
crm configure clone base-clone base-group \
meta interleave=true
fi
sleep 1
sleep 2
for ip in $NODE1 $NODE2
do
ssh $ip "pgrep dlm_controld > /dev/null" || {
Expand Down
2 changes: 1 addition & 1 deletion misc/mdcheck
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ do
systemctl stop mdcheck_continue.timer
exit 0;
fi
sleep 120
sleep 220
done

# We've waited, and there are still checks running.
Expand Down
2 changes: 1 addition & 1 deletion tests/01r1fail
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ check resync
mdadm $md0 --fail $dev2
check resync
mdadm $md0 --fail $dev1
sleep 1
sleep 2
check nosync
check state U___
mdadm $md0 --add $dev4 $dev3
Expand Down
4 changes: 2 additions & 2 deletions tests/01replace
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ do

mdadm $md0 --add $dev1 $dev5
mdadm $md0 --replace $dev0
sleep 1
sleep 2
mdadm $md0 --fail $dev2
check wait
sum2=`sha1sum < $md0`
Expand All @@ -42,7 +42,7 @@ do
mdadm $md0 --remove $dev0 $dev2
mdadm $md0 --add $dev0 $dev2
mdadm $md0 --replace $dev3
sleep 1
sleep 2
mdadm $md0 --fail $dev0 $dev2
check wait
sum3=`sha1sum < $md0`
Expand Down
6 changes: 3 additions & 3 deletions tests/05r1-re-add
Original file line number Diff line number Diff line change
Expand Up @@ -12,22 +12,22 @@ testdev $md0 1 $mdsize1a 64
sleep 6

mdadm $md0 -f $dev2
sleep 1
sleep 2
mdadm $md0 -r $dev2
mdadm $md0 -a $dev2
#cat /proc/mdstat
check nosync

mdadm $md0 -f $dev2
sleep 1
sleep 2
mdadm $md0 -r $dev2
testdev $md0 1 $mdsize1a 64
mdadm $md0 -a $dev2
check wait
blockdev --flushbufs $dev1 $dev2
cmp --ignore-initial=$[64*512] --bytes=$[$mdsize0*1024] $dev1 $dev2

mdadm $md0 -f $dev2; sleep 1
mdadm $md0 -f $dev2; sleep 2
mdadm $md0 -r $dev2
if dd if=/dev/zero of=$md0 ; then : ; fi
blockdev --flushbufs $md0 # ensure writes have been sent.
Expand Down
4 changes: 2 additions & 2 deletions tests/05r1-re-add-nosuper
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,14 @@
# readd should fail.
#
mdadm -B $md0 -l1 -n2 -d1 $dev1 $dev2
sleep 1
sleep 2
check resync
check wait
testdev $md0 1 $size 1
sleep 6

mdadm $md0 -f $dev2
sleep 1
sleep 2
mdadm $md0 -r $dev2
if mdadm $md0 --re-add $dev2; then
err "re-add should fail"
Expand Down
12 changes: 6 additions & 6 deletions tests/05r6tor0
Original file line number Diff line number Diff line change
Expand Up @@ -4,28 +4,28 @@ set -x -e
# then reshape back up to RAID5 and RAID5

mdadm -CR $md0 -l6 -n5 $dev0 $dev1 $dev2 $dev3 $dev4
check wait; sleep 1
check wait; sleep 2
check raid6
testdev $md0 3 19456 512
mdadm -G $md0 -l5
check wait; sleep 1
check wait; sleep 2
check raid5
testdev $md0 3 19456 512
mdadm -G $md0 -l0
check wait; sleep 1
check wait; sleep 2
while ps auxf | grep "mdadm -G" | grep -v grep
do
sleep 1
sleep 2
done
check raid0
testdev $md0 3 19456 512
mdadm -G $md0 -l5 --add $dev3 $dev4
check wait; sleep 1
check wait; sleep 2
check raid5
check algorithm 2
testdev $md0 3 19456 512
mdadm -G $md0 -l 6
check wait; sleep 1
check wait; sleep 2
check raid6
check algorithm 2
testdev $md0 3 19456 512
2 changes: 1 addition & 1 deletion tests/07autodetect
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ sfdisk $mdp1 >&2 << END
END
udevadm settle
mdadm -CR -e 0 $md0 -l1 -n2 ${mdp0}p1 ${mdp1}p1
sleep 1
sleep 2
check resync
check raid1
check wait
Expand Down
2 changes: 1 addition & 1 deletion tests/07changelevelintr
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ checkgeo() {
shift
sleep 0.5
check wait
sleep 1
sleep 2
for attr in level raid_disks chunk_size layout
do
if [ $# -gt 0 ] ; then
Expand Down
4 changes: 2 additions & 2 deletions tests/07changelevels
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@ checkgeo() {
# level raid_disks chunk_size layout
dev=$1
shift
sleep 15
sleep 25
check wait
sleep 1
sleep 2
for attr in level raid_disks chunk_size layout
do
if [ $# -gt 0 ] ; then
Expand Down
2 changes: 1 addition & 1 deletion tests/07reshape5intr
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ do
if [[ $(echo check > /sys/block/md0/md/sync_action) != 0 ]]; then
break;
fi
sleep 1
sleep 2
done

if [[ i == max ]]; then
Expand Down
2 changes: 1 addition & 1 deletion tests/09imsm-assemble
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ imsm_check_removal() {
success=1
break
fi
sleep 1
sleep 2
done

if [ $success -ne 1 ]; then
Expand Down
2 changes: 1 addition & 1 deletion tests/10ddf-assemble-missing
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ mdadm --wait $member0 || true
mdadm --wait $member1 || true

mdadm -Ss
sleep 1
sleep 2

# Add all devices except those for $member0
mdadm -I $dev10
Expand Down
4 changes: 2 additions & 2 deletions tests/10ddf-fail-readd
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ set -- $(get_raiddisks $member0)
fail0=$1
mdadm $member0 --fail $fail0

sleep 1
sleep 2
mdadm $container --remove $fail0

set -- $(get_raiddisks $member0)
Expand All @@ -27,7 +27,7 @@ good0=$1
# We re-add the disk now
mdadm $container --add $fail0

sleep 1
sleep 2
mdadm --wait $member0 || true

ret=0
Expand Down
4 changes: 2 additions & 2 deletions tests/10ddf-fail-readd-readonly
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ set -- $(get_raiddisks $member0)
fail0=$1
mdadm $member0 --fail $fail0

sleep 1
sleep 2
set -- $(get_raiddisks $member0)
case $1 in MISSING) shift;; esac
good0=$1
Expand Down Expand Up @@ -44,7 +44,7 @@ mdadm $container --remove $fail0
# We re-add the disk now
mdadm $container --add $fail0

sleep 1
sleep 2
mdadm --wait $member0 || true

set -- $(get_raiddisks $member0)
Expand Down
6 changes: 3 additions & 3 deletions tests/10ddf-fail-stop-readd
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ set -- $(get_raiddisks $member0)
fail0=$1
mdadm $member0 --fail $fail0

sleep 1
sleep 2
mdadm $container --remove $fail0

set -- $(get_raiddisks $member0)
Expand All @@ -27,7 +27,7 @@ good0=$1

mdadm -Ss

sleep 1
sleep 2
# Now simulate incremental assembly
mdadm -I $good0
mdadm -IRs || true
Expand All @@ -38,7 +38,7 @@ mke2fs -F $member0
# We re-add the disk now
mdadm $container --add $fail0

sleep 1
sleep 2
mdadm --wait $member0 || true

ret=0
Expand Down
2 changes: 1 addition & 1 deletion tests/10ddf-fail-two-spares
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ sum0=$(sha1sum $member0)
sum1=$(sha1sum $member1)

mdadm --fail $member1 $dev11
sleep 1
sleep 2
mdadm --fail $member1 $dev12

# We will have 4 resync procedures, 2 spares for 2 arrays.
Expand Down
8 changes: 4 additions & 4 deletions tests/10ddf-incremental-wrong-order
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ sha_0a=$(sha1_sum $member0)
sha_1a=$(sha1_sum $member1)

mdadm -Ss
sleep 1
sleep 2

# Add all devices except those for $member0
mdadm -I $dev10
Expand All @@ -35,7 +35,7 @@ e2fsck -fy $member1
sha_1b=$(sha1_sum $member1)

mdadm -Ss
sleep 1
sleep 2

# Seq number should be different now
seq8a=$(mdadm -E $dev8 | sed -n 's/^ *Seq : //p')
Expand Down Expand Up @@ -64,7 +64,7 @@ sha_0c=$(sha1_sum $member0)
sha_1c=$(sha1_sum $member1)

mdadm -Ss
sleep 1
sleep 2

seq8c=$(mdadm -E $dev8 | sed -n 's/^ *Seq : //p')
seq10c=$(mdadm -E $dev10 | sed -n 's/^ *Seq : //p')
Expand Down Expand Up @@ -103,7 +103,7 @@ if mdadm -A $container $dev8 $dev9 $dev10 $dev11; then
sha_0d=$(sha1_sum $member0)
sha_1d=$(sha1_sum $member1)
mdadm -Ss
sleep 1
sleep 2
seq8d=$(mdadm -E $dev8 | sed -n 's/^ *Seq : //p')
seq10d=$(mdadm -E $dev10 | sed -n 's/^ *Seq : //p')
if [ x$sha_0a != x$sha_0d ]; then
Expand Down
2 changes: 1 addition & 1 deletion tests/24raid10deadlock
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ clean_up_test()
kill -9 $pid
kill -9 $action_pid

sleep 1
sleep 2

if ps $action_pid | tail -1 | awk '{print $3}' | grep D; then
die "thread that is writing sysfs is stuck in D state, deadlock is triggered"
Expand Down
2 changes: 1 addition & 1 deletion tests/24raid456deadlock
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ clean_up_test()
pkill -9 fio
kill -9 $pid

sleep 1
sleep 2

if ps $pid | tail -1 | awk '{print $3}' | grep D; then
die "thread that is writing sysfs is stuck in D state, deadlock is triggered"
Expand Down
4 changes: 2 additions & 2 deletions tests/25raid456-recovery-while-reshape
Original file line number Diff line number Diff line change
Expand Up @@ -18,13 +18,13 @@ set_up_test || die "set up test failed"

# trigger reshape
mdadm --grow -l 6 $md0
sleep 1
sleep 2

# set up replacement
echo frozen > /sys/block/md0/md/sync_action
echo want_replacement > /sys/block/md0/md/rd0/state
echo reshape > /sys/block/md0/md/sync_action
sleep 1
sleep 2

# reassemeble array
mdadm -S $md0 || die "can't stop array"
Expand Down
2 changes: 1 addition & 1 deletion tests/25raid456-reshape-corrupt-data
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ set_up_test || die "set up test failed"
# trigger reshape
echo 1000 > /sys/block/md0/md/sync_speed_max
mdadm --grow -l 6 $md0
sleep 1
sleep 2

# stop and start reshape
echo frozen > /sys/block/md0/md/sync_action
Expand Down
2 changes: 1 addition & 1 deletion tests/25raid456-reshape-deadlock
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ set_up_test || die "set up test failed"

# trigger reshape
mdadm --grow -l 6 $md0
sleep 1
sleep 2

# stop reshape
echo frozen > /sys/block/md0/md/sync_action
Expand Down
4 changes: 2 additions & 2 deletions tests/25raid456-reshape-while-recovery
Original file line number Diff line number Diff line change
Expand Up @@ -18,12 +18,12 @@ set_up_test || die "set up test failed"

# set up replacement
echo want_replacement > /sys/block/md0/md/rd0/state
sleep 1
sleep 2

# trigger reshape
echo frozen > /sys/block/md0/md/sync_action
mdadm --grow -l 6 $md0
sleep 1
sleep 2

# reassemeble array
mdadm -S $md0 || die "can't stop array"
Expand Down
4 changes: 2 additions & 2 deletions tests/func.sh
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,7 @@ wait_for_reshape_end() {
do
sync_action=$(grep -Ec '(resync|recovery|reshape|check|repair) *=' /proc/mdstat)
if (( "$sync_action" != 0 )); then
sleep 1
sleep 2
continue
elif [[ $(pgrep -f "mdadm --grow --continue" > /dev/null) != "" ]]; then
echo "Grow continue did not finish but reshape is done" >&2
Expand Down Expand Up @@ -381,7 +381,7 @@ check() {
do
sync_action=$(grep -Ec '(resync|recovery|reshape|check|repair) *=' /proc/mdstat)
if (( "$sync_action" == 0 )); then
sleep 1
sleep 2
iterations=$(( $iterations + 1 ))
continue
else
Expand Down

0 comments on commit 999c963

Please sign in to comment.