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

Reply via email to