Daniel Erez has uploaded a new change for review. Change subject: core: exclude Cinder from storage space validations ......................................................................
core: exclude Cinder from storage space validations Exclude space validations for Cinder storage domains in StorageDomainValidator. Change-Id: I626fd0758cd1d026f35e8fc1ba5c02c26f28aae0 Bug-Url: https://bugzilla.redhat.com/1185826 Signed-off-by: Daniel Erez <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageDomainValidator.java 1 file changed, 12 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/00/39200/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageDomainValidator.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageDomainValidator.java index 26bc791..24dc9df 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageDomainValidator.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageDomainValidator.java @@ -174,6 +174,9 @@ } public ValidationResult hasSpaceForNewDisks(Collection<DiskImage> diskImages) { + if (storageDomain.getStorageType() == StorageType.CINDER) { + return ValidationResult.VALID; + } Long availableSize = storageDomain.getAvailableDiskSizeInBytes(); double totalSizeForDisks = getTotalSizeForNewDisks(diskImages); @@ -181,6 +184,9 @@ } public ValidationResult hasSpaceForClonedDisks(Collection<DiskImage> diskImages) { + if (storageDomain.getStorageType() == StorageType.CINDER) { + return ValidationResult.VALID; + } Long availableSize = storageDomain.getAvailableDiskSizeInBytes(); double totalSizeForDisks = getTotalSizeForClonedDisks(diskImages); @@ -188,6 +194,9 @@ } public ValidationResult hasSpaceForDisksWithSnapshots(Collection<DiskImage> diskImages) { + if (storageDomain.getStorageType() == StorageType.CINDER) { + return ValidationResult.VALID; + } Long availableSize = storageDomain.getAvailableDiskSizeInBytes(); double totalSizeForDisks = getTotalSizeForDisksWithSnapshots(diskImages); @@ -195,6 +204,9 @@ } public ValidationResult hasSpaceForAllDisks(Collection<DiskImage> newDiskImages, Collection<DiskImage> clonedDiskImages) { + if (storageDomain.getStorageType() == StorageType.CINDER) { + return ValidationResult.VALID; + } Long availableSize = storageDomain.getAvailableDiskSizeInBytes(); double totalSizeForNewDisks = getTotalSizeForNewDisks(newDiskImages); double totalSizeForClonedDisks = getTotalSizeForClonedDisks(clonedDiskImages); -- To view, visit https://gerrit.ovirt.org/39200 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I626fd0758cd1d026f35e8fc1ba5c02c26f28aae0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
