[stable/fluentd] Fix missing serviceName in HPA (helm#21202)

Signed-off-by: Jeremy Mathevet <jeremy@mathevet.net> Signed-off-by: Miguel Mingorance <miguel.mingorance@deliveryhero.com>
mmingorance-dh · Aug 28, 2020 · f90f439 · f90f439
1 parent bcd7c07
commit f90f439
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/stable/fluentd/Chart.yaml b/stable/fluentd/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v1
 description: A Fluentd Elasticsearch Helm chart for Kubernetes.
 icon: https://raw.githubusercontent.com/fluent/fluentd-docs/master/public/logo/Fluentd_square.png
 name: fluentd
-version: 2.4.0
+version: 2.4.1
 appVersion: v2.4.0
 home: https://www.fluentd.org/
 sources:

diff --git a/stable/fluentd/templates/_helpers.tpl b/stable/fluentd/templates/_helpers.tpl
@@ -40,4 +40,4 @@ Create the name of the service account to use
 {{- else -}}
     {{ default "default" .Values.serviceAccount.name }}
 {{- end -}}
-{{- end -}}
+{{- end -}}
diff --git a/stable/fluentd/templates/hpa.yaml b/stable/fluentd/templates/hpa.yaml
@@ -7,12 +7,13 @@ metadata:
     app: {{ template "fluentd.name" . }}
     chart: {{ template "fluentd.chart" . }}
     release: {{ .Release.Name }}
-    heritage: {{ .Release.Service }}    
+    heritage: {{ .Release.Service }}
 spec:
   scaleTargetRef:
     apiVersion: apps/v1
     kind: StatefulSet
     name: {{ include "fluentd.fullname" . }}
+    serviceName: {{ .Values.service.name | default (include "fluentd.fullname" .) }}
   minReplicas: {{ .Values.autoscaling.minReplicas }}
   maxReplicas: {{ .Values.autoscaling.maxReplicas }}
   metrics: