Feature conditional disabling service account token automount (#60)

* feature: add ability to toggle on/off service account token automount * fix: removed comment from values.yaml causing lint to fail
Multiwoven · Dec 6, 2024 · b06a0f5 · b06a0f5
1 parent e5914f2
commit b06a0f5
Show file tree

Hide file tree

Showing 7 changed files with 10 additions and 2 deletions.
diff --git a/charts/multiwoven/Chart.yaml b/charts/multiwoven/Chart.yaml
@@ -2,8 +2,8 @@ apiVersion: v2
 name: multiwoven
 description: Open-source reverse ETL, an alternative to Hightouch, Census etc. 🔥
 type: application
-version: 0.37.0
-appVersion: "0.37.0"
+version: 0.38.0
+appVersion: "0.38.0"
 maintainers:
   - name: subintp
   - name: RafaelOAiSquared

diff --git a/charts/multiwoven/templates/multiwoven-server-deployment.yaml b/charts/multiwoven/templates/multiwoven-server-deployment.yaml
@@ -24,6 +24,7 @@ spec:
       {{ if .Values.serviceAccount.enabled }}
       serviceAccountName: {{ .Values.serviceAccount.name }}
       {{ end }}
+      automountServiceAccountToken: {{ .Values.automountServiceAccountToken.enabled }}
       containers:
       - env:
         {{ if .Values.secretsStore.enabled }}

diff --git a/charts/multiwoven/templates/multiwoven-ui-deployment.yaml b/charts/multiwoven/templates/multiwoven-ui-deployment.yaml
@@ -21,6 +21,7 @@ spec:
         io.kompose.service: {{ include "chart.fullname" . }}-ui
       {{- include "chart.selectorLabels" . | nindent 8 }}
     spec:
+      automountServiceAccountToken: {{ .Values.automountServiceAccountToken.enabled }}
       containers:
       - env:
         envFrom:

diff --git a/charts/multiwoven/templates/multiwoven-worker-deployment.yaml b/charts/multiwoven/templates/multiwoven-worker-deployment.yaml
@@ -24,6 +24,7 @@ spec:
       {{ if .Values.serviceAccount.enabled }}
       serviceAccountName: {{ .Values.serviceAccount.name }}
       {{ end }}
+      automountServiceAccountToken: {{ .Values.automountServiceAccountToken.enabled }}
       containers:
       - args: {{- toYaml .Values.multiwovenWorker.multiwovenWorker.args | nindent 8 }}
         env:

diff --git a/charts/multiwoven/templates/temporal-deployment.yaml b/charts/multiwoven/templates/temporal-deployment.yaml
@@ -23,6 +23,7 @@ spec:
       {{ if .Values.serviceAccount.enabled }}
       serviceAccountName: {{ .Values.serviceAccount.name }}
       {{ end }}
+      automountServiceAccountToken: {{ .Values.automountServiceAccountToken.enabled }}
       containers:
       - env:
         - name: DB

diff --git a/charts/multiwoven/templates/temporal-ui-deployment.yaml b/charts/multiwoven/templates/temporal-ui-deployment.yaml
@@ -27,6 +27,7 @@ spec:
         kompose.cmd: kompose convert
         kompose.version: 1.32.0 (HEAD)
     spec:
+      automountServiceAccountToken: {{ .Values.automountServiceAccountToken.enabled }}
       containers:
       - env:
         - name: TEMPORAL_ADDRESS

diff --git a/charts/multiwoven/values.yaml b/charts/multiwoven/values.yaml
@@ -65,6 +65,9 @@ multiwovenConfig:
   viteFavIconUrl: ""
   workerHost: worker.multiwoven.com
 
+automountServiceAccountToken:
+  enabled: true
+
 externalDNS:
   enabled: false