remove spec.operatorVersion field (#13)

Signed-off-by: Predrag Knezevic <[email protected]>

api/v1/qdrantcluster_types.go

@@ -14,15 +14,6 @@ const (
 	ResourceQdrantCluster = "qdrantclusters"
 )
 
-type OperatorVersion string
-
-const (
-	// The operator written in Python
-	OperatorV1 OperatorVersion = "V1"
-	// The operator written in Go
-	OperatorV2 OperatorVersion = "V2"
-)
-
 const (
 	// The Annotation key to trigger a restart.
 	RestartedAtAnnotationKey = "restartedAt"
@@ -54,7 +45,6 @@ func GetQdrantClusterCrdForHash(qc QdrantCluster) QdrantCluster {
 	cloned.ClusterManager = nil
 	cloned.Distributed = false
 	cloned.Ingress = nil
-	cloned.OperatorVersion = nil
 	cloned.Pauses = nil
 	cloned.Resources.Storage = ""
 	cloned.Service = nil
@@ -74,11 +64,6 @@ func GetQdrantClusterCrdForHash(qc QdrantCluster) QdrantCluster {
 // QdrantClusterSpec defines the desired state of QdrantCluster
 // +kubebuilder:pruning:PreserveUnknownFields
 type QdrantClusterSpec struct {
-	// The version of the operator which reconciles this instance
-	// +kubebuilder:validation:Enum=V1;V2
-	// +kubebuilder:default=V1
-	// +optional
-	OperatorVersion *OperatorVersion `json:"operatorVersion,omitempty"`
 	// Id specifies the unique identifier of the cluster
 	Id string `json:"id"`
 	// Version specifies the version of Qdrant to deploy
@@ -146,13 +131,6 @@ type QdrantClusterSpec struct {
 	PodDisruptionBudget *policyv1.PodDisruptionBudgetSpec `json:"podDisruptionBudget,omitempty"`
 }
 
-func (s QdrantClusterSpec) GetOperatorVersion() OperatorVersion {
-	if s.OperatorVersion == nil {
-		return OperatorV1
-	}
-	return *s.OperatorVersion
-}
-
 // Validates if there are incorrect settings in the CRD
 func (s QdrantClusterSpec) Validate() error {
 	if err := s.Resources.Validate("Spec.Resources"); err != nil {

api/v1/qdrantclusterrestore_types.go

@@ -9,24 +9,12 @@ const (
 
 // QdrantClusterRestoreSpec defines the desired state of QdrantClusterRestore
 type QdrantClusterRestoreSpec struct {
-	// The version of the operator which reconciles this instance
-	// +kubebuilder:validation:Enum=V1;V2
-	// +kubebuilder:default=V1
-	// +optional
-	OperatorVersion *OperatorVersion `json:"operatorVersion,omitempty"`
 	// Source defines the source snapshot from which the restore will be done
 	Source RestoreSource `json:"source"`
 	// Destination defines the destination cluster where the source data will end up
 	Destination RestoreDestination `json:"destination"`
 }
 
-func (s QdrantClusterRestoreSpec) GetOperatorVersion() OperatorVersion {
-	if s.OperatorVersion == nil {
-		return OperatorV1
-	}
-	return *s.OperatorVersion
-}
-
 type RestoreSource struct {
 	// SnapshotName is the name of the snapshot from which we wish to restore
 	SnapshotName string `json:"snapshotName"`

api/v1/qdrantclusterscheduledsnapshot_types.go

@@ -9,11 +9,6 @@ const (
 
 // QdrantClusterScheduledSnapshotSpec defines the desired state of QdrantCluster
 type QdrantClusterScheduledSnapshotSpec struct {
-	// The version of the operator which reconciles this instance
-	// +kubebuilder:validation:Enum=V1;V2
-	// +kubebuilder:default=V1
-	// +optional
-	OperatorVersion *OperatorVersion `json:"operatorVersion,omitempty"`
 	// Id specifies the unique identifier of the cluster
 	ClusterId string `json:"cluster-id"`
 	// Specifies short Id which identifies a schedule
@@ -28,13 +23,6 @@ type QdrantClusterScheduledSnapshotSpec struct {
 	Retention string `json:"retention"`
 }
 
-func (s QdrantClusterScheduledSnapshotSpec) GetOperatorVersion() OperatorVersion {
-	if s.OperatorVersion == nil {
-		return OperatorV1
-	}
-	return *s.OperatorVersion
-}
-
 type ScheduledSnapshotPhase string
 
 const (

api/v1/qdrantclustersnapshot_types.go

@@ -8,11 +8,6 @@ const (
 )
 
 type QdrantClusterSnapshotSpec struct {
-	// The version of the operator which reconciles this instance
-	// +kubebuilder:validation:Enum=V1;V2
-	// +kubebuilder:default=V1
-	// +optional
-	OperatorVersion *OperatorVersion `json:"operatorVersion,omitempty"`
 	// The cluster ID for which a Snapshot need to be taken
 	// The cluster should be in the same namespace as this QdrantClusterSnapshot is located
 	ClusterId string `json:"cluster-id"`
@@ -31,13 +26,6 @@ type QdrantClusterSnapshotSpec struct {
 	Retention *string `json:"retention,omitempty"`
 }
 
-func (s QdrantClusterSnapshotSpec) GetOperatorVersion() OperatorVersion {
-	if s.OperatorVersion == nil {
-		return OperatorV1
-	}
-	return *s.OperatorVersion
-}
-
 type QdrantClusterSnapshotPhase string
 
 const (

charts/qdrant-operator-crds/crds/qdrant.io_qdrantclusterrestores.yaml

@@ -64,13 +64,6 @@ spec:
                 - name
                 - namespace
                 type: object
-              operatorVersion:
-                default: V1
-                description: The version of the operator which reconciles this instance
-                enum:
-                - V1
-                - V2
-                type: string
               source:
                 description: Source defines the source snapshot from which the restore
                   will be done

charts/qdrant-operator-crds/crds/qdrant.io_qdrantclusters.yaml

@@ -324,13 +324,6 @@ spec:
                 description: NodeSelector specifies the node selector for each Qdrant
                   node.
                 type: object
-              operatorVersion:
-                default: V1
-                description: The version of the operator which reconciles this instance
-                enum:
-                - V1
-                - V2
-                type: string
               pauses:
                 description: |-
                   Pauses specifies a list of pause request by developer for manual maintenance.

charts/qdrant-operator-crds/crds/qdrant.io_qdrantclusterscheduledsnapshots.yaml

@@ -66,13 +66,6 @@ spec:
               cluster-id:
                 description: Id specifies the unique identifier of the cluster
                 type: string
-              operatorVersion:
-                default: V1
-                description: The version of the operator which reconciles this instance
-                enum:
-                - V1
-                - V2
-                type: string
               retention:
                 description: Retention of schedule in hours
                 pattern: ^[0-9]+h$

charts/qdrant-operator-crds/crds/qdrant.io_qdrantclustersnapshots.yaml

@@ -65,13 +65,6 @@ spec:
                   epoch format)
                 format: int64
                 type: integer
-              operatorVersion:
-                default: V1
-                description: The version of the operator which reconciles this instance
-                enum:
-                - V1
-                - V2
-                type: string
               retention:
                 description: |-
                   The retention period of this snapshot in hours, if any.

docs/api.md

@@ -299,22 +299,6 @@ _Appears in:_
 
 
 
-#### OperatorVersion
-
-_Underlying type:_ _string_
-
-
-
-
-
-_Appears in:_
-- [QdrantClusterRestoreSpec](#qdrantclusterrestorespec)
-- [QdrantClusterScheduledSnapshotSpec](#qdrantclusterscheduledsnapshotspec)
-- [QdrantClusterSnapshotSpec](#qdrantclustersnapshotspec)
-- [QdrantClusterSpec](#qdrantclusterspec)
-
-
-
 #### Pause
 
 
@@ -477,7 +461,6 @@ _Appears in:_
 
 | Field | Description | Default | Validation |
 | --- | --- | --- | --- |
-| `operatorVersion` _[OperatorVersion](#operatorversion)_ | The version of the operator which reconciles this instance | V1 | Enum: [V1 V2] <br /> |
 | `source` _[RestoreSource](#restoresource)_ | Source defines the source snapshot from which the restore will be done |  |  |
 | `destination` _[RestoreDestination](#restoredestination)_ | Destination defines the destination cluster where the source data will end up |  |  |
 
@@ -534,7 +517,6 @@ _Appears in:_
 
 | Field | Description | Default | Validation |
 | --- | --- | --- | --- |
-| `operatorVersion` _[OperatorVersion](#operatorversion)_ | The version of the operator which reconciles this instance | V1 | Enum: [V1 V2] <br /> |
 | `cluster-id` _string_ | Id specifies the unique identifier of the cluster |  |  |
 | `scheduleShortId` _string_ | Specifies short Id which identifies a schedule |  | MaxLength: 8 <br /> |
 | `schedule` _string_ | Cron expression for frequency of creating snapshots, see https://en.wikipedia.org/wiki/Cron.<br />The schedule is specified in UTC. |  | Pattern: `^(@(annually|yearly|monthly|weekly|daily|hourly|reboot))|(@every (\d+(ns|us|µs|ms|s|m|h))+)|((((\d+,)+\d+|([\d\*]+(\/|-)\d+)|\d+|\*) ?){5,7})$` <br /> |
@@ -606,7 +588,6 @@ _Appears in:_
 
 | Field | Description | Default | Validation |
 | --- | --- | --- | --- |
-| `operatorVersion` _[OperatorVersion](#operatorversion)_ | The version of the operator which reconciles this instance | V1 | Enum: [V1 V2] <br /> |
 | `cluster-id` _string_ | The cluster ID for which a Snapshot need to be taken<br />The cluster should be in the same namespace as this QdrantClusterSnapshot is located |  |  |
 | `creation-timestamp` _integer_ | The CreationTimestamp of the backup (expressed in Unix epoch format) |  |  |
 | `scheduleShortId` _string_ | Specifies the short Id which identifies a schedule, if any.<br />This field should not be set if the backup is made manually. |  | MaxLength: 8 <br /> |
@@ -628,7 +609,6 @@ _Appears in:_
 
 | Field | Description | Default | Validation |
 | --- | --- | --- | --- |
-| `operatorVersion` _[OperatorVersion](#operatorversion)_ | The version of the operator which reconciles this instance | V1 | Enum: [V1 V2] <br /> |
 | `id` _string_ | Id specifies the unique identifier of the cluster |  |  |
 | `version` _string_ | Version specifies the version of Qdrant to deploy |  |  |
 | `size` _integer_ | Size specifies the desired number of Qdrant nodes in the cluster |  | Maximum: 30 <br />Minimum: 1 <br /> |