From 256f13660922c2d603cdd30dd076d4494c8a771d Mon Sep 17 00:00:00 2001 From: Nadav Gov-Ari Date: Tue, 11 Aug 2026 15:43:04 -0400 Subject: [PATCH] Remove readiness probe in favor of startup probe --- charts/quickwit/Chart.yaml | 2 +- .../templates/_metastore-deployment.tpl | 4 +- .../templates/compactor-deployment.yaml | 4 +- .../templates/control-plane-deployment.yaml | 4 +- .../templates/indexer-statefulset.yaml | 4 +- .../templates/janitor-deployment.yaml | 4 +- .../templates/searcher-statefulset.yaml | 4 +- charts/quickwit/values.yaml | 56 +++++-------------- 8 files changed, 33 insertions(+), 49 deletions(-) diff --git a/charts/quickwit/Chart.yaml b/charts/quickwit/Chart.yaml index 67c46c1..dcb60e8 100644 --- a/charts/quickwit/Chart.yaml +++ b/charts/quickwit/Chart.yaml @@ -2,7 +2,7 @@ apiVersion: v2 name: quickwit description: Sub-second search & analytics engine on cloud storage. type: application -version: 0.8.13 +version: 0.8.14 appVersion: v0.8.2 keywords: - quickwit diff --git a/charts/quickwit/templates/_metastore-deployment.tpl b/charts/quickwit/templates/_metastore-deployment.tpl index 8a4f45b..42a320f 100644 --- a/charts/quickwit/templates/_metastore-deployment.tpl +++ b/charts/quickwit/templates/_metastore-deployment.tpl @@ -89,8 +89,10 @@ spec: {{- toYaml $values.startupProbe | nindent 12 }} livenessProbe: {{- toYaml $values.livenessProbe | nindent 12 }} + {{- with $values.readinessProbe }} readinessProbe: - {{- toYaml $values.readinessProbe | nindent 12 }} + {{- toYaml . | nindent 12 }} + {{- end }} volumeMounts: - name: config mountPath: /quickwit/node.yaml diff --git a/charts/quickwit/templates/compactor-deployment.yaml b/charts/quickwit/templates/compactor-deployment.yaml index 6a54c48..8b560b6 100644 --- a/charts/quickwit/templates/compactor-deployment.yaml +++ b/charts/quickwit/templates/compactor-deployment.yaml @@ -79,8 +79,10 @@ spec: {{- toYaml .Values.compactor.startupProbe | nindent 12 }} livenessProbe: {{- toYaml .Values.compactor.livenessProbe | nindent 12 }} + {{- with .Values.compactor.readinessProbe }} readinessProbe: - {{- toYaml .Values.compactor.readinessProbe | nindent 12 }} + {{- toYaml . | nindent 12 }} + {{- end }} volumeMounts: - name: config mountPath: /quickwit/node.yaml diff --git a/charts/quickwit/templates/control-plane-deployment.yaml b/charts/quickwit/templates/control-plane-deployment.yaml index 23edd49..91dab73 100644 --- a/charts/quickwit/templates/control-plane-deployment.yaml +++ b/charts/quickwit/templates/control-plane-deployment.yaml @@ -75,8 +75,10 @@ spec: {{- toYaml .Values.control_plane.startupProbe | nindent 12 }} livenessProbe: {{- toYaml .Values.control_plane.livenessProbe | nindent 12 }} + {{- with .Values.control_plane.readinessProbe }} readinessProbe: - {{- toYaml .Values.control_plane.readinessProbe | nindent 12 }} + {{- toYaml . | nindent 12 }} + {{- end }} volumeMounts: - name: config mountPath: /quickwit/node.yaml diff --git a/charts/quickwit/templates/indexer-statefulset.yaml b/charts/quickwit/templates/indexer-statefulset.yaml index 0c5e42c..59e05a7 100644 --- a/charts/quickwit/templates/indexer-statefulset.yaml +++ b/charts/quickwit/templates/indexer-statefulset.yaml @@ -84,8 +84,10 @@ spec: {{- toYaml .Values.indexer.startupProbe | nindent 12 }} livenessProbe: {{- toYaml .Values.indexer.livenessProbe | nindent 12 }} + {{- with .Values.indexer.readinessProbe }} readinessProbe: - {{- toYaml .Values.indexer.readinessProbe | nindent 12 }} + {{- toYaml . | nindent 12 }} + {{- end }} volumeMounts: - name: config mountPath: /quickwit/node.yaml diff --git a/charts/quickwit/templates/janitor-deployment.yaml b/charts/quickwit/templates/janitor-deployment.yaml index f125f62..2d489fc 100644 --- a/charts/quickwit/templates/janitor-deployment.yaml +++ b/charts/quickwit/templates/janitor-deployment.yaml @@ -75,8 +75,10 @@ spec: {{- toYaml .Values.janitor.startupProbe | nindent 12 }} livenessProbe: {{- toYaml .Values.janitor.livenessProbe | nindent 12 }} + {{- with .Values.janitor.readinessProbe }} readinessProbe: - {{- toYaml .Values.janitor.readinessProbe | nindent 12 }} + {{- toYaml . | nindent 12 }} + {{- end }} volumeMounts: - name: config mountPath: /quickwit/node.yaml diff --git a/charts/quickwit/templates/searcher-statefulset.yaml b/charts/quickwit/templates/searcher-statefulset.yaml index 7fdc821..93d37c9 100644 --- a/charts/quickwit/templates/searcher-statefulset.yaml +++ b/charts/quickwit/templates/searcher-statefulset.yaml @@ -81,8 +81,10 @@ spec: {{- toYaml .Values.searcher.startupProbe | nindent 12 }} livenessProbe: {{- toYaml .Values.searcher.livenessProbe | nindent 12 }} + {{- with .Values.searcher.readinessProbe }} readinessProbe: - {{- toYaml .Values.searcher.readinessProbe | nindent 12 }} + {{- toYaml . | nindent 12 }} + {{- end }} volumeMounts: - name: config mountPath: /quickwit/node.yaml diff --git a/charts/quickwit/values.yaml b/charts/quickwit/values.yaml index 058311a..4520b2a 100644 --- a/charts/quickwit/values.yaml +++ b/charts/quickwit/values.yaml @@ -147,7 +147,7 @@ searcher: startupProbe: httpGet: - path: /health/livez + path: /health/readyz port: rest failureThreshold: 12 periodSeconds: 5 @@ -156,11 +156,7 @@ searcher: httpGet: path: /health/livez port: rest - - readinessProbe: - httpGet: - path: /health/readyz - port: rest + timeoutSeconds: 5 # StatefulSet allows you to relax its ordering guarantees # - OrderedReady @@ -257,7 +253,7 @@ indexer: startupProbe: httpGet: - path: /health/livez + path: /health/readyz port: rest failureThreshold: 12 periodSeconds: 5 @@ -266,11 +262,7 @@ indexer: httpGet: path: /health/livez port: rest - - readinessProbe: - httpGet: - path: /health/readyz - port: rest + timeoutSeconds: 5 # StatefulSet allows you to relax its ordering guarantees # - OrderedReady @@ -370,7 +362,7 @@ metastore: startupProbe: httpGet: - path: /health/livez + path: /health/readyz port: rest failureThreshold: 12 periodSeconds: 5 @@ -379,11 +371,7 @@ metastore: httpGet: path: /health/livez port: rest - - readinessProbe: - httpGet: - path: /health/readyz - port: rest + timeoutSeconds: 5 # Override args for starting container args: [] @@ -453,7 +441,7 @@ metastore_ro: startupProbe: httpGet: - path: /health/livez + path: /health/readyz port: rest failureThreshold: 12 periodSeconds: 5 @@ -462,11 +450,7 @@ metastore_ro: httpGet: path: /health/livez port: rest - - readinessProbe: - httpGet: - path: /health/readyz - port: rest + timeoutSeconds: 5 # Override args for starting the container. args: [] @@ -529,7 +513,7 @@ control_plane: startupProbe: httpGet: - path: /health/livez + path: /health/readyz port: rest failureThreshold: 12 periodSeconds: 5 @@ -538,11 +522,7 @@ control_plane: httpGet: path: /health/livez port: rest - - readinessProbe: - httpGet: - path: /health/readyz - port: rest + timeoutSeconds: 5 # Override args for starting container args: [] @@ -601,7 +581,7 @@ janitor: startupProbe: httpGet: - path: /health/livez + path: /health/readyz port: rest failureThreshold: 12 periodSeconds: 5 @@ -610,11 +590,7 @@ janitor: httpGet: path: /health/livez port: rest - - readinessProbe: - httpGet: - path: /health/readyz - port: rest + timeoutSeconds: 5 # Override args for starting container args: [] @@ -714,7 +690,7 @@ compactor: startupProbe: httpGet: - path: /health/livez + path: /health/readyz port: rest failureThreshold: 12 periodSeconds: 5 @@ -723,11 +699,7 @@ compactor: httpGet: path: /health/livez port: rest - - readinessProbe: - httpGet: - path: /health/readyz - port: rest + timeoutSeconds: 5 # Override args for starting container args: []