DaanHoogland commented on a change in pull request #3546: [FIX] Rethrow
takeVMSnapshot() exception
URL: https://github.com/apache/cloudstack/pull/3546#discussion_r350042914
##########
File path:
server/src/main/java/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java
##########
@@ -515,7 +515,7 @@ private VMSnapshot orchestrateCreateVMSnapshot(Long vmId,
Long vmSnapshotId, Boo
return snapshot;
} catch (Exception e) {
s_logger.debug("Failed to create vm snapshot: " + vmSnapshotId, e);
- return null;
+ throw new CloudRuntimeException("Failed to create vm snapshot: " +
vmSnapshotId, e);
Review comment:
ok, we'll have to find out in testing (or bad experience) I can not in good
conscience hold this given your answers in the comment, @onitake . Let's be
aware in case anything happens to the snapshot functionality
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
