templates: hook up simple probes and default limits
Use fetching the OpenAPI spec as a simple readiness/liveness, as there is not much else we can/need to verify. Set the default CPU and memory limits in accordance with AppSRE requirements. Signed-off-by: Tom Gundersen <teg@jklm.no>
This commit is contained in:
parent
b0f36fccd3
commit
6a671112f0
1 changed files with 43 additions and 0 deletions
|
|
@ -39,6 +39,31 @@ objects:
|
||||||
containers:
|
containers:
|
||||||
- image: "${IMAGE_NAME}:${IMAGE_TAG}"
|
- image: "${IMAGE_NAME}:${IMAGE_TAG}"
|
||||||
name: composer
|
name: composer
|
||||||
|
livenessProbe:
|
||||||
|
failureThreshold: 3
|
||||||
|
httpGet:
|
||||||
|
path: ${LIVENESS_URI}
|
||||||
|
port: ${{COMPOSER_API_PORT}}
|
||||||
|
scheme: HTTP
|
||||||
|
periodSeconds: 10
|
||||||
|
successThreshold: 1
|
||||||
|
timeoutSeconds: 1
|
||||||
|
readinessProbe:
|
||||||
|
failureThreshold: 3
|
||||||
|
httpGet:
|
||||||
|
path: ${READINESS_URI}
|
||||||
|
port: ${{COMPOSER_API_PORT}}
|
||||||
|
scheme: HTTP
|
||||||
|
periodSeconds: 10
|
||||||
|
successThreshold: 1
|
||||||
|
timeoutSeconds: 1
|
||||||
|
resources:
|
||||||
|
requests:
|
||||||
|
cpu: "${CPU_REQUEST}"
|
||||||
|
memory: "${MEMORY_REQUEST}"
|
||||||
|
limits:
|
||||||
|
cpu: "${CPU_LIMIT}"
|
||||||
|
memory: "${MEMORY_LIMIT}"
|
||||||
env:
|
env:
|
||||||
- name: PGHOST
|
- name: PGHOST
|
||||||
valueFrom:
|
valueFrom:
|
||||||
|
|
@ -218,3 +243,21 @@ parameters:
|
||||||
name: WORKER_API_PORT
|
name: WORKER_API_PORT
|
||||||
required: true
|
required: true
|
||||||
value: "8700"
|
value: "8700"
|
||||||
|
- name: LIVENESS_URI
|
||||||
|
description: URI to query for the liveness check
|
||||||
|
value: "/openapi"
|
||||||
|
- name: READINESS_URI
|
||||||
|
description: URI to query for the readiness check
|
||||||
|
value: "/openapi"
|
||||||
|
- name: CPU_REQUEST
|
||||||
|
description: CPU request per container
|
||||||
|
value: "200m"
|
||||||
|
- name: CPU_LIMIT
|
||||||
|
description: CPU limit per container
|
||||||
|
value: "1"
|
||||||
|
- name: MEMORY_REQUEST
|
||||||
|
description: Memory request per container
|
||||||
|
value: "256Mi"
|
||||||
|
- name: MEMORY_LIMIT
|
||||||
|
description: Memory limit per container
|
||||||
|
value: "512Mi"
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue