Skip creation of Virtualization SC in Provider Mode

- StorageClasses should only be created by StorageClassRequests when
StorageConsumers are allowed, i.e in Provider mode
- This commit skips virt sc creation if consumers are allowed in
StorageCluster CR

Signed-off-by: Leela Venkaiah G <[email protected]>

controllers/storagecluster/storageclasses.go

@@ -421,12 +421,17 @@ func (r *StorageClusterReconciler) newStorageClassConfigurations(initData *ocsv1
 		newCephFilesystemStorageClassConfiguration(initData),
 		newCephBlockPoolStorageClassConfiguration(initData),
 	}
-	// If kubevirt crd is present, we create a specialized rbd storageclass for virtualization environment
-	kvcrd := &extv1.CustomResourceDefinition{}
-	err := r.Client.Get(context.TODO(), types.NamespacedName{Name: "virtualmachines.kubevirt.io", Namespace: ""}, kvcrd)
-	if err == nil {
-		ret = append(ret, newCephBlockPoolVirtualizationStorageClassConfiguration(initData))
+
+	// when allowing consumers, creation of storage classes should only be done via storageclassrequests
+	if !initData.Spec.AllowRemoteStorageConsumers {
+		// If kubevirt crd is present, we create a specialized rbd storageclass for virtualization environment
+		kvcrd := &extv1.CustomResourceDefinition{}
+		err := r.Client.Get(context.TODO(), types.NamespacedName{Name: "virtualmachines.kubevirt.io", Namespace: ""}, kvcrd)
+		if err == nil {
+			ret = append(ret, newCephBlockPoolVirtualizationStorageClassConfiguration(initData))
+		}
 	}
+
 	if initData.Spec.ManagedResources.CephNonResilientPools.Enable {
 		ret = append(ret, newNonResilientCephBlockPoolStorageClassConfiguration(initData))
 	}