Skip to content

Commit

Permalink
test/system/255-auto-update.bats: multiple services
Browse files Browse the repository at this point in the history
Wait for all generated services to be ready to be sure we can iron out
race conditions.  Also disable rollbacks to make sure we can analyze
the error if restarting a service fails.  This information may be
crucial to understand the flakes on Debian as tracked in containers#17607.

Signed-off-by: Valentin Rothberg <vrothberg@redhat.com>
  • Loading branch information
vrothberg committed Mar 14, 2023
1 parent 40d0d23 commit 7a7af73
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions test/system/255-auto-update.bats
Original file line number Diff line number Diff line change
Expand Up @@ -338,11 +338,13 @@ EOF
fi
done

# Only check that the last service is started. Previous services should already be activated.
_wait_service_ready container-$cname.service
# Make sure all services are ready.
for cname in "${cnames[@]}"; do
_wait_service_ready container-$cname.service
done
run_podman commit --change CMD=/bin/bash $local_cname quay.io/libpod/localtest:latest
# Exit code is expected, due to invalid 'fakevalue'
run_podman 125 auto-update
run_podman 125 auto-update --rollback=false
update_log=$output
is "$update_log" ".*invalid auto-update policy.*" "invalid policy setup"
is "$update_log" ".*Error: invalid auto-update policy.*" "invalid policy setup"
Expand Down

0 comments on commit 7a7af73

Please sign in to comment.