Skip to content

Commit

Permalink
Correct basic test to pass in the default case. (fluent#440)
Browse files Browse the repository at this point in the history
Signed-off-by: Paulo E. Castro <[email protected]>
  • Loading branch information
pecastro committed Nov 16, 2023
1 parent 5d1a323 commit 5703c20
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/fluentd/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: fluentd
description: A Helm chart for Kubernetes
# type: application
version: 0.5.0
version: 0.5.1
appVersion: v1.16.2
icon: https://www.fluentd.org/images/miscellany/fluentd-logo_2x.png
home: https://www.fluentd.org/
Expand Down
10 changes: 9 additions & 1 deletion charts/fluentd/templates/tests/test-connection.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,10 @@
{{/*
Target the very simple case where
fluentd is deployed with the default values
If the fluentd config is overriden and the metrics server removed
this will fail.
*/}}
{{ if empty .Values.service.ports }}
apiVersion: v1
kind: Pod
metadata:
Expand All @@ -11,5 +18,6 @@ spec:
- name: wget
image: busybox
command: ['wget']
args: ['{{ include "fluentd.fullname" . }}:{{ .Values.service.port }}']
args: ['{{ include "fluentd.fullname" . }}:24231/metrics']
restartPolicy: Never
{{ end }}

0 comments on commit 5703c20

Please sign in to comment.