Skip to content

Commit 6956436

Browse files
authored
Merge pull request #217 from k8s-infra-cherrypick-robot/cherry-pick-216-to-release-0.5
[release-0.5] 🐛 Fix imagePullSecrets bug
2 parents d9e33a9 + e5bb68e commit 6956436

File tree

2 files changed

+5
-11
lines changed

2 files changed

+5
-11
lines changed

hack/charts/cluster-api-operator/templates/deployment.yaml

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,10 @@ spec:
5151
securityContext:
5252
{{- toYaml . | nindent 8 }}
5353
{{- end }}
54+
{{- with .Values.imagePullSecrets }}
55+
imagePullSecrets:
56+
{{- toYaml . | nindent 8 }}
57+
{{- end }}
5458
containers:
5559
- args:
5660
{{- if .Values.logLevel }}
@@ -80,10 +84,6 @@ spec:
8084
image: "{{- if .registry -}}{{ .registry }}/{{- end -}}{{ .repository }}{{- if (.digest) -}} @{{ .digest }}{{- else -}}:{{ default $.Chart.AppVersion .tag }} {{- end -}}"
8185
{{- end }}
8286
imagePullPolicy: {{ .Values.image.manager.pullPolicy }}
83-
{{- with .Values.imagePullSecrets.manager }}
84-
imagePullSecrets:
85-
{{- toYaml . | nindent 12 }}
86-
{{- end }}
8787
name: manager
8888
ports:
8989
- containerPort: 9443
@@ -112,10 +112,6 @@ spec:
112112
image: "{{- if .registry -}}{{ .registry }}/{{- end -}}{{ .repository }}{{- if (.digest) -}} @{{ .digest }}{{- else -}}:{{ default $.Chart.AppVersion .tag }} {{- end -}}"
113113
{{- end }}
114114
imagePullPolicy: {{ .Values.image.kubeRBACProxy.pullPolicy }}
115-
{{- with .Values.imagePullSecrets.kubeRBACProxy }}
116-
imagePullSecrets:
117-
{{- toYaml . | nindent 12 }}
118-
{{- end }}
119115
name: kube-rbac-proxy
120116
ports:
121117
- containerPort: 8443

hack/charts/cluster-api-operator/values.yaml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,7 @@ image:
3232
pullPolicy: IfNotPresent
3333
healthAddr: ":8081"
3434
metricsBindAddr: "127.0.0.1:8080"
35-
imagePullSecrets:
36-
manager: {}
37-
kubeRBACProxy: {}
35+
imagePullSecrets: {}
3836
resources:
3937
manager:
4038
limits:

0 commit comments

Comments
 (0)