Skip to content

Commit

Permalink
systests: kube with policies test: fix race
Browse files Browse the repository at this point in the history
Add a wait_for_ready() to one kube-play test, to make sure
container output has made it to the journal.

Probably does not fix containers#18501, but I think it might fix its
most common presentation.

Signed-off-by: Ed Santiago <[email protected]>
  • Loading branch information
edsantiago committed Jan 11, 2024
1 parent 83f89db commit 0e09527
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/system/260-sdnotify.bats
Original file line number Diff line number Diff line change
Expand Up @@ -325,7 +325,7 @@ spec:
- command:
- /bin/sh
- -c
- 'printenv NOTIFY_SOCKET; while ! test -f /stop;do sleep 0.1;done'
- 'printenv NOTIFY_SOCKET; echo READY; while ! test -f /stop;do sleep 0.1;done'
image: $SYSTEMD_IMAGE
name: a
- command:
Expand Down Expand Up @@ -367,6 +367,7 @@ EOF
die "container $container_a and/or $container_b did not start"
fi

wait_for_ready $container_a
# Make sure the containers have the correct policy
run_podman container inspect $container_a $container_b $service_container --format "{{.Config.SdNotifyMode}}"
is "$output" "container
Expand Down

0 comments on commit 0e09527

Please sign in to comment.