Skip to content

Commit bf6f73f

Browse files
committed
apply code review suggestion
Signed-off-by: rene <rene.raab@gmx.net>
1 parent fe44323 commit bf6f73f

2 files changed

Lines changed: 13 additions & 8 deletions

File tree

charts/nextcloud/templates/deployment.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,7 @@ spec:
293293
{{- if .Values.mariadb.enabled }}
294294
- name: mariadb-isalive
295295
image: {{ .Values.mariadb.image.registry | default "docker.io" }}/{{ .Values.mariadb.image.repository }}:{{ .Values.mariadb.image.tag }}
296-
{{- with .Values.nextcloud.mariadbInitContainerSecurityContext }}
296+
{{- with .Values.nextcloud.mariaDbInitContainer.securityContext }}
297297
securityContext:
298298
{{- toYaml . | nindent 12 }}
299299
{{- end }}
@@ -315,7 +315,7 @@ spec:
315315
{{- else if .Values.postgresql.enabled }}
316316
- name: postgresql-isready
317317
image: {{ .Values.postgresql.image.registry | default "docker.io" }}/{{ .Values.postgresql.image.repository }}:{{ .Values.postgresql.image.tag }}
318-
{{- with .Values.nextcloud.postgresqlInitContainerSecurityContext }}
318+
{{- with .Values.nextcloud.postgreSqlInitContainer.securityContext }}
319319
securityContext:
320320
{{- toYaml . | nindent 12 }}
321321
{{- end }}

charts/nextcloud/values.yaml

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -213,19 +213,24 @@ nextcloud:
213213
# runAsNonRoot: true
214214
# readOnlyRootFilesystem: false
215215

216-
# Set mariadb initContainer securityContext parameters. For example, you may need to define runAsNonRoot directive
217-
mariadbInitContainerSecurityContext: {}
218-
219-
# Set postgresql initContainer securityContext parameters. For example, you may need to define runAsNonRoot directive
220-
postgresqlInitContainerSecurityContext: {}
221-
222216
# Set securityContext parameters for the entire pod. For example, you may need to define runAsNonRoot directive
223217
podSecurityContext: {}
224218
# runAsUser: 33
225219
# runAsGroup: 33
226220
# runAsNonRoot: true
227221
# readOnlyRootFilesystem: false
228222

223+
# Settings for the MariaDB init container
224+
mariaDbInitContainer:
225+
# Set mariadb initContainer securityContext parameters. For example, you may need to define runAsNonRoot directive
226+
securityContext: {}
227+
228+
# Settings for the PostgreSQL init container
229+
postgreSqlInitContainer:
230+
# Set postgresql initContainer securityContext parameters. For example, you may need to define runAsNonRoot directive
231+
securityContext: {}
232+
233+
229234
nginx:
230235
## You need to set an fpm version of the image for nextcloud if you want to use nginx!
231236
enabled: false

0 commit comments

Comments
 (0)