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 <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/42736
Continuous-Integration: Jenkins CI
Reviewed-by: Michal Skrivanek <[email protected]>
Reviewed-by: Francesco Romani <[email protected]>
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 <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to