Skip to content

Commit 87e6dd9

Browse files
rolandkoolyu-croco
andauthored
chore(argo-rollouts): add option to disable creation of notifications configmap (argoproj#2849)
* chore(argo-rollouts): add option to disable creation of notifications configmap This allows for using the upstream notifications configmap from https://github.com/argoproj/argo-rollouts/blob/master/manifests/notifications-install.yaml Signed-off-by: Roland Kool <[email protected]> * chore: update changelog Signed-off-by: Roland Kool <[email protected]> * chore: fix chart version bump Signed-off-by: Roland Kool <[email protected]> --------- Signed-off-by: Roland Kool <[email protected]> Co-authored-by: Aikawa <[email protected]>
1 parent 89bc396 commit 87e6dd9

File tree

4 files changed

+9
-2
lines changed

4 files changed

+9
-2
lines changed

charts/argo-rollouts/Chart.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: v2
22
appVersion: v1.7.1
33
description: A Helm chart for Argo Rollouts
44
name: argo-rollouts
5-
version: 2.37.2
5+
version: 2.37.3
66
home: https://github.com/argoproj/argo-helm
77
icon: https://argoproj.github.io/argo-rollouts/assets/logo.png
88
keywords:
@@ -19,4 +19,4 @@ annotations:
1919
url: https://argoproj.github.io/argo-helm/pgp_keys.asc
2020
artifacthub.io/changes: |
2121
- kind: added
22-
description: Added traefik.io apiGroup to Role and ClusterRole
22+
description: Added setting to disable creation of the notifications ConfigMap

charts/argo-rollouts/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ For full list of changes please check ArtifactHub [changelog].
5757
| keepCRDs | bool | `true` | Keep CRD's on helm uninstall |
5858
| kubeVersionOverride | string | `""` | Override the Kubernetes version, which is used to evaluate certain manifests |
5959
| nameOverride | string | `nil` | String to partially override "argo-rollouts.fullname" template |
60+
| notifications.configmap.create | bool | `true` | Whether to create notifications configmap |
6061
| notifications.notifiers | object | `{}` | Configures notification services |
6162
| notifications.secret.annotations | object | `{}` | Annotations to be added to the notifications secret |
6263
| notifications.secret.create | bool | `false` | Whether to create notifications secret |

charts/argo-rollouts/templates/controller/notifcations-configmap.yaml renamed to charts/argo-rollouts/templates/controller/notifications-configmap.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
{{ if .Values.notifications.configmap.create }}
12
apiVersion: v1
23
kind: ConfigMap
34
metadata:
@@ -20,3 +21,4 @@ data:
2021
subscriptions: |
2122
{{- toYaml . | nindent 4 }}
2223
{{- end }}
24+
{{- end }}

charts/argo-rollouts/values.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -449,6 +449,10 @@ dashboard:
449449
volumeMounts: []
450450

451451
notifications:
452+
configmap:
453+
# -- Whether to create notifications configmap
454+
create: true
455+
452456
secret:
453457
# -- Whether to create notifications secret
454458
create: false

0 commit comments

Comments
 (0)