Francesco Romani has submitted this change and it was merged.

Change subject: hostdev: decode XML string when parsing devices
......................................................................


hostdev: decode XML string when parsing devices

As there is no encoding of information and libvirt does not handle
encoding, our best bet is ascii. This patch therefore converts device
XMLs to ascii, ignoring the errors found.

Change-Id: Iaf9b9699790c83bb82a88a700c63d05604588d02
Bug-Url: https://bugzilla.redhat.com/1315435
Signed-off-by: Martin Polednik <mpoled...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/54802
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <from...@redhat.com>
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/54980
---
M tests/devices/data/Makefile.am
A tests/devices/data/computer.xml
M tests/hostdevTests.py
M tests/vmfakelib.py
M vdsm/hostdev.py
5 files changed, 32 insertions(+), 2 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, but someone else must approve
  Francesco Romani: Looks good to me, approved
  Martin Polednik: Verified



-- 
To view, visit https://gerrit.ovirt.org/54980
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf9b9699790c83bb82a88a700c63d05604588d02
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Martin Polednik <mpoled...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to