fix(vi,vd): fix review comments

Signed-off-by: Roman Sysoev <roman.sysoev@flant.com>

Author: Roman Sysoev

api/core/v1alpha2/vicondition/condition.go

@@ -79,6 +79,8 @@ const (
 	ProvisioningNotStarted ReadyReason = "ProvisioningNotStarted"
 	// ProvisioningFailed indicates that the provisioning process has failed.
 	ProvisioningFailed ReadyReason = "ProvisioningFailed"
+	// StorageClassNotReady indicates that the provisioning process pending because `StorageClass` not ready.
+	StorageClassNotReady ReadyReason = "StorageClassNotReady"
 	// Ready indicates that the import process is complete and the `VirtualImage` is ready for use.
 	Ready ReadyReason = "Ready"
 	// QuotaExceeded indicates that the VirtualImage is reached project quotas and can not be provisioned.
@@ -91,8 +93,6 @@ const (
 
 	// StorageClassReady indicates that the chosen StorageClass exists.
 	StorageClassReady StorageClassReadyReason = "StorageClassReady"
-	// StorageClassNotReady indicates that the provisioning process pending because `StorageClass` not ready.
-	StorageClassNotReady ReadyReason = "StorageClassNotReady"
 	// StorageClassNotFound indicates that the chosen StorageClass not found.
 	StorageClassNotFound StorageClassReadyReason = "StorageClassNotFound"
 	// DVCRTypeUsed indicates that the DVCR provisioning chosen.

images/virtualization-artifact/pkg/controller/service/vi_storage_class_service.go

@@ -39,7 +39,7 @@ func NewVirtualImageStorageClassService(settings config.VirtualImageStorageClass
 	}
 }
 
-// GetStorageClass determines the storage class for VI from global settings and resource spec.
+// GetValidatedStorageClass determines the storage class for VI from global settings and resource spec.
 //
 // Global settings contain a default storage class and an array of allowed storageClasses from the ModuleConfig.
 // Storage class is allowed if contained in the "allowed" array.