DevFW-CICD/argocd-helm
15
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat(argo-cd): Add priorityClassName to notifications (#1439)

Browse source 
Allow to set up a priorityClassName for notifications controller Pods

Signed-off-by: Paweł Kalemba <pawel.kalemba@getresponse.com>

Signed-off-by: Paweł Kalemba <pawel.kalemba@getresponse.com>
This commit is contained in:
Paweł Kalemba 2022-09-06 12:13:28 +02:00 committed by GitHub
parent 0bac122e98
commit db0957d3cb
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
charts/argo-cd/Chart.yaml
View file

@ -2,7 +2,7 @@ apiVersion: v2
appVersion: v2.4.11 appVersion: v2.4.11
description: A Helm chart for Argo CD, a declarative, GitOps continuous delivery tool for Kubernetes. description: A Helm chart for Argo CD, a declarative, GitOps continuous delivery tool for Kubernetes.
name: argo-cd name: argo-cd
version: 5.4.1 version: 5.4.2
home: https://github.com/argoproj/argo-helm home: https://github.com/argoproj/argo-helm
icon: https://argo-cd.readthedocs.io/en/stable/assets/logo.png icon: https://argo-cd.readthedocs.io/en/stable/assets/logo.png
keywords: keywords:
@ -19,4 +19,4 @@ dependencies:
condition: redis-ha.enabled condition: redis-ha.enabled
annotations: annotations:
artifacthub.io/changes: | artifacthub.io/changes: |
- "[Fixed]: Template existing value automountServiceAccountToken into repo-server" - "[Added]: Add priorityClassName to notifications controller"

1
charts/argo-cd/README.md
View file

@ -960,6 +960,7 @@ If you want to use an existing Redis (eg. a managed service from a cloud provide
| notifications.notifiers | object | See [values.yaml] | Configures notification services such as slack, email or custom webhook | | notifications.notifiers | object | See [values.yaml] | Configures notification services such as slack, email or custom webhook |
| notifications.podAnnotations | object | `{}` | Annotations to be applied to the controller Pods | | notifications.podAnnotations | object | `{}` | Annotations to be applied to the controller Pods |
| notifications.podLabels | object | `{}` | Labels to be applied to the controller Pods | | notifications.podLabels | object | `{}` | Labels to be applied to the controller Pods |
| notifications.priorityClassName | string | `""` | Priority class for the controller pods |
| notifications.resources | object | `{}` | Resource limits and requests for the controller | | notifications.resources | object | `{}` | Resource limits and requests for the controller |
| notifications.secret.annotations | object | `{}` | key:value pairs of annotations to be added to the secret | | notifications.secret.annotations | object | `{}` | key:value pairs of annotations to be added to the secret |
| notifications.secret.create | bool | `true` | Whether helm chart creates controller secret | | notifications.secret.create | bool | `true` | Whether helm chart creates controller secret |

3
charts/argo-cd/templates/argocd-notifications/deployment.yaml
View file

@ -100,4 +100,7 @@ spec:
tolerations: tolerations:
{{- toYaml . | nindent 8 }} {{- toYaml . | nindent 8 }}
{{- end }} {{- end }}
{{- with .Values.notifications.priorityClassName }}
priorityClassName: {{ . }}
{{- end }}
{{- end }} {{- end }}

3
charts/argo-cd/values.yaml
View file

@ -2277,6 +2277,9 @@ notifications:
# -- Container Security Context # -- Container Security Context
containerSecurityContext: {} containerSecurityContext: {}
# -- Priority class for the controller pods
priorityClassName: ""
# -- Resource limits and requests for the controller # -- Resource limits and requests for the controller
resources: {} resources: {}
# limits: # limits: