Dan Kenigsberg has submitted this change and it was merged. Change subject: Live Merge: Return more info from _driveGetActualVolumeChain ......................................................................
Live Merge: Return more info from _driveGetActualVolumeChain In followup patches, we want to return more information about each path component from _driveGetActualVolumeChain. We need the actual path as given in the domain XML in order to properly update our metadata after an active layer merge. We also want to use this function to return high write watermark information for block volumes. This patch just converts the format returned by _driveGetActualVolumeChain from a simple list to a list of named tuples. Change-Id: I01d142a68903048accc95eb04d9930c326965db0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1109920 Signed-off-by: Adam Litke <[email protected]> Reviewed-on: http://gerrit.ovirt.org/31364 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-on: http://gerrit.ovirt.org/32319 Reviewed-by: Federico Simoncelli <[email protected]> Reviewed-by: Francesco Romani <[email protected]> --- M vdsm/virt/vm.py 1 file changed, 12 insertions(+), 6 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/32319 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I01d142a68903048accc95eb04d9930c326965db0 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
