CLOUDSTACK-7700: Volume Snapshot Async Job returns Success for a failed
operation.


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/0db0e5e3
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/0db0e5e3
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/0db0e5e3

Branch: refs/heads/master
Commit: 0db0e5e3cc91106002892a582e9843284eee19b4
Parents: a6d575b
Author: Min Chen <min.c...@citrix.com>
Authored: Mon Oct 13 14:25:52 2014 -0700
Committer: Min Chen <min.c...@citrix.com>
Committed: Mon Oct 13 15:03:59 2014 -0700

----------------------------------------------------------------------
 .../src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/0db0e5e3/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java
----------------------------------------------------------------------
diff --git 
a/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java
 
b/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java
index 821c939..d97ad60 100644
--- 
a/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java
+++ 
b/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java
@@ -1534,6 +1534,7 @@ public class VolumeServiceImpl implements VolumeService {
             snapshot = snapshotMgr.takeSnapshot(volume);
         } catch (Exception e) {
             s_logger.debug("Take snapshot: " + volume.getId() + " failed", e);
+            throw new CloudRuntimeException("Failed to take snapshot", e);
         }
 
         return snapshot;

Reply via email to