From 9ba5f64e329e3e41101fe02199827857381780d3 Mon Sep 17 00:00:00 2001 From: Derek Yu Date: Wed, 20 Sep 2023 09:24:22 +0100 Subject: [PATCH] fix helm notifications config Signed-off-by: Derek Yu --- charts/flyte-core/templates/admin/configmap.yaml | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/charts/flyte-core/templates/admin/configmap.yaml b/charts/flyte-core/templates/admin/configmap.yaml index f0e0e01eb5..fb5820c842 100644 --- a/charts/flyte-core/templates/admin/configmap.yaml +++ b/charts/flyte-core/templates/admin/configmap.yaml @@ -45,7 +45,16 @@ data: notifications.yaml: | notifications: type: {{ .Values.workflow_notifications.config.notifications.type }} - region: {{ tpl .Values.workflow_notifications.config.notifications.region $ }} + {{- if eq .Values.workflow_notifications.config.notifications.type "aws" }} + {{- with .Values.workflow_notifications.config.notifications.aws }} + aws: {{ tpl (toYaml .) $ | nindent 8 }} + {{- end }} + {{- end }} + {{- if eq .Values.workflow_notifications.config.notifications.type "gcp" }} + {{- with .Values.workflow_notifications.config.notifications.gcp }} + gcp: {{ tpl (toYaml .) $ | nindent 8 }} + {{- end }} + {{- end }} {{- with .Values.workflow_notifications.config.notifications.publisher }} publisher: {{- tpl (toYaml .) $ | nindent 8 }} {{- end }}