Maor Lipchuk has uploaded a new change for review. Change subject: Images Handler <core | restapi | tools | history | engine | userportal | webadmin>: short summary under 50 chars ......................................................................
Images Handler <core | restapi | tools | history | engine | userportal | webadmin>: short summary under 50 chars Longer description using lines' length under 72 chars. With multiple paragraphs if necessary. Change-Id: Ib02f0bf3ce4ece4fc444407721808d1e119dd2d3 Bug-Url: https://bugzilla.redhat.com/?????? Signed-off-by: Maor Lipchuk <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java 1 file changed, 25 insertions(+), 8 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/44/40744/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java index 7c78ba0..905f4ed 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java @@ -30,7 +30,7 @@ import org.ovirt.engine.core.common.businessentities.VmTemplate; import org.ovirt.engine.core.common.businessentities.network.VmNetworkInterface; import org.ovirt.engine.core.common.businessentities.storage.BaseDisk; -import org.ovirt.engine.core.common.businessentities.storage.CinderDisk; +import org.ovirt.engine.core.common.businessentities.storage.Disk; import org.ovirt.engine.core.common.businessentities.storage.Disk; import org.ovirt.engine.core.common.businessentities.storage.DiskStorageType; import org.ovirt.engine.core.common.businessentities.storage.DiskImage; @@ -573,6 +573,23 @@ return diskImages; } + public static List<Disk> filterDisksByType(Collection<? extends Disk> listOfDisks, + boolean allowOnlyNotShareableDisks, + boolean allowOnlySnapableDisks, + boolean allowOnlyActiveDisks, + DiskStorageType... filteredStorageTypes) { + List<Disk> diskImages = new ArrayList<Disk>(); + for (Disk disk : listOfDisks) { + if (filteredStorageTypes.contains(disk.getDiskStorageType()) && + (!allowOnlyNotShareableDisks || !disk.isShareable()) && + (!allowOnlySnapableDisks || disk.isAllowSnapshot()) && + (!allowOnlyActiveDisks || Boolean.TRUE.equals(((DiskImage)disk).getActive()))) { + diskImages.add(disk); + } + } + return diskImages; + } + public static List<LunDisk> filterDiskBasedOnLuns(Collection<Disk> listOfDisks) { List<LunDisk> lunDisks = new ArrayList<LunDisk>(); for (Disk disk : listOfDisks) { @@ -583,21 +600,21 @@ return lunDisks; } - public static List<CinderDisk> filterDisksBasedOnCinder(Collection<? extends Disk> listOfDisks) { - return filterDisksBasedOnCinder(listOfDisks, false); + public static List<Disk> filterDisksBasedOn(Collection<? extends Disk> listOfDisks) { + return filterDisksBasedOn(listOfDisks, false); } - public static List<CinderDisk> filterDisksBasedOnCinder(Collection<? extends Disk> listOfDisks, + public static List<Disk> filterDisksBasedOn(Collection<? extends Disk> listOfDisks, boolean onlyPluggedDisks) { - List<CinderDisk> cinderDisks = new ArrayList<>(); + List<Disk> Disks = new ArrayList<>(); for (Disk disk : listOfDisks) { - if (disk.getDiskStorageType() == DiskStorageType.CINDER) { + if (disk.getDiskStorageType() == DiskStorageType.) { if (!onlyPluggedDisks || disk.getPlugged()) { - cinderDisks.add((CinderDisk) disk); + Disks.add((Disk) disk); } } } - return cinderDisks; + return Disks; } public static void removeDiskImage(DiskImage diskImage, Guid vmId) { -- To view, visit https://gerrit.ovirt.org/40744 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ib02f0bf3ce4ece4fc444407721808d1e119dd2d3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
