Skip to content

Commit 36afab2

Browse files
authored
fix: add hook-weight 3 to taskbroker and taskworker to prevent CrashLoopBackOff (#2225)
1 parent ef7a2e4 commit 36afab2

3 files changed

Lines changed: 28 additions & 0 deletions

File tree

charts/sentry/templates/sentry/taskbroker/statefulset-taskbroker.yaml

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,13 @@ metadata:
1212
heritage: "{{ $root.Release.Service }}"
1313
role: taskbroker-{{ $broker.name }}
1414
{{- include "sentry.component.labels" (dict "component" (printf "taskbroker-%s" $broker.name) "ctx" $root) | nindent 4 }}
15+
{{- if $root.Values.asHook }}
16+
annotations:
17+
meta.helm.sh/release-name: "{{ $root.Release.Name }}"
18+
meta.helm.sh/release-namespace: "{{ $root.Release.Namespace }}"
19+
"helm.sh/hook": "post-install,post-upgrade"
20+
"helm.sh/hook-weight": "3"
21+
{{- end }}
1522
spec:
1623
serviceName: {{ template "sentry.fullname" $root }}-taskbroker-{{ $broker.name }}
1724
replicas: {{ default $root.Values.sentry.taskBroker.replicas $broker.replicas }}
@@ -168,6 +175,13 @@ metadata:
168175
app: {{ template "sentry.fullname" $root }}
169176
release: "{{ $root.Release.Name }}"
170177
role: taskbroker-{{ $broker.name }}
178+
{{- if $root.Values.asHook }}
179+
annotations:
180+
meta.helm.sh/release-name: "{{ $root.Release.Name }}"
181+
meta.helm.sh/release-namespace: "{{ $root.Release.Namespace }}"
182+
"helm.sh/hook": "post-install,post-upgrade"
183+
"helm.sh/hook-weight": "3"
184+
{{- end }}
171185
spec:
172186
ports:
173187
- port: 50051

charts/sentry/templates/sentry/taskworker/deployment-taskworker.yaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,13 @@ metadata:
1212
heritage: "{{ $root.Release.Service }}"
1313
role: taskworker-{{ $worker.name }}
1414
{{- include "sentry.component.labels" (dict "component" (printf "taskworker-%s" $worker.name) "ctx" $root) | nindent 4 }}
15+
{{- if $root.Values.asHook }}
16+
annotations:
17+
meta.helm.sh/release-name: "{{ $root.Release.Name }}"
18+
meta.helm.sh/release-namespace: "{{ $root.Release.Namespace }}"
19+
"helm.sh/hook": "post-install,post-upgrade"
20+
"helm.sh/hook-weight": "3"
21+
{{- end }}
1522
spec:
1623
revisionHistoryLimit: {{ $root.Values.revisionHistoryLimit }}
1724
{{- $workerAutoscaling := default dict $worker.autoscaling }}

charts/sentry/templates/sentry/taskworker/hpa-taskworker.yaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,13 @@ apiVersion: {{ template "sentry.autoscaling.apiVersion" $root }}
2121
kind: HorizontalPodAutoscaler
2222
metadata:
2323
name: {{ template "sentry.fullname" $root }}-taskworker-{{ $worker.name }}
24+
{{- if $root.Values.asHook }}
25+
annotations:
26+
meta.helm.sh/release-name: "{{ $root.Release.Name }}"
27+
meta.helm.sh/release-namespace: "{{ $root.Release.Namespace }}"
28+
"helm.sh/hook": "post-install,post-upgrade"
29+
"helm.sh/hook-weight": "3"
30+
{{- end }}
2431
labels:
2532
{{- include "sentry.component.labels" (dict "component" (printf "taskworker-%s" $worker.name) "ctx" $root) | nindent 4 }}
2633
spec:

0 commit comments

Comments
 (0)