Skip to content

Commit

Permalink
systests: test instrumentation
Browse files Browse the repository at this point in the history
for containers#18514: if we get a timeout in teardown(), run and show
the output of podman system locks

for containers#18831: if we hit unmount/EINVAL, nothing will ever work
again, so signal all future tests to skip.

Signed-off-by: Ed Santiago <[email protected]>
  • Loading branch information
edsantiago committed Jun 21, 2023
1 parent 80dddd2 commit 00292ae
Showing 1 changed file with 21 additions and 1 deletion.
22 changes: 21 additions & 1 deletion test/system/helpers.bash
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,18 @@ function basic_setup() {
for line in "${lines[@]}"; do
set $line
echo "# setup(): removing stray external container $1 ($2)" >&3
run_podman rm -f $1
run_podman '?' rm -f $1
if [[ $status -ne 0 ]]; then
echo "# [setup] $_LOG_PROMPT podman rm -f $1" >&3
for errline in "${lines[@]}"; do
echo "# $errline" >&3
done
# FIXME FIXME FIXME: temporary hack for #18831. If we see the
# unmount/EINVAL flake, nothing will ever work again.
if [[ $output =~ unmounting.*invalid ]]; then
touch "${BATS_SUITE_TMPDIR}/forget-it"
fi
fi
done

# Clean up all images except those desired
Expand Down Expand Up @@ -142,6 +153,15 @@ function basic_teardown() {
for line in "${lines[*]}"; do
echo "# $line" >&3
done

# Special case for timeout: check for locks (#18514)
if [[ $status -eq 124 ]]; then
echo "# [teardown] $_LOG_PROMPT podman system locks" >&3
run $PODMAN system locks
for line in "${lines[*]}"; do
echo "# $line" >&3
done
fi
fi
done

Expand Down

0 comments on commit 00292ae

Please sign in to comment.