Maor Lipchuk has uploaded a new change for review. Change subject: core: Introduce private class at Cinder validator. ......................................................................
core: Introduce private class at Cinder validator. Adding a private class CinderStorageRelatedDisksAndProxy which encapsulate the relation between Storage Domain id, list of disks and CinderProxy. Change-Id: I45c4584f336c606e48d51858d967b25cbd125aad Bug-Url: https://bugzilla.redhat.com/1185826 Signed-off-by: Maor Lipchuk <mlipc...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/CinderDisksValidator.java 1 file changed, 38 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/61/41461/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/CinderDisksValidator.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/CinderDisksValidator.java index e99a68f..961394e 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/CinderDisksValidator.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/CinderDisksValidator.java @@ -3,8 +3,10 @@ import com.woorea.openstack.base.client.OpenStackResponseException; import com.woorea.openstack.cinder.model.Limits; +import java.util.ArrayList; import java.util.Collections; import java.util.HashMap; +import java.util.List; import java.util.Map; import java.util.concurrent.Callable; @@ -63,6 +65,42 @@ }); } + private class CinderStorageRelatedDisksAndProxy { + private Guid storageDomainId; + private List<CinderDisk> cinderDisks = new ArrayList<>(); + private OpenStackVolumeProviderProxy proxy; + + public CinderStorageRelatedDisksAndProxy(Guid storageDomainId, List<CinderDisk> cinderDisks, OpenStackVolumeProviderProxy proxy) { + setStorageDomainId(storageDomainId); + setCinderDisks(cinderDisks); + setProxy(proxy); + } + + public Guid getStorageDomainId() { + return storageDomainId; + } + + public void setStorageDomainId(Guid storageDomainId) { + this.storageDomainId = storageDomainId; + } + + public List<CinderDisk> getCinderDisks() { + return cinderDisks; + } + + public void setCinderDisks(List<CinderDisk> cinderDisks) { + this.cinderDisks = cinderDisks; + } + + public OpenStackVolumeProviderProxy getProxy() { + return proxy; + } + + public void setProxy(OpenStackVolumeProviderProxy proxy) { + this.proxy = proxy; + } + } + public ValidationResult validateCinderDisksAlreadyRegistered() { return validate(new Callable<ValidationResult>() { @Override -- To view, visit https://gerrit.ovirt.org/41461 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I45c4584f336c606e48d51858d967b25cbd125aad Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipc...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
