fix: add readiness probe for lms

eduNEXT · Jul 17, 2024 · 5e2af5d · 5e2af5d
1 parent 397d7dc
commit 5e2af5d
Showing 1 changed file with 15 additions and 7 deletions.
diff --git a/drydock/templates/drydock/k8s/lifecycle/lms.yml b/drydock/templates/drydock/k8s/lifecycle/lms.yml
@@ -13,15 +13,23 @@ spec:
                 command:
                 - "sleep"
                 - "30"
-          startupProbe:
+          startupProbe: # Make sure the service has started correctly
+          # We provide 10 * 5 seconds to the service to start before restarting the pod
+            successThreshold: 1
+            failureThreshold: 10
+            initialDelaySeconds: 5
+            timeoutSeconds: 3
+            periodSeconds: 5
             httpGet:
-              httpHeaders:
-                - name: Host
-                  value: {{ LMS_HOST }}
               path: /heartbeat
               port: 8000
-            initialDelaySeconds: 5
-            timeoutSeconds: 3
+          readinessProbe: # Make sure the pod can answer traffic before handling any requests
+          # Only route traffic if the pod is alive, and has passed two succesful checks (max 10s)
+            successThreshold: 2
+            failureThreshold: 2
             periodSeconds: 5
-            failureThreshold: 5
+            timeoutSeconds: 3
+            httpGet:
+              path: /heartbeat
+              port: 8000
       terminationGracePeriodSeconds: 60