Skip to content

Commit

Permalink
fix: ServiceAccount are correctly annotated
Browse files Browse the repository at this point in the history
Signed-off-by: Jorge Turrado <jorge.turrado@scrm.lidl>
  • Loading branch information
JorTurFer committed May 1, 2024
1 parent 9ab7049 commit 29aae76
Show file tree
Hide file tree
Showing 8 changed files with 121 additions and 89 deletions.
2 changes: 1 addition & 1 deletion 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 search repo kedacore
NAME CHART VERSION APP VERSION DESCRIPTION
kedacore/external-scaler-azure-cosmos-db 0.1.0 0.1.0 Event-based autoscaler for Azure Cosmos DB chan...
kedacore/keda 2.14.1 2.14.0 Event-based autoscaler for workloads on Kubernetes
kedacore/keda 2.14.2 2.14.0 Event-based autoscaler for workloads on Kubernetes
kedacore/keda-add-ons-http 0.7.0 0.7.0 Event-based autoscaler for HTTP workloads on Ku...
```

Expand Down
175 changes: 99 additions & 76 deletions docs/index.yaml

Large diffs are not rendered by default.

Binary file added docs/keda-2.14.2.tgz
Binary file not shown.
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.14.1
version: 2.14.2

# 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
2 changes: 1 addition & 1 deletion keda/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -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.14.1
$ helm install keda kedacore/keda --namespace keda --version 2.14.2
```

## Uninstalling the Chart
Expand Down
11 changes: 7 additions & 4 deletions keda/templates/manager/serviceaccount.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{{- if (.Values.serviceAccount.operator).create | default .Values.serviceAccount.create -}}
{{- if or (.Values.serviceAccount.operator).create .Values.serviceAccount.create -}}
apiVersion: v1
kind: ServiceAccount
metadata:
Expand All @@ -8,7 +8,7 @@ metadata:
azure.workload.identity/use: "true"
{{- end }}
{{- include "keda.labels" . | nindent 4 }}
{{- if or .Values.podIdentity.azureWorkload.enabled .Values.podIdentity.aws.irsa.enabled ((.Values.serviceAccount.operator).annotations | default .Values.serviceAccount.annotations) .Values.podIdentity.gcp.enabled }}
{{- if or .Values.podIdentity.azureWorkload.enabled .Values.podIdentity.aws.irsa.enabled (.Values.serviceAccount.operator).annotations .Values.serviceAccount.annotations .Values.podIdentity.gcp.enabled }}
annotations:
{{- if .Values.additionalAnnotations }}
{{- toYaml .Values.additionalAnnotations | nindent 4 }}
Expand Down Expand Up @@ -39,8 +39,11 @@ metadata:
iam.gke.io/gcp-service-account: {{ .Values.podIdentity.gcp.gcpIAMServiceAccount }}
{{- end }}
{{- end }}
{{- if (.Values.serviceAccount.operator).annotations | default .Values.serviceAccount.annotations }}
{{- toYaml (.Values.serviceAccount.operator).annotations | default .Values.serviceAccount.annotations | nindent 4}}
{{- if (.Values.serviceAccount.operator).annotations }}
{{- toYaml (.Values.serviceAccount.operator).annotations | nindent 4}}
{{- end }}
{{- if .Values.serviceAccount.annotations }}
{{- toYaml .Values.serviceAccount.annotations | nindent 4}}
{{- end }}
{{- end }}
name: {{ (.Values.serviceAccount.operator).name | default .Values.serviceAccount.name }}
Expand Down
9 changes: 6 additions & 3 deletions keda/templates/metrics-server/serviceaccount.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{{- if (.Values.serviceAccount.metricServer).create | default .Values.serviceAccount.create -}}
{{- if or (.Values.serviceAccount.metricServer).create .Values.serviceAccount.create -}}
apiVersion: v1
kind: ServiceAccount
metadata:
Expand All @@ -9,8 +9,11 @@ metadata:
{{- if .Values.additionalAnnotations }}
{{- toYaml .Values.additionalAnnotations | nindent 4 }}
{{- end }}
{{- if (.Values.serviceAccount.metricServer).annotations | default .Values.serviceAccount.annotations }}
{{- toYaml ((.Values.serviceAccount.metricServer).annotations | default .Values.serviceAccount.annotations) | nindent 4}}
{{- if (.Values.serviceAccount.metricServer).annotations }}
{{- toYaml (.Values.serviceAccount.metricServer).annotations | nindent 4}}
{{- end }}
{{- if .Values.serviceAccount.annotations }}
{{- toYaml .Values.serviceAccount.annotations | nindent 4}}
{{- end }}
name: {{ (.Values.serviceAccount.metricServer).name | default .Values.serviceAccount.name }}
namespace: {{ .Release.Namespace }}
Expand Down
9 changes: 6 additions & 3 deletions keda/templates/webhooks/serviceaccount.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{{- if (.Values.serviceAccount.webhooks).create | default .Values.serviceAccount.create -}}
{{- if or (.Values.serviceAccount.webhooks).create .Values.serviceAccount.create -}}
apiVersion: v1
kind: ServiceAccount
metadata:
Expand All @@ -9,8 +9,11 @@ metadata:
{{- if .Values.additionalAnnotations }}
{{- toYaml .Values.additionalAnnotations | nindent 4 }}
{{- end }}
{{- if (.Values.serviceAccount.webhooks).annotations | default .Values.serviceAccount.annotations }}
{{- toYaml ((.Values.serviceAccount.webhooks).annotations | default .Values.serviceAccount.annotations) | nindent 4}}
{{- if (.Values.serviceAccount.webhooks).annotations }}
{{- toYaml (.Values.serviceAccount.webhooks).annotations | nindent 4}}
{{- end }}
{{- if .Values.serviceAccount.annotations }}
{{- toYaml .Values.serviceAccount.annotations | nindent 4}}
{{- end }}
name: {{ (.Values.serviceAccount.webhooks).name | default .Values.serviceAccount.name }}
namespace: {{ .Release.Namespace }}
Expand Down

0 comments on commit 29aae76

Please sign in to comment.