Skip to content

Commit

Permalink
Wait a bit for fluentd to be really up. (#440)
Browse files Browse the repository at this point in the history
It will time-out after 5 min if things are really broken in line with
the helm test default.

Signed-off-by: Paulo E. Castro <[email protected]>
  • Loading branch information
pecastro committed Nov 16, 2023
1 parent 5703c20 commit 4a505ec
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions charts/fluentd/templates/tests/test-connection.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,13 @@ spec:
containers:
- name: wget
image: busybox
command: ['wget']
args: ['{{ include "fluentd.fullname" . }}:24231/metrics']
command:
- sh
- -c
- |
set -e
# Give fluentd some time to start up
while :; do nc -vz {{ include "fluentd.fullname" . }}:24231 && break; sleep 1; done
wget '{{ include "fluentd.fullname" . }}:24231/metrics'
restartPolicy: Never
{{ end }}

0 comments on commit 4a505ec

Please sign in to comment.