cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chris Suich" <>
Subject Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot
Date Tue, 22 Oct 2013 16:42:04 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Oct. 22, 2013, 4:42 p.m.)

Review request for cloudstack, Brian Federle, edison su, John Burwell, and Mike Tutkowski.

Repository: cloudstack-git


After the last batch of work to the revertSnapshot API, SnapshotServiceImpl was not tied into
the workflow to be used by storage providers. I have added the logic in a similar fashion
to takeSnapshot(), backupSnapshot() and deleteSnapshot().

I have also added a 'Revert to Snapshot' action to the volume snapshots list in the UI.


  server/src/com/cloud/api/ 6978c9b 



I have tested all of this locally with a custom storage provider.

Unfortunately, I'm still in the middle of figuring out how to properly unit test this type
of code. If anyone has any recommendations, please let me know.

File Attachments (updated)


Chris Suich

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message