Daniel Erez has uploaded a new change for review. Change subject: core: add deleteDisk CinderBroker ......................................................................
core: add deleteDisk CinderBroker Adding deleteDisk method to CinderBroker. Change-Id: Idd44fe8902f1912b3f8bf980acc781953be6b8de Bug-Url: https://bugzilla.redhat.com/185826 Signed-off-by: Daniel Erez <de...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/CinderBroker.java 1 file changed, 28 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/95/39195/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/CinderBroker.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/CinderBroker.java index b9d9290..71b4c45 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/CinderBroker.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/CinderBroker.java @@ -3,6 +3,7 @@ import com.woorea.openstack.base.client.OpenStackResponseException; import com.woorea.openstack.cinder.model.Volume; import com.woorea.openstack.cinder.model.VolumeForCreate; +import org.apache.commons.httpclient.HttpStatus; import org.ovirt.engine.core.bll.provider.storage.OpenStackVolumeProviderProxy; import org.ovirt.engine.core.common.businessentities.storage.CinderDisk; import org.ovirt.engine.core.common.businessentities.storage.CinderVolumeStatus; @@ -50,6 +51,33 @@ }); } + public Void deleteDisk(final CinderDisk cinderDisk) { + return execute(new Callable<Void>() { + @Override + public Void call() { + proxy.deleteVolume(cinderDisk.getId().toString()); + return null; + } + }); + } + + public boolean isDiskExist(final Guid id) { + return execute(new Callable<Boolean>() { + @Override + public Boolean call() { + try { + Volume volume = proxy.getVolumeById(id.toString()); + return volume != null; + } catch (OpenStackResponseException ex) { + if (ex.getStatus() == HttpStatus.SC_NOT_FOUND) { + return false; + } + throw ex; + } + } + }); + } + public ImageStatus getDiskStatus(final Guid id) { return execute(new Callable<ImageStatus>() { @Override -- To view, visit https://gerrit.ovirt.org/39195 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Idd44fe8902f1912b3f8bf980acc781953be6b8de Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <de...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches