Skip to content

Commit 01e393b

Browse files
committed
testsuite: fix occasional broker kill error
Problem: flux-start will kill all brokers after the first broker exits. By default the timeout to kill all the other brokers is 20s. A broker in a test in t3203-instance-recovery.t would sometimes be killed due to a slow running rc3 cleanup. This would lead to a invalid test error. Solution: Increase the kill timeout to 300s with the--test-exit-timeout option. Fixes #5290
1 parent b9c6523 commit 01e393b

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

t/t3203-instance-recovery.t

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,12 @@ test -n "$FLUX_TESTS_LOGFILE" && set -- "$@" --logfile
99

1010
runpty="flux ${SHARNESS_TEST_SRCDIR}/scripts/runpty.py"
1111

12+
# N.B. Increase test-exit-timeout from default of 20s, on slower / busy
13+
# systems timeout can trigger and kill broker.
1214
test_expect_success 'start a persistent instance of size 4' '
1315
mkdir -p test1 &&
14-
flux start --test-size=4 -o,-Sstatedir=$(pwd)/test1 /bin/true
16+
flux start --test-size=4 --test-exit-timeout=300s \
17+
-o,-Sstatedir=$(pwd)/test1 /bin/true
1518
'
1619
test_expect_success 'expected broker attributes are set in recovery mode' '
1720
cat >recov_attrs.exp <<-EOT &&

0 commit comments

Comments
 (0)