Hello Vinzenz Feenstra, Michal Skrivanek, Martin Polednik, I'd like you to do a code review. Please visit
http://gerrit.ovirt.org/19950 to review the following change. Change subject: vm.Vm._getUnderlyingDriveInfo: extract path of gluster disks ...................................................................... vm.Vm._getUnderlyingDriveInfo: extract path of gluster disks _getUnderlyingDriveInfo() is broken by design. It has no reliable means to match Engine-requested devices with libvirt-produced ones. As a heuristic, it uses the device's as matching key. However, before this patch, it failed to extract the path from the xml definition of gluster disks. Bug-Url: https://bugzilla.redhat.com/1007980 Change-Id: I4459916cab24d735c067a6eb7020d4f43505fc97 Signed-off-by: Dan Kenigsberg <dan...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/19906 Reviewed-by: Vinzenz Feenstra <vfeen...@redhat.com> Reviewed-by: Michal Skrivanek <michal.skriva...@redhat.com> Reviewed-by: Martin Polednik <mpole...@redhat.com> Tested-by: Martin Polednik <mpole...@redhat.com> --- M vdsm/vm.py 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/50/19950/1 diff --git a/vdsm/vm.py b/vdsm/vm.py index f0690b7..662eacc 100644 --- a/vdsm/vm.py +++ b/vdsm/vm.py @@ -4579,7 +4579,8 @@ sources = x.getElementsByTagName('source') if sources: devPath = (sources[0].getAttribute('file') or - sources[0].getAttribute('dev')) + sources[0].getAttribute('dev') or + sources[0].getAttribute('name')) else: devPath = '' -- To view, visit http://gerrit.ovirt.org/19950 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4459916cab24d735c067a6eb7020d4f43505fc97 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.3.0 Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Martin Polednik <mpole...@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches