Skip to content

Commit

Permalink
130-kill.bats: increase timeouts from 10s to 60s
Browse files Browse the repository at this point in the history
Increase the amount of time we're willing to wait for a log message that
a container should be printing to show up in the output of `logs -f`,
since on at least one CI configuration we're seeing a turnaround as high
as 46s, but it's not something we can directly control, so that's not a
hard maximum.

Signed-off-by: Nalin Dahyabhai <[email protected]>
  • Loading branch information
nalind committed Aug 23, 2021
1 parent 21f396d commit 3007bd4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/system/130-kill.bats
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ load helpers
exec 5<$fifo

# First container emits READY when ready; wait for it.
read -t 10 -u 5 ready
read -t 60 -u 5 ready
is "$ready" "READY" "first log message from container"

# Helper function: send the given signal, verify that it's received.
Expand All @@ -42,7 +42,7 @@ load helpers
local signum=${2:-$1} # e.g. if signal=HUP, we expect to see '1'

run_podman kill -s $signal $cid
read -t 10 -u 5 actual || die "Timed out: no ACK for kill -s $signal"
read -t 60 -u 5 actual || die "Timed out: no ACK for kill -s $signal"
is "$actual" "got: $signum" "Signal $signal handled by container"
}

Expand Down

0 comments on commit 3007bd4

Please sign in to comment.