Skip to content

Commit 5a9ccbf

Browse files
committed
fix: properly capitalize 'filesystem' in values
Use the proper capitalization in the reference to the value reloader.readOnlyRootFileSystem: FileSystem instead of Filesystem.
1 parent 451e4f6 commit 5a9ccbf

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

deployments/kubernetes/chart/reloader/templates/deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ spec:
149149
successThreshold: {{ .Values.reloader.deployment.readinessProbe.successThreshold | default "1" }}
150150

151151
{{- $containerSecurityContext := .Values.reloader.deployment.containerSecurityContext | default dict }}
152-
{{- if .Values.reloader.readOnlyRootFilesystem }}
152+
{{- if .Values.reloader.readOnlyRootFileSystem }}
153153
{{- $_ := set $containerSecurityContext "readOnlyRootFilesystem" true }}
154154
{{- end }}
155155

deployments/kubernetes/chart/reloader/tests/deployment_test.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@ templates:
44
- deployment.yaml
55

66
tests:
7-
- it: sets readOnlyRootFilesystem in container securityContext when reloader.readOnlyRootFilesystem is true
7+
- it: sets readOnlyRootFilesystem in container securityContext when reloader.readOnlyRootFileSystem is true
88
set:
99
reloader:
10-
readOnlyRootFilesystem: true
10+
readOnlyRootFileSystem: true
1111
deployment:
1212
containerSecurityContext:
1313
readOnlyRootFilesystem: false
@@ -19,18 +19,18 @@ tests:
1919
- it: sets readOnlyRootFilesystem in container securityContext even if reloader.deployment.containerSecurityContext is null
2020
set:
2121
reloader:
22-
readOnlyRootFilesystem: true
22+
readOnlyRootFileSystem: true
2323
deployment:
2424
containerSecurityContext: null
2525
asserts:
2626
- equal:
2727
path: spec.template.spec.containers[0].securityContext.readOnlyRootFilesystem
2828
value: true
2929

30-
- it: does not override readOnlyRootFilesystem in container securityContext based on reloader.readOnlyRootFilesystem
30+
- it: does not override readOnlyRootFilesystem in container securityContext based on reloader.readOnlyRootFileSystem
3131
set:
3232
reloader:
33-
readOnlyRootFilesystem: false
33+
readOnlyRootFileSystem: false
3434
deployment:
3535
containerSecurityContext:
3636
readOnlyRootFilesystem: true
@@ -42,7 +42,7 @@ tests:
4242
- it: template is still valid with no defined containerSecurityContext
4343
set:
4444
reloader:
45-
readOnlyRootFilesystem: false
45+
readOnlyRootFileSystem: false
4646
deployment:
4747
containerSecurityContext: null
4848
asserts:

0 commit comments

Comments
 (0)