trying some liveness and readiness probes. Removing ranges on default…

… configmap
FairwindsOps · Mar 28, 2019 · c53f07a · c53f07a
1 parent 05791b1
commit c53f07a
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 4 deletions.
diff --git a/charts/fairwinds/templates/fairwinds.deployment-dashboard.yaml b/charts/fairwinds/templates/fairwinds.deployment-dashboard.yaml
@@ -37,6 +37,19 @@ spec:
         name: dashboard
         ports:
         - containerPort: 8080
+        livenessProbe:
+          exec:
+            command:
+            - sh 
+            - -c
+            - ps -ef | grep fairwinds
+          initialDelaySeconds: 5
+          periodSeconds: 5
+        readinessProbe:
+          tcpSocket:
+            port: 8080
+          initialDelaySeconds: 15
+          periodSeconds: 20
         resources:
           limits:
             cpu: 100m

diff --git a/charts/fairwinds/templates/fairwinds.deployment-webook.yaml b/charts/fairwinds/templates/fairwinds.deployment-webook.yaml
@@ -37,6 +37,19 @@ spec:
         name: webhook
         ports:
         - containerPort: 9876
+        livenessProbe:
+          exec:
+            command:
+            - sh 
+            - -c
+            - ps -ef | grep fairwinds
+          initialDelaySeconds: 5
+          periodSeconds: 5
+        readinessProbe:
+          tcpSocket:
+            port: 8080
+          initialDelaySeconds: 15
+          periodSeconds: 20
         resources:
           limits:
             cpu: 100m

diff --git a/charts/fairwinds/values.yaml b/charts/fairwinds/values.yaml
@@ -8,11 +8,9 @@ config: |
         require: true
     limits:
       cpu:
-        min: 0.1
-        max: 1
+        require: true
       memory:
-        min: 10m
-        max: 2000M
+        require: true
   healthChecks:
     readiness:
       require: true