Feat: add service account names for cronjobs (#681)

sentry-kubernetes · Jul 22, 2022 · 0bc212d · 0bc212d
1 parent 2418ba8
commit 0bc212d
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 1 deletion.
diff --git a/sentry/Chart.yaml b/sentry/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: sentry
 description: A Helm chart for Kubernetes
 type: application
-version: 15.0.1
+version: 15.0.2
 appVersion: 22.6.0
 dependencies:
   - name: memcached

diff --git a/sentry/templates/cronjob-sentry-cleanup.yaml b/sentry/templates/cronjob-sentry-cleanup.yaml
@@ -129,4 +129,7 @@ spec:
           {{- if .Values.sentry.cleanup.priorityClassName }}
           priorityClassName: "{{ .Values.sentry.cleanup.priorityClassName }}"
           {{- end }}
+          {{- if .Values.sentry.cleanup.serviceAccount }}
+          serviceAccountName: {{ .Values.sentry.cleanup.serviceAccount.name }}
+          {{- end }}
 {{- end }}
diff --git a/sentry/templates/cronjob-snuba-cleanup-errors.yaml b/sentry/templates/cronjob-snuba-cleanup-errors.yaml
@@ -98,4 +98,7 @@ spec:
           {{- if .Values.snuba.cleanupErrors.priorityClassName }}
           priorityClassName: "{{ .Values.snuba.cleanupErrors.priorityClassName }}"
           {{- end }}
+          {{- if .Values.snuba.cleanupErrors.serviceAccount }}
+          serviceAccountName: {{ .Values.snuba.cleanupErrors.serviceAccount.name }}
+          {{- end }}
 {{- end }}
diff --git a/sentry/templates/cronjob-snuba-cleanup-transactions.yaml b/sentry/templates/cronjob-snuba-cleanup-transactions.yaml
@@ -98,4 +98,7 @@ spec:
           {{- if .Values.snuba.cleanupTransactions.priorityClassName }}
           priorityClassName: "{{ .Values.snuba.cleanupTransactions.priorityClassName }}"
           {{- end }}
+          {{- if .Values.snuba.cleanupTransactions.serviceAccount }}
+          serviceAccountName: {{ .Values.snuba.cleanupTransactions.serviceAccount.name }}
+          {{- end }}
 {{- end }}
diff --git a/sentry/values.yaml b/sentry/values.yaml
@@ -222,6 +222,7 @@ sentry:
     days: 90
     sidecars: []
     volumes: []
+    serviceAccount: {}
 
 snuba:
   api:
@@ -408,6 +409,7 @@ snuba:
     schedule: "0 * * * *"
     sidecars: []
     volumes: []
+    serviceAccount: {}
 
   cleanupTransactions:
     activeDeadlineSeconds: 100
@@ -416,6 +418,7 @@ snuba:
     schedule: "0 * * * *"
     sidecars: []
     volumes: []
+    serviceAccount: {}
 
 hooks:
   enabled: true