From 111018ce2466fccead367bcf94ef9f936a3961cd Mon Sep 17 00:00:00 2001 From: Felipe Monteiro Date: Mon, 7 May 2018 18:39:49 +0100 Subject: [PATCH] [chart] Remove liveness probe to stop DH pod from being killed This is to stop the DH pod from being killed in production whenever DH receives multiple concurrent requests from another service, causing all its threads to become occupied with servicing those requests, causing the liveness probe to fail, causing the DH pod to be killed. This is highly undesirable and as a temporary workaround we will drop the liveness probe altogether. This partially reverts I1a1c107706862431e53668a864db622499e63c6f Additional reading: Id2d4deaaf8bf73d6df4639810e6dee3acf79b05c Change-Id: Ic81c0c1d6e3cd3ab3b326054b9c882962d240968 --- charts/deckhand/templates/deployment.yaml | 8 -------- 1 file changed, 8 deletions(-) diff --git a/charts/deckhand/templates/deployment.yaml b/charts/deckhand/templates/deployment.yaml index 9532ebf8..9db22f32 100644 --- a/charts/deckhand/templates/deployment.yaml +++ b/charts/deckhand/templates/deployment.yaml @@ -61,14 +61,6 @@ spec: {{ tuple $envAll $envAll.Values.pod.resources.api | include "helm-toolkit.snippets.kubernetes_resources" | indent 10 }} ports: - containerPort: {{ tuple "deckhand" "internal" "api" . | include "helm-toolkit.endpoints.endpoint_port_lookup" }} - livenessProbe: - httpGet: - scheme: HTTP - path: /api/v1.0/health - port: {{ tuple "deckhand" "internal" "api" . | include "helm-toolkit.endpoints.endpoint_port_lookup" }} - initialDelaySeconds: 15 - periodSeconds: 20 - timeoutSeconds: 10 readinessProbe: httpGet: scheme: HTTP