Dan Kenigsberg has submitted this change and it was merged.

Change subject: caps: workaround for libvirt's canonical name in machine types
......................................................................


caps: workaround for libvirt's canonical name in machine types

Libvirt is supposed to return the machine type as 'text' attribute of the
XML. For the default machine type there is an alias in the "canonical"
attribute. Still each machine type is a separate <machine> tag.
In EL7 there is a regression in libvirt that the proper machine type's
name is not reported, only the "canonical" alias is - as part of the
default machine type entry.

This patch is workaround so we don't miss the latest machine type
and can be dropped once libvirt is fixed (bug 1229666).

Change-Id: Ic0bade940d7d548be18eb779cde324e37231ce73
Bug-Url: https://bugzilla.redhat.com/1229396
Signed-off-by: Martin Polednik <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/42082
Reviewed-by: Francesco Romani <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
Continuous-Integration: Dan Kenigsberg <[email protected]>
---
M debian/vdsm-tests.install
M tests/Makefile.am
M tests/capsTests.py
A tests/caps_libvirt_intel_E5606.out
M vdsm.spec.in
M vdsm/caps.py
6 files changed, 178 insertions(+), 7 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic0bade940d7d548be18eb779cde324e37231ce73
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Martin Polednik <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to