simpler append of subscriptions

codefresh-io · May 12, 2024 · 6b60d78 · 6b60d78
1 parent 36ae990
commit 6b60d78
Show file tree

Hide file tree

Showing 2 changed files with 258 additions and 263 deletions.
diff --git a/charts/argo-cd/templates/argocd-configs/argocd-notifications-cm.yaml b/charts/argo-cd/templates/argocd-configs/argocd-notifications-cm.yaml
@@ -1,10 +1,4 @@
 {{- if and .Values.notifications.enabled .Values.notifications.cm.create }}
-  {{- $subscriptions := slice }}
-  {{- range $key, $value := .Values.notifications.subscriptions }}
-    {{- $subscriptions = append $subscriptions $key }}
-  {{- end }}
-
-  {{- $subscriptions = append $subscriptions .Values.codefresh.promotions.notifications.subscriptions }}
 apiVersion: v1
 kind: ConfigMap
 metadata:
@@ -21,7 +15,7 @@ data:
   {{- with $notifiers }}
     {{- toYaml . | nindent 2 }}
   {{- end }}
-  {{- with $subscriptions }}
+  {{- with (append .Values.notifications.subscriptions .Values.codefresh.promotions.notifications.subscriptions) }}
   subscriptions: |
     {{- toYaml . | nindent 4 }}
   {{- end }}