Skip to content

Commit

Permalink
[CU-86b1a2uaj] Update probes so they meet our needs (#53)
Browse files Browse the repository at this point in the history
- remove readiness probe as app can accept traffic right after startup
- add startup probe
- update liveness probe to reasonable values
  • Loading branch information
jstromsky authored Jul 29, 2024
1 parent e7540d6 commit ecaa87b
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions helm/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -177,22 +177,23 @@ spec:
memory: {{ .Values.app.memoryLimit }}
requests:
memory: {{ .Values.app.memoryLimit }}
readinessProbe:
startupProbe:
httpGet:
path: /actuator/health
port: 8080
scheme: HTTP
initialDelaySeconds: 10
periodSeconds: 5
timeoutSeconds: 120
periodSeconds: 10
timeoutSeconds: 2
failureThreshold: 18 # this will result in a 3 minutes timeout (periodSeconds * failureThreshold)
livenessProbe:
httpGet:
path: /actuator/health
port: 8080
scheme: HTTP
initialDelaySeconds: 30
periodSeconds: 10
timeoutSeconds: 10
timeoutSeconds: 2
failureThreshold: 1
env:
- name: JAVA_OPTS
value: "-XX:+PrintFlagsFinal -XX:+ExitOnOutOfMemoryError"
Expand Down

0 comments on commit ecaa87b

Please sign in to comment.