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