[node-patches] Change in ovirt-node[master]: handle case where there missing are cpu details

mburns at redhat.com mburns at redhat.com
Thu Jan 3 21:59:09 UTC 2013


Michael Burns has submitted this change and it was merged.

Change subject: handle case where there missing are cpu details
......................................................................


handle case where there missing are cpu details

1.  Look only at the <cpu> section in the virsh capabilities
       otherwise, it could pull info from the wrong section
2.  put try/except block around each lookup
       vendor, model, topology
3.  Print Unknown for each if we can't determine it from xml
4.  If we don't know the vendow, don't print anything for
       NX or EVP

rhbz#887149

Change-Id: I1811bb17bf48d66cf601f564fd6cb75af681e004
Signed-off-by: Mike Burns <mburns at redhat.com>
---
M scripts/ovirtnode/ovirtfunctions.py
1 file changed, 28 insertions(+), 11 deletions(-)

Approvals:
  Michael Burns: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/10204
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1811bb17bf48d66cf601f564fd6cb75af681e004
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>



More information about the node-patches mailing list