[ https://issues.apache.org/jira/browse/CLOUDSTACK-8302?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15054348#comment-15054348 ]
ASF GitHub Bot commented on CLOUDSTACK-8302: -------------------------------------------- GitHub user dmytro-shevchenko opened a pull request: https://github.com/apache/cloudstack/pull/1230 CLOUDSTACK-8302: Removing snapshots on RBD Snapshot removing implemented if primary datastore is RBD https://issues.apache.org/jira/browse/CLOUDSTACK-8302 You can merge this pull request into a Git repository by running: $ git pull https://github.com/SafeSwissCloud/cloudstack CLOUDSTACK-8302 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1230.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1230 ---- commit 32df243422d5985c8631686a0b2a29835f804079 Author: Amin <amina...@gmail.com> Date: 2015-11-30T13:39:50Z Merge pull request #1 from apache/master Volume snapshot lifecycle patch on RBD as primary storage commit c4d749f3b41090c47d98431aef67fa769642c798 Author: Dmytro Shevchenko <dshevchenko.m...@gmail.com> Date: 2015-12-10T15:07:58Z Merge from apache/master commit a47b9300e414039734017fde0a41d162b54891f2 Author: Dmytro Shevchenko <dshevchenko.m...@gmail.com> Date: 2015-12-12T14:44:09Z https://issues.apache.org/jira/browse/CLOUDSTACK-8302 Snapshot removing implemented on RBD ---- > Cleanup snapshot on KVM with RBD > -------------------------------- > > Key: CLOUDSTACK-8302 > URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8302 > Project: CloudStack > Issue Type: Bug > Security Level: Public(Anyone can view this level - this is the > default.) > Components: KVM, Snapshot, Storage Controller > Affects Versions: 4.4.0, 4.4.1, 4.4.2 > Environment: CloudStack 4.4.2 + KVM on CentOS 6.6 + Ceph/RBD 0.80.8 > Reporter: Star Guo > Assignee: Wido den Hollander > Priority: Critical > > I just build a lab with CloudStack 4.4.2 + CentOS 6.6 KVM + Ceph/RBD 0.80.8. > I deploy an instance on RBD and I create the ROOT volume snapshots. When > delete a snapshot the UI show OK, but the snapshot of the volume in the RBD > pool is still exist. > And I find the code in > com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java: > … > @Override > public Answer deleteSnapshot(DeleteCommand cmd) { > return new Answer(cmd); > } > … > deleteSnapshot() does not be implememented. And I also find the code: > ... > @Override > public Answer createTemplateFromSnapshot(CopyCommand cmd) { > return null; //To change body of implemented methods use File | > Settings | File Templates. > } > ... > So does createTenokateFromSnapshot(). I just look for it in MASTER branch but > not do that yet. Will CloudStack Dev Team plan to do that ? Thanks . -- This message was sent by Atlassian JIRA (v6.3.4#6332)