[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