fix helm notifications config

flyteorg · Sep 20, 2023 · 16871a5 · 16871a5
1 parent 248d208
commit 16871a5
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 }}