feat: add liveness and readiness probe for ui

Closes #227
apache · Jan 11, 2024 · 86ab1f7 · 86ab1f7
1 parent ba09914
commit 86ab1f7
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 0 deletions.
diff --git a/HelmSetup.md b/HelmSetup.md
@@ -191,6 +191,8 @@ Some useful parameters for the chart, you could also check them in values.yaml
 | ui.extraLabels                            | extra labels for ui's statefulset                                                     | {}                       |
 | ui.securityContext                        | pod security context values                                                           | {}                       |
 | ui.containerSecurityContext               | container security context values                                                     | {}                       |
+| ui.livenessProbe                          | container livenessprobe                                                               | see Values.yaml          |
+| ui.readinessProbe                         | container readinessProbe                                                              | {}                       |
 | service.type                              | Service type for exposed service                                                      | NodePort                 |
 | service.uiPort                            | Node port for config ui                                                               | 32001                    |
 | service.ingress.enabled                   | If enable ingress                                                                     | false                    |

diff --git a/charts/devlake/templates/deployments.yaml b/charts/devlake/templates/deployments.yaml
@@ -52,6 +52,14 @@ spec:
           imagePullPolicy: {{ .Values.ui.image.pullPolicy }}
           ports:
             - containerPort: 4000
+          {{- with .Values.ui.livenessProbe }}
+          livenessProbe:
+            {{- toYaml . | nindent 12 }}
+          {{- end }}
+          {{- with .Values.ui.readinessProbe }}
+          readinessProbe:
+            {{- toYaml . | nindent 12 }}
+          {{- end }}
           {{- if .Values.ui.basicAuth.enabled }}
           envFrom:
             - secretRef:

diff --git a/charts/devlake/values.yaml b/charts/devlake/values.yaml
@@ -259,6 +259,15 @@ ui:
     # capabilities:
     #   drop:
     #       - all
+  livenessProbe:
+    httpGet:
+      path: /
+      port: 4000
+      scheme: HTTP
+    initialDelaySeconds: 60
+    timeoutSeconds: 30
+
+  readinessProbe: { }
 
 # alpine image for some init containers
 alpine: