Skip to content

Commit

Permalink
chore(argo-rollouts): add option to disable creation of notifications…
Browse files Browse the repository at this point in the history
… configmap (#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]>
  • Loading branch information
rolandkool and yu-croco committed Jul 29, 2024
1 parent 89bc396 commit 87e6dd9
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 2 deletions.
4 changes: 2 additions & 2 deletions charts/argo-rollouts/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
appVersion: v1.7.1
description: A Helm chart for Argo Rollouts
name: argo-rollouts
version: 2.37.2
version: 2.37.3
home: https://github.com/argoproj/argo-helm
icon: https://argoproj.github.io/argo-rollouts/assets/logo.png
keywords:
Expand All @@ -19,4 +19,4 @@ annotations:
url: https://argoproj.github.io/argo-helm/pgp_keys.asc
artifacthub.io/changes: |
- kind: added
description: Added traefik.io apiGroup to Role and ClusterRole
description: Added setting to disable creation of the notifications ConfigMap
1 change: 1 addition & 0 deletions charts/argo-rollouts/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ For full list of changes please check ArtifactHub [changelog].
| keepCRDs | bool | `true` | Keep CRD's on helm uninstall |
| kubeVersionOverride | string | `""` | Override the Kubernetes version, which is used to evaluate certain manifests |
| nameOverride | string | `nil` | String to partially override "argo-rollouts.fullname" template |
| notifications.configmap.create | bool | `true` | Whether to create notifications configmap |
| notifications.notifiers | object | `{}` | Configures notification services |
| notifications.secret.annotations | object | `{}` | Annotations to be added to the notifications secret |
| notifications.secret.create | bool | `false` | Whether to create notifications secret |
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
{{ if .Values.notifications.configmap.create }}
apiVersion: v1
kind: ConfigMap
metadata:
Expand All @@ -20,3 +21,4 @@ data:
subscriptions: |
{{- toYaml . | nindent 4 }}
{{- end }}
{{- end }}
4 changes: 4 additions & 0 deletions charts/argo-rollouts/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -449,6 +449,10 @@ dashboard:
volumeMounts: []

notifications:
configmap:
# -- Whether to create notifications configmap
create: true

secret:
# -- Whether to create notifications secret
create: false
Expand Down

0 comments on commit 87e6dd9

Please sign in to comment.