Skip to content

Commit

Permalink
Use container ports for proxy stateful set probes (#410)
Browse files Browse the repository at this point in the history
Using service ports cause probes to fail.

Co-authored-by: emre <[email protected]>
  • Loading branch information
emre-aydin and emre authored Dec 11, 2023
1 parent 8cb3c18 commit 9542c7b
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions charts/pulsar/templates/proxy-statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ spec:
livenessProbe:
httpGet:
path: /status.html
port: {{ .Values.proxy.ports.http }}
port: {{ .Values.proxy.ports.containerPorts.http }}
initialDelaySeconds: {{ .Values.proxy.probe.liveness.initialDelaySeconds }}
periodSeconds: {{ .Values.proxy.probe.liveness.periodSeconds }}
timeoutSeconds: {{ .Values.proxy.probe.liveness.timeoutSeconds }}
Expand All @@ -157,7 +157,7 @@ spec:
readinessProbe:
httpGet:
path: /status.html
port: {{ .Values.proxy.ports.http }}
port: {{ .Values.proxy.ports.containerPorts.http }}
initialDelaySeconds: {{ .Values.proxy.probe.readiness.initialDelaySeconds }}
periodSeconds: {{ .Values.proxy.probe.readiness.periodSeconds }}
timeoutSeconds: {{ .Values.proxy.probe.readiness.timeoutSeconds }}
Expand All @@ -167,7 +167,7 @@ spec:
startupProbe:
httpGet:
path: /status.html
port: {{ .Values.proxy.ports.http }}
port: {{ .Values.proxy.ports.containerPorts.http }}
initialDelaySeconds: {{ .Values.proxy.probe.startup.initialDelaySeconds }}
periodSeconds: {{ .Values.proxy.probe.startup.periodSeconds }}
timeoutSeconds: {{ .Values.proxy.probe.startup.timeoutSeconds }}
Expand Down

0 comments on commit 9542c7b

Please sign in to comment.