Skip to content

Commit

Permalink
fix: add configurable automountServiceAccountToken
Browse files Browse the repository at this point in the history
Signed-off-by: Stefan Caraiman <[email protected]>
  • Loading branch information
stefan-caraiman committed Apr 6, 2024
1 parent 3fb8a8d commit 5b2f372
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 6 deletions.
2 changes: 1 addition & 1 deletion keda/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ kubeVersion: ">=v1.23.0-0"

# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
version: 2.13.2
version: 2.13.3

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application.
Expand Down
5 changes: 3 additions & 2 deletions keda/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ helm repo add kedacore https://kedacore.github.io/charts
helm repo update

kubectl create namespace keda
helm install keda kedacore/keda --namespace keda --version 2.13.0
helm install keda kedacore/keda --namespace keda --version 2.13.1
```

## Introduction
Expand All @@ -36,7 +36,7 @@ To install the chart with the release name `keda`:

```console
$ kubectl create namespace keda
$ helm install keda kedacore/keda --namespace keda --version 2.13.0
$ helm install keda kedacore/keda --namespace keda --version 2.13.1
```

## Uninstalling the Chart
Expand All @@ -62,6 +62,7 @@ their default values.
| `additionalLabels` | object | `{}` | Custom labels to add into metadata |
| `affinity` | object | `{}` | [Affinity] for pod scheduling for both KEDA operator and Metrics API Server |
| `asciiArt` | bool | `true` | Capability to turn on/off ASCII art in Helm installation notes |
| `automountServiceAccountToken` | bool | `true` | Automount API credentials for the Service Account into the pod. |
| `certificates.autoGenerated` | bool | `true` | Enables the self generation for KEDA TLS certificates inside KEDA operator |
| `certificates.certManager.caSecretName` | string | `"kedaorg-ca"` | Secret name where the CA is stored (generatedby cert-manager or user given) |
| `certificates.certManager.duration` | string | `"8760h0m0s"` | Certificate duration |
Expand Down
2 changes: 1 addition & 1 deletion keda/templates/manager/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ spec:
{{- toYaml . | nindent 8 }}
{{- end }}
serviceAccountName: {{ .Values.serviceAccount.name }}
automountServiceAccountToken: true
automountServiceAccountToken: {{ .Values.automountServiceAccountToken }}
securityContext:
{{- if .Values.podSecurityContext.operator }}
{{- toYaml .Values.podSecurityContext.operator | nindent 8 }}
Expand Down
2 changes: 1 addition & 1 deletion keda/templates/metrics-server/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ spec:
{{- toYaml . | nindent 8 }}
{{- end }}
serviceAccountName: {{ .Values.serviceAccount.name }}
automountServiceAccountToken: true
automountServiceAccountToken: {{ .Values.automountServiceAccountToken }}
securityContext:
{{- if .Values.podSecurityContext.metricServer }}
{{- toYaml .Values.podSecurityContext.metricServer | nindent 8 }}
Expand Down
2 changes: 1 addition & 1 deletion keda/templates/webhooks/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ spec:
{{- toYaml . | nindent 8 }}
{{- end }}
serviceAccountName: {{ .Values.serviceAccount.name }}
automountServiceAccountToken: true
automountServiceAccountToken: {{ .Values.automountServiceAccountToken }}
securityContext:
{{- if .Values.podSecurityContext.webhooks }}
{{- toYaml .Values.podSecurityContext.webhooks | nindent 8 }}
Expand Down
3 changes: 3 additions & 0 deletions keda/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -275,6 +275,9 @@ rbac:
# -- Specifies whether RBAC for CRDs should be [aggregated](https://kubernetes.io/docs/reference/access-authn-authz/rbac/#aggregated-clusterroles) to default roles (view, edit, admin)
aggregateToDefaultRoles: false

# -- Automount API credentials for the Service Account into the pod.
automountServiceAccountToken: true

serviceAccount:
# -- Specifies whether a service account should be created
create: true
Expand Down

0 comments on commit 5b2f372

Please sign in to comment.