Skip to content

Commit be7c789

Browse files
authored
fix helm notifications config (#4055)
Signed-off-by: Derek Yu <[email protected]>
1 parent 248d208 commit be7c789

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

charts/flyte-core/templates/admin/configmap.yaml

+10-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,16 @@ data:
4545
notifications.yaml: |
4646
notifications:
4747
type: {{ .Values.workflow_notifications.config.notifications.type }}
48-
region: {{ tpl .Values.workflow_notifications.config.notifications.region $ }}
48+
{{- if eq .Values.workflow_notifications.config.notifications.type "aws" }}
49+
{{- with .Values.workflow_notifications.config.notifications.aws }}
50+
aws: {{ tpl (toYaml .) $ | nindent 8 }}
51+
{{- end }}
52+
{{- end }}
53+
{{- if eq .Values.workflow_notifications.config.notifications.type "gcp" }}
54+
{{- with .Values.workflow_notifications.config.notifications.gcp }}
55+
gcp: {{ tpl (toYaml .) $ | nindent 8 }}
56+
{{- end }}
57+
{{- end }}
4958
{{- with .Values.workflow_notifications.config.notifications.publisher }}
5059
publisher: {{- tpl (toYaml .) $ | nindent 8 }}
5160
{{- end }}

0 commit comments

Comments
 (0)