Skip to content

Commit 51a3121

Browse files
authored
test-crudd can collect more info, test_up can be gentle (#997)
Update test-crudd to have a timeout and gather more info when things go wrong. Update test_up.sh to wait a little between tests.
1 parent 82fbf1e commit 51a3121

File tree

2 files changed

+24
-3
lines changed

2 files changed

+24
-3
lines changed

.github/buildomat/jobs/test-crudd-benchmark.sh

+18
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
#: target = "helios-2.0"
66
#: output_rules = [
77
#: "/tmp/crudd-speed-battery-results.json",
8+
#: "%/tmp/debug/*.txt",
9+
#: "/tmp/dsc/*.txt",
810
#: "/tmp/core.*",
911
#: ]
1012
#: skip_clone = true
@@ -42,6 +44,22 @@ done
4244

4345
export BINDIR=/var/tmp/bins
4446

47+
banner setup
48+
pfexec plimit -n 9123456 $$
49+
50+
echo "Setup self timeout"
51+
jobpid=$$; (sleep $(( 2 * 60 * 60 )); ps -ef; zfs list;kill $jobpid) &
52+
53+
echo "Setup debug logging"
54+
mkdir /tmp/debug
55+
psrinfo -v > /tmp/debug/psrinfo.txt
56+
df -h > /tmp/debug/df.txt
57+
prstat -d d -mLc 1 > /tmp/debug/prstat.txt 2>&1 &
58+
iostat -T d -xn 1 > /tmp/debug/iostat.txt 2>&1 &
59+
mpstat -T d 1 > /tmp/debug/mpstat.txt 2>&1 &
60+
vmstat -T d -p 1 < /dev/null > /tmp/debug/paging.txt 2>&1 &
61+
pfexec dtrace -Z -s $input/scripts/perf-downstairs-tick.d > /tmp/debug/dtrace.txt 2>&1 &
62+
4563
banner "crudd bench"
4664
pfexec plimit -n 9123456 $$
4765

tools/test_up.sh

+6-3
Original file line numberDiff line numberDiff line change
@@ -155,8 +155,8 @@ for tt in ${test_list}; do
155155
echo "" >> "${log_prefix}_out.txt"
156156
echo "Running test: $tt" | tee -a "${log_prefix}_out.txt"
157157
echo "Running test: $tt" >> "${log_prefix}_out.txt"
158-
echo "$ct" "$tt" -g "$gen" --verify-at-end -q "${args[@]}" >> "${log_prefix}_out.txt"
159-
if ! "$ct" "$tt" -g "$gen" --verify-at-end -q "${args[@]}" >> "${log_prefix}_out.txt" 2>&1; then
158+
echo "$ct" "$tt" -g "$gen" --verify-at-end --stable "${args[@]}" >> "${log_prefix}_out.txt"
159+
if ! "$ct" "$tt" -g "$gen" --verify-at-end --stable "${args[@]}" >> "${log_prefix}_out.txt" 2>&1; then
160160
(( res += 1 ))
161161
echo ""
162162
echo "Failed crutest $tt test"
@@ -166,7 +166,10 @@ for tt in ${test_list}; do
166166
else
167167
echo "Completed test: $tt"
168168
fi
169-
(( gen += 1 ))
169+
(( gen += 2 ))
170+
# Yes, this is bad. Sometimes we just need a little time for the test
171+
# to finish everything before the next one starts.
172+
sleep 5
170173
done
171174
(( gen += 10 ))
172175

0 commit comments

Comments
 (0)