Dan Kenigsberg has submitted this change and it was merged. Change subject: sd: fix volume path returned by linkBCImage ......................................................................
sd: fix volume path returned by linkBCImage In a previous commit (c072945 One shot prepare) we involuntarily changed the path used for virtual machine images from: /rhev/data-center/<spUUID>/<sdUUID>/images/<imgUUID>/<volUUID> to: /rhev/data-center/mnt/blockSD/<sdUUID>/images/<imgUUID>/<volUUID> This patch reverts to the previous path in order to minimize the differences between different versions of vdsm that could lead to live migration issues, e.g. libvirtError: invalid argument: invalid path ... not assigned to domain Change-Id: Iea497b574d9a9a7e0e9b8e234e2bd9b9d983a60c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1059482 Signed-off-by: Federico Simoncelli <[email protected]> Reviewed-on: http://gerrit.ovirt.org/24315 Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/storage/blockSD.py M vdsm/storage/fileSD.py M vdsm/storage/sd.py 3 files changed, 8 insertions(+), 4 deletions(-) Approvals: Federico Simoncelli: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24315 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iea497b574d9a9a7e0e9b8e234e2bd9b9d983a60c Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
