From 1d255553909e07ae27bd326cd48964fff4f7b07d Mon Sep 17 00:00:00 2001 From: Karl Egly Date: Wed, 1 Jul 2026 05:59:27 +0000 Subject: [PATCH] feat: add startupProbe to Helm Chart --- helm/opensoho/Chart.yaml | 4 ++-- helm/opensoho/README.md | 1 + helm/opensoho/templates/deployment.yaml | 12 ++++++++++-- helm/opensoho/values.yaml | 19 +++++++++++++++---- 4 files changed, 28 insertions(+), 8 deletions(-) diff --git a/helm/opensoho/Chart.yaml b/helm/opensoho/Chart.yaml index eca317e..6442add 100644 --- a/helm/opensoho/Chart.yaml +++ b/helm/opensoho/Chart.yaml @@ -2,8 +2,8 @@ apiVersion: v2 name: opensoho description: A Helm chart for OpenSoho - OpenWrt management platform type: application -version: 0.2.0 -appVersion: "v0.7.3" +version: 0.2.1 +appVersion: "v0.13.0" keywords: - opensoho - openwrt diff --git a/helm/opensoho/README.md b/helm/opensoho/README.md index e2da374..953e2a3 100644 --- a/helm/opensoho/README.md +++ b/helm/opensoho/README.md @@ -110,6 +110,7 @@ The command removes all the Kubernetes components associated with the chart and | --------------------------- | -------------------------------------------------------------------------------------------------------------------------------- | ------- | | `healthCheck.enabled` | Enable readiness probe | `true` | | `livenessProbe.enabled` | Enable liveness probe | `true` | +| `startupProbe.enabled` | Enable startup probe | `true` | ## Configuration and installation details diff --git a/helm/opensoho/templates/deployment.yaml b/helm/opensoho/templates/deployment.yaml index 2d561e1..75263af 100644 --- a/helm/opensoho/templates/deployment.yaml +++ b/helm/opensoho/templates/deployment.yaml @@ -50,7 +50,6 @@ spec: httpGet: path: /api/health port: http - initialDelaySeconds: {{ .Values.healthCheck.initialDelaySeconds }} periodSeconds: {{ .Values.healthCheck.periodSeconds }} timeoutSeconds: {{ .Values.healthCheck.timeoutSeconds }} failureThreshold: {{ .Values.healthCheck.failureThreshold }} @@ -61,12 +60,21 @@ spec: httpGet: path: /api/health port: http - initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds }} periodSeconds: {{ .Values.livenessProbe.periodSeconds }} timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds }} failureThreshold: {{ .Values.livenessProbe.failureThreshold }} successThreshold: {{ .Values.livenessProbe.successThreshold }} {{- end }} + {{- if .Values.startupProbe.enabled }} + startupProbe: + httpGet: + path: /api/health + port: http + periodSeconds: {{ .Values.startupProbe.periodSeconds }} + timeoutSeconds: {{ .Values.startupProbe.timeoutSeconds }} + failureThreshold: {{ .Values.startupProbe.failureThreshold }} + successThreshold: {{ .Values.startupProbe.successThreshold }} + {{- end }} resources: {{- toYaml .Values.resources | nindent 12 }} volumeMounts: diff --git a/helm/opensoho/values.yaml b/helm/opensoho/values.yaml index 3179a4d..ea91e39 100644 --- a/helm/opensoho/values.yaml +++ b/helm/opensoho/values.yaml @@ -57,6 +57,11 @@ ingress: # hosts: # - opensoho.local +httproute: + enabled: false + hostname: opensoho.local + parentRef: + resources: limits: cpu: 500m @@ -92,21 +97,27 @@ persistence: # Mount path for persistent data mountPath: /ko-app/pb_data -# Health checks +# readinessProbe, should traffic go to this replica? healthCheck: enabled: true - initialDelaySeconds: 30 periodSeconds: 10 timeoutSeconds: 5 failureThreshold: 3 successThreshold: 1 -# Liveness probe +# Liveness probe, should we kill this replica? livenessProbe: enabled: true - initialDelaySeconds: 30 periodSeconds: 10 timeoutSeconds: 5 failureThreshold: 3 successThreshold: 1 +# Startup probe, is this replica ready for service? +startupProbe: + enabled: true + periodSeconds: 10 + timeoutSeconds: 5 + # 30 * 10 => 5 minutes to start up, before giving up + failureThreshold: 30 + successThreshold: 1