Skip to content

Commit

Permalink
feat: configurable automountServiceAccountToken
Browse files Browse the repository at this point in the history
Signed-off-by: Kyle Cogswell <[email protected]>
Signed-off-by: Kyle Cogswell <[email protected]>
  • Loading branch information
Kyle Cogswell committed Nov 13, 2024
1 parent 9361018 commit 92d7cd0
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 4 deletions.
6 changes: 3 additions & 3 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.2
description: A Helm chart for Argo Rollouts
name: argo-rollouts
version: 2.37.8
version: 2.37.9
home: https://github.com/argoproj/argo-helm
icon: https://argoproj.github.io/argo-rollouts/assets/logo.png
keywords:
Expand All @@ -18,5 +18,5 @@ annotations:
fingerprint: 2B8F22F57260EFA67BE1C5824B11F800CD9D2252
url: https://argoproj.github.io/argo-helm/pgp_keys.asc
artifacthub.io/changes: |
- kind: fixed
description: Fixed rendering of plugins in the ConfigMap
- kind: changed
description: Add configurable automountServiceAccountToken
1 change: 1 addition & 0 deletions charts/argo-rollouts/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,7 @@ For full list of changes please check ArtifactHub [changelog].
| podLabels | object | `{}` | Labels to be added to the Rollout pods |
| podSecurityContext | object | `{"runAsNonRoot":true}` | Security Context to set on pod level |
| serviceAccount.annotations | object | `{}` | Annotations to add to the service account |
| serviceAccount.automount | bool | `true` | Specifies wether a service account should be automounted |
| serviceAccount.create | bool | `true` | Specifies whether a service account should be created |
| serviceAccount.name | string | `""` | The name of the service account to use. If not set and create is true, a name is generated using the fullname template |
| serviceAnnotations | object | `{}` | Annotations to be added to the Rollout service |
Expand Down
1 change: 1 addition & 0 deletions charts/argo-rollouts/templates/controller/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ spec:
{{- toYaml . | nindent 8 }}
{{- end }}
serviceAccountName: {{ include "argo-rollouts.serviceAccountName" . }}
automountServiceAccountToken: {{ .Values.serviceAccount.automount }}
containers:
- image: "{{ .Values.controller.image.registry }}/{{ .Values.controller.image.repository }}:{{ default .Chart.AppVersion .Values.controller.image.tag }}"
args:
Expand Down
1 change: 1 addition & 0 deletions charts/argo-rollouts/templates/dashboard/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@ spec:
{{- toYaml . | nindent 8 }}
{{- end }}
serviceAccountName: {{ include "argo-rollouts.serviceAccountName" . }}-dashboard
automountServiceAccountToken: {{ .Values.dashboard.serviceAccount.automount }}
containers:
- image: "{{ .Values.dashboard.image.registry }}/{{ .Values.dashboard.image.repository }}:{{ default .Chart.AppVersion .Values.dashboard.image.tag }}"
imagePullPolicy: {{ .Values.dashboard.image.pullPolicy }}
Expand Down
6 changes: 5 additions & 1 deletion charts/argo-rollouts/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -229,6 +229,8 @@ serviceAccount:
# -- The name of the service account to use.
# If not set and create is true, a name is generated using the fullname template
name: ""
# -- Specifies wether a service account should be automounted
automount: true

# -- Annotations to be added to all CRDs
crdAnnotations: {}
Expand Down Expand Up @@ -384,7 +386,9 @@ dashboard:
# -- The name of the service account to use.
# If not set and create is true, a name is generated using the fullname template
name: ""

# -- Specifies wether a service account should be automounted
automount: true

## Configure Pod Disruption Budget for the dashboard
pdb:
# -- Labels to be added to dashboard [Pod Disruption Budget]
Expand Down

0 comments on commit 92d7cd0

Please sign in to comment.