Dan Kenigsberg has submitted this change and it was merged. Change subject: Live Merge: Update path using libvirt supplied value ......................................................................
Live Merge: Update path using libvirt supplied value When libvirt pivots the active layer after a live merge, the live XML is updated to reflect the path of the new active layer. The new path is copied from the backingStore pointer of the old active layer. Since that is a relative path, libvirt makes it absolute but it will not remove the '../' path component since that is not always correct (even though it would be correct for our environment). When syncing the volume chain, we must use the same path as libvirt does, otherwise future calls to libvirt may fail. Change-Id: Ie341333ad934645c0fe4f2fd31733c459d306d9a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1109920 Signed-off-by: Adam Litke <[email protected]> Reviewed-on: http://gerrit.ovirt.org/31365 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-on: http://gerrit.ovirt.org/32320 Reviewed-by: Federico Simoncelli <[email protected]> Reviewed-by: Francesco Romani <[email protected]> --- M vdsm/virt/vm.py 1 file changed, 8 insertions(+), 3 deletions(-) Approvals: Adam Litke: Verified Federico Simoncelli: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Francesco Romani: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/32320 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie341333ad934645c0fe4f2fd31733c459d306d9a Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Adam Litke <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
