Skip to content

Commit

Permalink
fix podman system connection - tcp flake
Browse files Browse the repository at this point in the history
The test was only waiting for the port to be ready but that doesn't
imply the server being ready to serve requests.  Hence, add a loop
waiting for the `info` call to succeed.

Fixes: #16916
Signed-off-by: Valentin Rothberg <[email protected]>
  • Loading branch information
vrothberg committed Jan 2, 2023
1 parent 3fbf62e commit a741299
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions test/system/272-system-connection.bats
Original file line number Diff line number Diff line change
Expand Up @@ -112,9 +112,17 @@ $c2[ ]\+tcp://localhost:54321[ ]\+true" \
--runroot ${PODMAN_TMPDIR}/runroot \
system service -t 99 tcp://localhost:$_SERVICE_PORT &
_SERVICE_PID=$!
# Wait for the port and the podman-service to be ready.
wait_for_port localhost $_SERVICE_PORT

_run_podman_remote info --format '{{.Host.RemoteSocket.Path}}'
local timeout=10
while [[ $timeout -gt 1 ]]; do
_run_podman_remote ? info --format '{{.Host.RemoteSocket.Path}}'
if [[ $status == 0 ]]; then
break
fi
sleep 1
let timeout=$timeout-1
done
is "$output" "tcp://localhost:$_SERVICE_PORT" \
"podman info works, and talks to the correct server"

Expand Down

0 comments on commit a741299

Please sign in to comment.