replication: add pvcRef to volumegroupreplication

adding the pvcRef to the volumegroupreplication to give a representation of the pvc's which are part of the volumegroupreplication. Signed-off-by: Madhu Rajanna <[email protected]>
csi-addons · Jun 24, 2024 · e3878f5 · e3878f5
1 parent d8727ec
commit e3878f5
Show file tree

Hide file tree

Showing 4 changed files with 56 additions and 0 deletions.
diff --git a/apis/replication.storage/v1alpha1/volumegroupreplication_types.go b/apis/replication.storage/v1alpha1/volumegroupreplication_types.go
@@ -17,6 +17,7 @@ limitations under the License.
 package v1alpha1
 
 import (
+	corev1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
@@ -72,6 +73,10 @@ type VolumeGroupReplicationSource struct {
 // VolumeGroupReplicationStatus defines the observed state of VolumeGroupReplication
 type VolumeGroupReplicationStatus struct {
 	VolumeReplicationStatus `json:",inline"`
+	// PersistentVolumeClaimsRefList is the list of PVCs for the volume group replication.
+	// The maximum number of allowed PVCs in the group is 100.
+	// +optional
+	PersistentVolumeClaimsRefList []corev1.LocalObjectReference `json:"persistentVolumeClaimsRefList,omitempty"`
 }
 
 //+kubebuilder:object:root=true

diff --git a/apis/replication.storage/v1alpha1/zz_generated.deepcopy.go b/apis/replication.storage/v1alpha1/zz_generated.deepcopy.go
diff --git a/config/crd/bases/replication.storage.openshift.io_volumegroupreplications.yaml b/config/crd/bases/replication.storage.openshift.io_volumegroupreplications.yaml
@@ -246,6 +246,29 @@ spec:
                   operator has dealt with
                 format: int64
                 type: integer
+              persistentVolumeClaimsRefList:
+                description: |-
+                  PersistentVolumeClaimsRefList is the list of PVCs for the volume group replication.
+                  The maximum number of allowed PVCs in the group is 100.
+                items:
+                  description: |-
+                    LocalObjectReference contains enough information to let you locate the
+                    referenced object inside the same namespace.
+                  properties:
+                    name:
+                      default: ""
+                      description: |-
+                        Name of the referent.
+                        This field is effectively required, but due to backwards compatibility is
+                        allowed to be empty. Instances of this type with an empty value here are
+                        almost certainly wrong.
+                        TODO: Add other useful fields. apiVersion, kind, uid?
+                        More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+                        TODO: Drop `kubebuilder:default` when controller-gen doesn't need it https://github.com/kubernetes-sigs/kubebuilder/issues/3896.
+                      type: string
+                  type: object
+                  x-kubernetes-map-type: atomic
+                type: array
               state:
                 description: State captures the latest state of the replication operation.
                 type: string

diff --git a/deploy/controller/crds.yaml b/deploy/controller/crds.yaml
@@ -1269,6 +1269,29 @@ spec:
                   operator has dealt with
                 format: int64
                 type: integer
+              persistentVolumeClaimsRefList:
+                description: |-
+                  PersistentVolumeClaimsRefList is the list of PVCs for the volume group replication.
+                  The maximum number of allowed PVCs in the group is 100.
+                items:
+                  description: |-
+                    LocalObjectReference contains enough information to let you locate the
+                    referenced object inside the same namespace.
+                  properties:
+                    name:
+                      default: ""
+                      description: |-
+                        Name of the referent.
+                        This field is effectively required, but due to backwards compatibility is
+                        allowed to be empty. Instances of this type with an empty value here are
+                        almost certainly wrong.
+                        TODO: Add other useful fields. apiVersion, kind, uid?
+                        More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+                        TODO: Drop `kubebuilder:default` when controller-gen doesn't need it https://github.com/kubernetes-sigs/kubebuilder/issues/3896.
+                      type: string
+                  type: object
+                  x-kubernetes-map-type: atomic
+                type: array
               state:
                 description: State captures the latest state of the replication operation.
                 type: string