Amador Pahim has uploaded a new change for review. Change subject: libvirtvm: Removing libvirt snapshot metadata. ......................................................................
libvirtvm: Removing libvirt snapshot metadata. Bug-Url: http://bugzilla.redhat.com/872277 After creating a new snapshot, it is not possible to live migrate vm: libvirtError: Requested operation is not valid: cannot migrate domain with 1 snapshots Issue is also affecting live storage migration due to use of the same vm snapshot function during the live storage migration process. This pach removes unused libvirt snapshot metadata just after snapshot creation allowing live migration. Change-Id: I928983a404f822522a0a48218f9c58fa8d6296b1 Signed-off-by: Amador Pahim <[email protected]> --- M vdsm/libvirtvm.py 1 file changed, 5 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/39/9139/1 diff --git a/vdsm/libvirtvm.py b/vdsm/libvirtvm.py index ca31a43..b213206 100644 --- a/vdsm/libvirtvm.py +++ b/vdsm/libvirtvm.py @@ -1913,6 +1913,11 @@ finally: self.startDisksStatsCollection() + # Removing libvirt snapshot metadata + curSnap = self._dom.snapshotCurrent(0) + snapDelFlags = (libvirt.VIR_DOMAIN_SNAPSHOT_DELETE_METADATA_ONLY) + curSnap.delete(snapDelFlags) + # Successful break -- To view, visit http://gerrit.ovirt.org/9139 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I928983a404f822522a0a48218f9c58fa8d6296b1 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Amador Pahim <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
