Skip to content

Commit cc4bdfa

Browse files
committed
Making use of .nginx.containerPort when fpm is enabled
Signed-off-by: Heðin Ejdesgaard <hej@ejdesgaard.fo>
1 parent c4c9bf4 commit cc4bdfa

2 files changed

Lines changed: 8 additions & 4 deletions

File tree

charts/nextcloud/templates/deployment.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -150,13 +150,13 @@ spec:
150150
ports:
151151
- name: http
152152
protocol: TCP
153-
containerPort: {{ .Values.nextcloud.containerPort }}
153+
containerPort: {{ .Values.nginx.containerPort }}
154154
{{- with .Values.livenessProbe }}
155155
{{- if .enabled }}
156156
livenessProbe:
157157
httpGet:
158158
path: /status.php
159-
port: {{ $.Values.nextcloud.containerPort }}
159+
port: {{ $.Values.nginx.containerPort }}
160160
httpHeaders:
161161
- name: Host
162162
value: {{ $.Values.nextcloud.host | quote }}
@@ -172,7 +172,7 @@ spec:
172172
readinessProbe:
173173
httpGet:
174174
path: /status.php
175-
port: {{ $.Values.nextcloud.containerPort }}
175+
port: {{ $.Values.nginx.containerPort }}
176176
httpHeaders:
177177
- name: Host
178178
value: {{ $.Values.nextcloud.host | quote }}
@@ -188,7 +188,7 @@ spec:
188188
startupProbe:
189189
httpGet:
190190
path: /status.php
191-
port: {{ $.Values.nextcloud.containerPort }}
191+
port: {{ $.Values.nginx.containerPort }}
192192
httpHeaders:
193193
- name: Host
194194
value: {{ $.Values.nextcloud.host | quote }}

charts/nextcloud/templates/service.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,11 @@ spec:
3434
ports:
3535
- name: http
3636
port: {{ .Values.service.port }}
37+
{{- if .Values.nginx.enabled }}
38+
targetPort: {{ .Values.nginx.containerPort }}
39+
{{- else }}
3740
targetPort: {{ .Values.nextcloud.containerPort }}
41+
{{- end }}
3842
protocol: TCP
3943
{{- with .Values.service.nodePort }}
4044
nodePort: {{ . }}

0 commit comments

Comments
 (0)