feat: allow some snuba deployments to be pushed as hooks (#198)

sentry-kubernetes · Oct 12, 2020 · 9bda4ee · 9bda4ee
1 parent 945e4c0
commit 9bda4ee
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 2 deletions.
diff --git a/sentry/templates/deployment-relay.yaml b/sentry/templates/deployment-relay.yaml
@@ -7,7 +7,7 @@ metadata:
     chart: "{{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}"
     release: "{{ .Release.Name }}"
     heritage: "{{ .Release.Service }}"
-  {{- if .Values.relay.asHook }}
+  {{- if .Values.asHook }}
   annotations:
     "helm.sh/hook": "post-install,post-upgrade"
     "helm.sh/hook-weight": "15"

diff --git a/sentry/templates/deployment-snuba-outcomes-consumer.yaml b/sentry/templates/deployment-snuba-outcomes-consumer.yaml
@@ -7,6 +7,11 @@ metadata:
     chart: "{{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}"
     release: "{{ .Release.Name }}"
     heritage: "{{ .Release.Service }}"
+  {{- if .Values.asHook }}
+  annotations:
+    "helm.sh/hook": "post-install,post-upgrade"
+    "helm.sh/hook-weight": "17"
+  {{- end }}
 spec:
   selector:
     matchLabels:

diff --git a/sentry/templates/deployment-snuba-replacer.yaml b/sentry/templates/deployment-snuba-replacer.yaml
@@ -7,6 +7,11 @@ metadata:
     chart: "{{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}"
     release: "{{ .Release.Name }}"
     heritage: "{{ .Release.Service }}"
+  {{- if .Values.asHook }}
+  annotations:
+    "helm.sh/hook": "post-install,post-upgrade"
+    "helm.sh/hook-weight": "18"
+  {{- end }}
 spec:
   selector:
     matchLabels:

diff --git a/sentry/templates/deployment-snuba-sessions-consumer.yaml b/sentry/templates/deployment-snuba-sessions-consumer.yaml
@@ -7,6 +7,11 @@ metadata:
     chart: "{{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}"
     release: "{{ .Release.Name }}"
     heritage: "{{ .Release.Service }}"
+  {{- if .Values.asHook }}
+  annotations:
+    "helm.sh/hook": "post-install,post-upgrade"
+    "helm.sh/hook-weight": "16"
+  {{- end }}
 spec:
   selector:
     matchLabels:

diff --git a/sentry/values.yaml b/sentry/values.yaml
@@ -5,6 +5,8 @@ user:
   email: admin@sentry.local
   password: aaaa
 
+asHook: true
+
 images:
   sentry:
     # repository: getsentry/sentry
@@ -28,7 +30,6 @@ relay:
   # this is required on the first installation, as sentry has to be initialized first
   # recommended to set false for updating the helm chart afterwards,
   # as you will have some downtime on each update if it's a hook
-  asHook: true
   env: []
   probeInitialDelaySeconds: 10
   resources: {}