Skip to content

Commit fc16965

Browse files
Make it possible to not create service accounts for charts (#23)
This was already possible in the azure-loganalytics-exporter chart and support for this had already been added to the `_helper.yaml` files. I've bumped the patch version of each chart since the change is mostly backwards compatible. If the user has explicitly changed the serviceAccountName to nil then there service account used changes from default to the fullname of the chart. But in this case Helm would have been trying to create a service account with the name default, which would not work. --------- Co-authored-by: Markus Blaschke <mblaschke82@gmail.com>
1 parent a268f36 commit fc16965

File tree

18 files changed

+24
-6
lines changed

18 files changed

+24
-6
lines changed

charts/azure-janitor/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ name: azure-janitor
33
type: application
44
description: A Helm chart for azure-janitor
55
home: https://github.com/webdevops/azure-janitor
6-
version: 1.0.7
6+
version: 1.0.8
77
# renovate: image=webdevops/azure-janitor
88
appVersion: 22.9.0
99
keywords:

charts/azure-janitor/templates/serviceaccount.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
{{- if .Values.serviceAccount.create }}
12
apiVersion: v1
23
kind: ServiceAccount
34
metadata:
@@ -15,3 +16,4 @@ metadata:
1516
imagePullSecrets:
1617
{{ include "azure-janitor.imagePullSecrets" . | trim | indent 2 }}
1718
{{- end }}
19+
{{- end }}

charts/azure-janitor/values.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ readinessProbe:
7171
failureThreshold: 5
7272

7373
serviceAccount:
74+
create: true
7475
name: azure-janitor
7576
labels: {}
7677
annotations: {}

charts/azure-keyvault-exporter/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ name: azure-keyvault-exporter
33
type: application
44
description: A Helm chart for azure-keyvault-exporter
55
home: https://github.com/webdevops/azure-keyvault-exporter
6-
version: 1.0.6
6+
version: 1.0.7
77
# renovate: image=webdevops/azure-keyvault-exporter
88
appVersion: 23.7.1
99
keywords:

charts/azure-keyvault-exporter/templates/serviceaccount.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
{{- if .Values.serviceAccount.create }}
12
apiVersion: v1
23
kind: ServiceAccount
34
metadata:
@@ -15,3 +16,4 @@ metadata:
1516
imagePullSecrets:
1617
{{ include "azure-keyvault-exporter.imagePullSecrets" . | trim | indent 2 }}
1718
{{- end }}
19+
{{- end }}

charts/azure-keyvault-exporter/values.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ readinessProbe:
7171
failureThreshold: 5
7272

7373
serviceAccount:
74+
create: true
7475
name: azure-keyvault-exporter
7576
labels: {}
7677
annotations: {}

charts/azure-metrics-exporter/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ name: azure-metrics-exporter
33
type: application
44
description: A Helm chart for azure-metrics-exporter
55
home: https://github.com/webdevops/azure-metrics-exporter
6-
version: 1.0.6
6+
version: 1.0.7
77
# renovate: image=webdevops/azure-metrics-exporter
88
appVersion: 23.7.0
99
keywords:

charts/azure-metrics-exporter/templates/serviceaccount.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
{{- if .Values.serviceAccount.create }}
12
apiVersion: v1
23
kind: ServiceAccount
34
metadata:
@@ -15,3 +16,4 @@ metadata:
1516
imagePullSecrets:
1617
{{ include "azure-metrics-exporter.imagePullSecrets" . | trim | indent 2 }}
1718
{{- end }}
19+
{{- end }}

charts/azure-metrics-exporter/values.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ readinessProbe:
7171
failureThreshold: 5
7272

7373
serviceAccount:
74+
create: true
7475
name: azure-metrics-exporter
7576
labels: {}
7677
annotations: {}

charts/azure-resourcemanager-exporter/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ name: azure-resourcemanager-exporter
33
type: application
44
description: A Helm chart for azure-resourcemanager-exporter
55
home: https://github.com/webdevops/azure-resourcemanager-exporter
6-
version: 1.2.1
6+
version: 1.2.2
77
# renovate: image=webdevops/azure-resourcemanager-exporter
88
appVersion: 23.6.1
99
keywords:

0 commit comments

Comments
 (0)