Bugfix to kubetail chart

* Added missing `serviceAccountName` to deployment
kubetail-org · Feb 11, 2024 · 797210c · 797210c
1 parent 050456a
commit 797210c
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 3 deletions.
diff --git a/charts/kubetail/Chart.yaml b/charts/kubetail/Chart.yaml
@@ -5,7 +5,7 @@ keywords:
 - kubetail
 - logging
 type: application
-version: 0.1.1
+version: 0.1.2
 appVersion: "0.1.0"
 home: https://github.com/kubetail-org/kubetail
 maintainers:

diff --git a/charts/kubetail/templates/_helpers.tpl b/charts/kubetail/templates/_helpers.tpl
@@ -66,3 +66,10 @@ app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
 {{- end }}
 app.kubernetes.io/managed-by: {{ .Release.Service }}
 {{- end }}
+
+{{/*
+ServiceAccount name
+*/}}
+{{- define "kubetail.serviceAccountName" -}}
+{{ if .Values.serviceAccount.name }}{{ .Values.serviceAccount.name }}{{ else }}{{ include "kubetail.fullname" . }}{{ end }}
+{{- end }}
diff --git a/charts/kubetail/templates/deployment.yaml b/charts/kubetail/templates/deployment.yaml
@@ -19,7 +19,7 @@ spec:
         checksum/config: {{ include (print $.Template.BasePath "/config-map.yaml") . | sha256sum }}
     spec:
       {{- if eq .Values.authMode "cluster" }}
-      serviceAccountName: {{ .Values.serviceAccount.name }}
+      serviceAccountName: {{ include "kubetail.serviceAccountName" . }}
       {{- end }}
       containers:
       - name: kubetail

diff --git a/charts/kubetail/templates/service-account.yaml b/charts/kubetail/templates/service-account.yaml
@@ -2,6 +2,6 @@ apiVersion: v1
 kind: ServiceAccount
 metadata:
   {{- include "kubetail.metadataNamespace" $ | nindent 2 }}
-  name: {{ if .Values.serviceAccount.name }}{{ .Values.serviceAccount.name }}{{ else }}{{ include "kubetail.fullname" . }}{{ end }}
+  name: {{ include "kubetail.serviceAccountName" . }}
   labels:
     {{- include "kubetail.labels" $ | nindent 4 }}