Skip to content

Commit

Permalink
feat(fluentd): Allow service to be optional for the daemonset deploym…
Browse files Browse the repository at this point in the history
…ent; enabled by default (#429)

* Allow service to be optional; enabled by default

Signed-off-by: Thomas Spear <[email protected]>
  • Loading branch information
tspearconquest authored Oct 24, 2023
1 parent 9c38396 commit a3b2d8e
Show file tree
Hide file tree
Showing 3 changed files with 4 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.4.4
version: 0.4.5
appVersion: v1.16.2
icon: https://www.fluentd.org/images/miscellany/fluentd-logo_2x.png
home: https://www.fluentd.org/
Expand Down
3 changes: 2 additions & 1 deletion charts/fluentd/templates/service.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@

{{- if .Values.service.enabled -}}
apiVersion: v1
kind: Service
metadata:
Expand Down Expand Up @@ -32,3 +32,4 @@ spec:
{{- end }}
selector:
{{- include "fluentd.selectorLabels" . | nindent 4 }}
{{- end -}}
1 change: 1 addition & 0 deletions charts/fluentd/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -200,6 +200,7 @@ persistence:
## Fluentd service
##
service:
enabled: true
type: "ClusterIP"
annotations: {}
# loadBalancerIP:
Expand Down

0 comments on commit a3b2d8e

Please sign in to comment.