Change in vdsm[master]: caps: Do not use lscpu on ppc64

danken at redhat.com danken at redhat.com
Tue Mar 24 11:38:00 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: caps: Do not use lscpu on ppc64
......................................................................


caps: Do not use lscpu on ppc64

Recent RHEL for power and powerKVM correctly show the
topology of ppc64 hosts, so it is not required the use of 'lscpu'
to obtain this information.

Change-Id: I421c82f46a433387b5d7eb1e4624b5f166ee344c
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1148092
Signed-off-by: Vitor de Lima <vdelima at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/33872
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-by: Michal Skrivanek <michal.skrivanek at redhat.com>
Tested-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-by: Madhu Pavan <kmp at linux.vnet.ibm.com>
---
M debian/vdsm-tests.install
M tests/Makefile.am
M tests/capsTests.py
A tests/caps_libvirt_ibm_S822L.out
D tests/caps_lscpu_ppc64_1_4_4.out
D tests/caps_lscpu_ppc64_2_4_8.out
M vdsm.spec.in
M vdsm/caps.py
M vdsm/ppc64HardwareInfo.py
9 files changed, 193 insertions(+), 90 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Madhu Pavan: Looks good to me, but someone else must approve
  Michal Skrivanek: Looks good to me, but someone else must approve
  Martin Polednik: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I421c82f46a433387b5d7eb1e4624b5f166ee344c
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vitor de Lima <vdelima at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Madhu Pavan <kmp at linux.vnet.ibm.com>
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vitor de Lima <vdelima at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list