Change in vdsm[ovirt-3.6]: virt: add logic for POWER cpu xml element
fromani at redhat.com
fromani at redhat.com
Mon Sep 21 13:16:59 UTC 2015
Francesco Romani has submitted this change and it was merged.
Change subject: virt: add logic for POWER cpu xml element
......................................................................
virt: add logic for POWER cpu xml element
Current flow in appendCpu caused ppc to be skipped, generating empty
element (<cpu/>). In order to specify cpu model on ppc, we have to use
libvirt's cpu with POWER8 model.
This is currently the only way of safely adding CPU to ppc64le host,
and hopefully will be enhanced in future.
Change-Id: I9303b76904ef1344508136343104b37c09d2a370
Bug-Url: https://bugzilla.redhat.com/1259318
Signed-off-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/42736
Continuous-Integration: Jenkins CI
Reviewed-by: Michal Skrivanek <michal.skrivanek at redhat.com>
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/46451
---
M tests/vmTestsData.py
M vdsm/virt/vmxml.py
2 files changed, 33 insertions(+), 1 deletion(-)
Approvals:
Jenkins CI: Passed CI tests
Francesco Romani: Looks good to me, approved
Michal Skrivanek: Looks good to me, but someone else must approve
Martin Polednik: Verified
--
To view, visit https://gerrit.ovirt.org/46451
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9303b76904ef1344508136343104b37c09d2a370
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the vdsm-patches
mailing list