Merge pull request #10 from alexandrud/api-chart

change healthcheck defaults to allow the pod to start
alexandrud · Jan 23, 2021 · cbd2002 · cbd2002
2 parents 3c14ac8 + 8f5b5f1
commit cbd2002
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/charts/routing/Chart.yaml b/charts/routing/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: routing
 description: Helm chart for routing app
 type: application
-version: 0.0.3
+version: 0.0.4
 appVersion: "1.11.1"
 keywords:
 - routing

diff --git a/charts/routing/templates/deployment.yaml b/charts/routing/templates/deployment.yaml
@@ -48,10 +48,14 @@ spec:
             httpGet:
               path: {{ .Values.healthCheckPath }}
               port: http
+            timeoutSeconds: {{ .Values.healthCheckTimeout }}
+            initialDelaySeconds: {{ .Values.healthCheckDelay }}
           readinessProbe:
             httpGet:
               path: {{ .Values.healthCheckPath }}
               port: http
+            timeoutSeconds: {{ .Values.healthCheckTimeout }}
+            initialDelaySeconds: {{ .Values.healthCheckDelay }}
           resources:
             {{- toYaml .Values.resources | nindent 12 }}
       {{- with .Values.nodeSelector }}

diff --git a/charts/routing/values.yaml b/charts/routing/values.yaml
@@ -40,6 +40,8 @@ env: {}
 
 containerPort: 3001
 healthCheck: ""
+healthCheckTimeout: 3
+healthCheckDelay: 10
 
 service:
   type: ClusterIP