Skip to content

Commit f586829

Browse files
knabben-mmsheubeck
andauthored
[Application] Fix securityContext for sidecars (#163)
The previous implementation did override the default value which then were applied to the next sidecar if no securityContext was set for this sidecar. --------- Co-authored-by: Florian Heubeck <40993644+heubeck@users.noreply.github.com>
1 parent 2a55aa7 commit f586829

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

charts/application/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,4 @@ maintainers:
77
- name: MediaMarktSaturn
88
url: https://github.com/MediaMarktSaturn
99
appVersion: 1.0.0
10-
version: 1.29.0
10+
version: 1.29.1

charts/application/templates/_podTemplate.tpl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ spec:
6363
{{- end }}
6464
securityContext:
6565
{{- if and $i.securityContext $.Values.initDefaults.securityContext }}
66-
{{- toYaml (mergeOverwrite $.Values.initDefaults.securityContext $i.securityContext) | nindent 8 }}
66+
{{- toYaml (merge $i.securityContext $.Values.initDefaults.securityContext) | nindent 8 }}
6767
{{- else }}
6868
{{- toYaml (or $i.securityContext $.Values.initDefaults.securityContext) | nindent 8 }}
6969
{{- end }}
@@ -127,7 +127,7 @@ spec:
127127
{{- end }}
128128
securityContext:
129129
{{- if and $s.securityContext $.Values.sidecarDefaults.securityContext }}
130-
{{- toYaml (mergeOverwrite $.Values.sidecarDefaults.securityContext $s.securityContext) | nindent 8 }}
130+
{{- toYaml (merge $s.securityContext $.Values.sidecarDefaults.securityContext) | nindent 8 }}
131131
{{- else }}
132132
{{- toYaml (or $s.securityContext $.Values.sidecarDefaults.securityContext) | nindent 8 }}
133133
{{- end }}

0 commit comments

Comments
 (0)