feat: readiness/liveliness probe for multiwoven-worker (#10)

* feat: readiness probe for multiwoven-worker * feat: added liveliness probe to worker * chore: using healthPort variable instead of hardcoded port
Multiwoven · May 2, 2024 · 93b2ec2 · 93b2ec2
1 parent 0c6134b
commit 93b2ec2
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 3 deletions.
diff --git a/charts/multiwoven/Chart.yaml b/charts/multiwoven/Chart.yaml
@@ -2,8 +2,8 @@ apiVersion: v2
 name: multiwoven
 description: Open-source reverse ETL, an alternative to Hightouch, Census etc. 🔥
 type: application
-version: 0.4.0
-appVersion: "0.4.0"
+version: 0.5.0
+appVersion: "0.5.0"
 maintainers:
   - name: subintp
   - name: RafaelOAiSquared

diff --git a/charts/multiwoven/templates/multiwoven-worker-deployment.yaml b/charts/multiwoven/templates/multiwoven-worker-deployment.yaml
@@ -3,7 +3,6 @@ kind: Deployment
 metadata:
   name: {{ include "chart.fullname" . }}-worker
   namespace: {{ .Values.kubernetesNamespace }}
-
   labels:
     app: {{ include "chart.fullname" . }}-worker
     io.kompose.service: {{ include "chart.fullname" . }}-worker
@@ -27,10 +26,26 @@ spec:
         env:
         - name: KUBERNETES_CLUSTER_DOMAIN
           value: {{ quote .Values.kubernetesClusterDomain }}
+        - name: MULTIWOVEN_WORKER_HEALTH_CHECK_PORT
+          value: {{ quote .Values.multiwovenWorker.healthPort }}
         envFrom:
         - configMapRef:
             name: {{ include "chart.fullname" . }}-config
         image: {{ .Values.multiwovenWorker.multiwovenWorker.image.repository }}:{{ .Values.multiwovenWorker.multiwovenWorker.image.tag | default .Chart.AppVersion }}
+        livenessProbe:
+          httpGet:
+            path: /health
+            port: {{ .Values.multiwovenWorker.healthPort }}
+          initialDelaySeconds: 15
+          periodSeconds: 10
         name: {{ include "chart.fullname" . }}-worker
+        ports:
+        - containerPort: {{ .Values.multiwovenWorker.healthPort }}
+        readinessProbe:
+          httpGet:
+            path: /health
+            port: {{ .Values.multiwovenWorker.healthPort }}
+          initialDelaySeconds: 5
+          periodSeconds: 10
         resources: {{- toYaml .Values.multiwovenWorker.multiwovenWorker.resources | nindent 10 }}
       restartPolicy: Always
diff --git a/charts/multiwoven/values.yaml b/charts/multiwoven/values.yaml
@@ -95,6 +95,7 @@ multiwovenUI:
   replicas: 1
   type: ClusterIP
 multiwovenWorker:
+  healthPort: 4567
   multiwovenWorker:
     args:
       - sh