fix helm notifications config (#4055)

Signed-off-by: Derek Yu <[email protected]>
flyteorg · Sep 20, 2023 · be7c789 · be7c789
1 parent 248d208
commit be7c789
Showing 1 changed file with 10 additions and 1 deletion.
diff --git 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 }}