fix: serviceaccount name on deployment template

neighborhoods · Nov 15, 2023 · 8575094 · 8575094
1 parent 7fbc665
commit 8575094
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/charts/neighborhoods-apps/CHANGELOG.md b/charts/neighborhoods-apps/CHANGELOG.md
@@ -1,4 +1,6 @@
 # Changelog
+## 2.12.3
+- Fix serviceaccount name on deployment template
 ## 2.12.2
 - Removed iam role from deployment. Use serviceaccount instead
 ## 2.12.1

diff --git a/charts/neighborhoods-apps/Chart.yaml b/charts/neighborhoods-apps/Chart.yaml
@@ -1,7 +1,7 @@
 apiVersion: v2
 description: Neighborhoods standard chart setup with Nginx fronting the HTTP deployment
 name: neighborhoods-app
-version: 2.12.2
+version: 2.12.3
 kubeVersion: ">=1.23.0-0"
 home: https://github.com/neighborhoods/neighborhoods-chart
 sources:

diff --git a/charts/neighborhoods-apps/templates/http-deployment.yaml b/charts/neighborhoods-apps/templates/http-deployment.yaml
@@ -30,7 +30,7 @@ spec:
     spec:
 {{ include "util.v1.extra.affinity" (.Values.http.affinity | default .Values.affinity) | indent 6 }}
 {{ include "util.v1.extra.tolerations" (.Values.http.tolerations | default .Values.tolerations) | indent 6 }}
-      serviceAccountName: {{ .Values.http.serviceAccountName | default .Values.serviceAccountName | default "" | quote }}
+      serviceAccountName: {{ include "standard-app.v1.serviceAccountName" . }}
       priorityClassName: {{ .Values.http.priorityClassName | default .Values.priorityClassName | default "" | quote }}
       containers:
       - name: app