Skip to content

Commit

Permalink
tests/int/tty: increase the timeout
Browse files Browse the repository at this point in the history
Earlier, commit fce8dd4 already increased this timeout from 1 to 5 seconds.
Yet, I just saw this timeout being hit in actuated-arm CI.

Increase the timeout again, this time from 5 to 50 (100 * 0.5) seconds.

Also, use wait_pids_gone, and improve some comments.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
  • Loading branch information
kolyshkin committed Apr 26, 2024
1 parent e74ff0f commit 6bf1d3a
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions tests/integration/tty.bats
Original file line number Diff line number Diff line change
Expand Up @@ -171,15 +171,13 @@ function teardown() {
EOF
)

# run the exec
# Run the detached exec.
runc exec -t --pid-file pid.txt -d --console-socket "$CONSOLE_SOCKET" -p <(echo "$tty_info_with_consize_size") test_busybox
[ "$status" -eq 0 ]

# check the pid was generated
[ -e pid.txt ]

# wait for the process to finish
timeout 5 tail --pid="$(head -n 1 pid.txt)" -f /dev/null
# Wait for the exec to finish.
wait_pids_gone 100 0.5 "$(cat pid.txt)"

tty_info=$(
cat <<EOF
Expand Down

0 comments on commit 6bf1d3a

Please sign in to comment.