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 authored and mheon committed Feb 2, 2024
1 parent cb799fd commit 4a58adf
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 @@ -269,7 +269,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 @@ -309,6 +309,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 4a58adf

Please sign in to comment.