Reapply "Added QC.Spec.RestartAllPodsConcurrently for adding one-shot…

…-restart functionality" (#40)

This reverts commit f70991d.

api/v1/qdrantcluster_types.go

@@ -139,6 +139,12 @@ type QdrantClusterSpec struct {
 	// PodDisruptionBudget specifies the pod disruption budget for the cluster.
 	// +optional
 	PodDisruptionBudget *policyv1.PodDisruptionBudgetSpec `json:"podDisruptionBudget,omitempty"`
+	// RestartAllPodsConcurrently specifies whether to restart all pods concurrently (also called one-shot-restart).
+	// If enabled, all the pods in the cluster will be restarted concurrently in situations where multiple pods
+	// need to be restarted like when RestartedAtAnnotationKey is added/updated or the Qdrant version need to be upgraded.
+	// This helps sharded but not replicated clusters to reduce downtime to possible minimum during restart.
+	// +optional
+	RestartAllPodsConcurrently bool `json:"restartAllPodsConcurrently,omitempty"`
 }
 
 // Validates if there are incorrect settings in the CRD

charts/qdrant-kubernetes-api/templates/region-crds/qdrant.io_qdrantclusters.yaml

@@ -492,6 +492,13 @@ spec:
                       node.
                     type: string
                 type: object
+              restartAllPodsConcurrently:
+                description: |-
+                  RestartAllPodsConcurrently specifies whether to restart all pods concurrently (also called one-shot-restart).
+                  If enabled, all the pods in the cluster will be restarted concurrently in situations where multiple pods
+                  need to be restarted like when RestartedAtAnnotationKey is added/updated or the Qdrant version need to be upgraded.
+                  This helps sharded but not replicated clusters to reduce downtime to possible minimum during restart.
+                type: boolean
               security:
                 description: Security specifies the security context for each Qdrant
                   node.

docs/api.md

@@ -628,6 +628,7 @@ _Appears in:_
 | `storageClassNames` _[StorageClassNames](#storageclassnames)_ | StorageClassNames specifies the storage class names for db and snapshots. |  |  |
 | `topologySpreadConstraints` _[TopologySpreadConstraint](https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.28/#topologyspreadconstraint-v1-core)_ | TopologySpreadConstraints specifies the topology spread constraints for the cluster. |  |  |
 | `podDisruptionBudget` _[PodDisruptionBudgetSpec](https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.28/#poddisruptionbudgetspec-v1-policy)_ | PodDisruptionBudget specifies the pod disruption budget for the cluster. |  |  |
+| `restartAllPodsConcurrently` _boolean_ | RestartAllPodsConcurrently specifies whether to restart all pods concurrently (also called one-shot-restart).<br />If enabled, all the pods in the cluster will be restarted concurrently in situations where multiple pods<br />need to be restarted like when RestartedAtAnnotationKey is added/updated or the Qdrant version need to be upgraded.<br />This helps sharded but not replicated clusters to reduce downtime to possible minimum during restart. |  |  |