Skip to content

Commit

Permalink
feat: add share process namespace to hook jobs (#688)
Browse files Browse the repository at this point in the history
  • Loading branch information
nishaero authored Jul 27, 2022
1 parent 50b8f1e commit b81316d
Show file tree
Hide file tree
Showing 7 changed files with 17 additions and 1 deletion.
2 changes: 1 addition & 1 deletion sentry/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: sentry
description: A Helm chart for Kubernetes
type: application
version: 15.0.7
version: 15.0.8
appVersion: 22.6.0
dependencies:
- name: memcached
Expand Down
3 changes: 3 additions & 0 deletions sentry/templates/hooks/sentry-db-check.job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -143,4 +143,7 @@ spec:
{{- end }}
resources:
{{ toYaml .Values.hooks.dbCheck.resources | indent 10 }}
{{- if .Values.hooks.shareProcessNamespace }}
shareProcessNamespace: {{ .Values.hooks.shareProcessNamespace }}
{{- end }}
{{- end }}
3 changes: 3 additions & 0 deletions sentry/templates/hooks/sentry-db-init.job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -99,4 +99,7 @@ spec:
{{- if .Values.hooks.dbInit.volumes }}
{{ toYaml .Values.hooks.dbInit.volumes | indent 6 }}
{{- end }}
{{- if .Values.hooks.shareProcessNamespace }}
shareProcessNamespace: {{ .Values.hooks.shareProcessNamespace }}
{{- end }}
{{- end -}}
3 changes: 3 additions & 0 deletions sentry/templates/hooks/snuba-db-init.job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -93,4 +93,7 @@ spec:
- name: config
configMap:
name: {{ template "sentry.fullname" . }}-snuba
{{- if .Values.hooks.shareProcessNamespace }}
shareProcessNamespace: {{ .Values.hooks.shareProcessNamespace }}
{{- end }}
{{- end }}
3 changes: 3 additions & 0 deletions sentry/templates/hooks/snuba-migrate.job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -93,4 +93,7 @@ spec:
- name: config
configMap:
name: {{ template "sentry.fullname" . }}-snuba
{{- if .Values.hooks.shareProcessNamespace }}
shareProcessNamespace: {{ .Values.hooks.shareProcessNamespace }}
{{- end }}
{{- end }}
3 changes: 3 additions & 0 deletions sentry/templates/hooks/user-create.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -104,4 +104,7 @@ spec:
- name: config
configMap:
name: {{ template "sentry.fullname" . }}-sentry
{{- if .Values.hooks.shareProcessNamespace }}
shareProcessNamespace: {{ .Values.hooks.shareProcessNamespace }}
{{- end }}
{{- end -}}
1 change: 1 addition & 0 deletions sentry/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -429,6 +429,7 @@ hooks:
enabled: true
removeOnSuccess: true
activeDeadlineSeconds: 100
shareProcessNamespace: false
dbCheck:
image:
# repository: subfuzion/netcat
Expand Down

0 comments on commit b81316d

Please sign in to comment.