Use / for probes instead of /readyz (#114)

* Use / for probes instead of /readyz

The new implementation of /readyz does not work well together with how StatefulSets do roll outs and restarts

* Fix test

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## [qdrant-0.7.2](https://github.com/qdrant/qdrant-helm/tree/qdrant-0.7.2) (2023-12-12)
+
+- Use / for probes instead of /readyz endpoint
+
 ## [qdrant-0.7.1](https://github.com/qdrant/qdrant-helm/tree/qdrant-0.7.1) (2023-12-12)
 
 - Update Qdrant to v1.7.1

charts/qdrant/CHANGELOG.md

@@ -1,5 +1,5 @@
 # Changelog
 
-## [qdrant-0.7.1](https://github.com/qdrant/qdrant-helm/tree/qdrant-0.7.1) (2023-12-12)
+## [qdrant-0.7.2](https://github.com/qdrant/qdrant-helm/tree/qdrant-0.7.2) (2023-12-12)
 
-- Update Qdrant to v1.7.1
+- Use / for probes instead of /readyz endpoint

charts/qdrant/Chart.yaml

@@ -13,10 +13,10 @@ maintainers:
     url: https://github.com/qdrant
 icon: https://qdrant.github.io/qdrant-helm/logo_with_text.svg
 type: application
-version: 0.7.1
+version: 0.7.2
 appVersion: v1.7.1
 annotations:
   artifacthub.io/category: database
   artifacthub.io/changes: |
-    - kind: added
-      description: Update Qdrant to v1.7.1
+    - kind: fixed
+      description: Use / for probes instead of /readyz endpoint

charts/qdrant/templates/statefulset.yaml

@@ -98,7 +98,7 @@ spec:
             {{- end }}
             {{- if eq .name "http"}}
             httpGet:
-              path: /livez
+              path: /
               port: {{ .targetPort }}
               {{- if and $values.config.service $values.config.service.enable_tls }}
               scheme: HTTPS
@@ -118,7 +118,7 @@ spec:
             {{- end }}
             {{- if eq .name "http"}}
             httpGet:
-              path: /readyz
+              path: /
               port: {{ .targetPort }}
               {{- if and $values.config.service $values.config.service.enable_tls }}
               scheme: HTTPS
@@ -138,7 +138,7 @@ spec:
             {{- end }}
             {{- if eq .name "http"}}
             httpGet:
-              path: /readyz
+              path: /
               port: {{ .targetPort }}
               {{- if and $values.config.service $values.config.service.enable_tls }}
               scheme: HTTPS

test/qdrant_probes_test.go

@@ -39,6 +39,6 @@ func TestDefaultProbesOnStatefulset(t *testing.T) {
 	})
 
 	require.Empty(t, container.StartupProbe)
-	require.Equal(t, "/readyz", container.ReadinessProbe.HTTPGet.Path)
+	require.Equal(t, "/", container.ReadinessProbe.HTTPGet.Path)
 	require.Empty(t, container.LivenessProbe)
 }