Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 1012984 - Host detail sub-tab: values of CPU Name and CPU Type are switched
Summary: Host detail sub-tab: values of CPU Name and CPU Type are switched
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine-webadmin-portal
Version: 3.3.0
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
: 3.5.0
Assignee: Frantisek Kobzik
QA Contact: Pavel Novotny
Whiteboard: virt
Depends On:
Blocks: 1056586 rhev3.5beta 1156165
TreeView+ depends on / blocked
Reported: 2013-09-27 13:49 UTC by Pavel Novotny
Modified: 2015-02-17 08:29 UTC (History)
8 users (show)

Fixed In Version: ovirt-3.5.0-alpha1
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1056586 (view as bug list)
Last Closed: 2015-02-17 08:29:30 UTC
oVirt Team: ---
Target Upstream Version:

Attachments (Terms of Use)
screenshot - host's General sub-tab (deleted)
2013-09-27 13:49 UTC, Pavel Novotny
no flags Details

System ID Priority Status Summary Last Updated
oVirt gerrit 20111 None None None Never
oVirt gerrit 20114 None None None Never

Description Pavel Novotny 2013-09-27 13:49:55 UTC
Created attachment 803969 [details]
screenshot - host's General sub-tab

Description of problem:
The values of host CPU Name and CPU Type on the General sub-tab are obviously switched.
See attached screenshot.

Version-Release number of selected component (if applicable):
rhevm-3.3.0-0.23.master.el6ev.noarch (is16)

How reproducible:

Steps to Reproduce:
1. Assuming we have host with CPU 'Intel(R) Core(TM)2 Quad CPU Q8400 @ 2.66GHz', which belongs to the 'Intel Penryn Family'...
2. In the Hosts main tab, select the host, go to General sub-tab and check the values of CPU Name and CPU Type. 

Actual results:
CPU Name: Intel Penryn Family
CPU Type: Intel(R) Core(TM)2 Quad CPU Q8400 @ 2.66GHz

Expected results:
Exactly the opposite, i.e.,
CPU Name: Intel(R) Core(TM)2 Quad CPU Q8400 @ 2.66GHz
CPU Type: Intel Penryn Family

Additional info:

This seems to be related only to Webadmin, REST API reports the CPU name correctly:
  <topology sockets="1" cores="4" threads="1"/>
  <name>Intel(R) Core(TM)2 Quad CPU    Q8400  @ 2.66GHz</name>

Comment 1 Frantisek Kobzik 2013-10-08 09:47:35 UTC
Guys, it seems that this bug reveals quite a lot of inconsistencies in the engine:

1, Inconsistency between Backend, Frontend and Restapi:
The backend field "VDS.cpuModel" (value e.g. "Intel(R) Core(TM)2 Quad CPU    Q8400  @ 2.66GHz") is called "type" in Host general subtab and "name" in restapi.

2, Inconsistency between Backend and Frontend:
The backend field "VDS.cpuName" (its type is ServerInfo (lol) and value e.g. "Intel Penryn Family") is called "name" in Host general subtab. 

3, Inconsistency in frontend - Clusters tab:
In this tab there is a column "Cluster CPU Type", its values looks like "Intel Penryn Family". (btw, in the code the method for retrieving this string is called "cpuNameCluster" and the column is called "cpuNameColumn" - another lol). 

...and we could go on.

Therefore I propose unifying the field names to be consistent to overcome confusion. For this I need to know what the right naming of following values should be:
1, "Intel(R) Core(TM)2 Quad CPU    Q8400  @ 2.66GHz" -> type, model, name?
2, "Intel Penryn Family" -> type, family?

ad 1, vdsm reports this value as "cpuModel", in /proc/cpuinfo this is reported as "name", dmidecode reports this as a "version" - obviously there are inconsistencies on system level as well. 

So could someone give me a pointer on how to name these things to make them unified?

Comment 2 Frantisek Kobzik 2013-10-11 11:36:42 UTC
I've just posted a patch that renames fields on frontend. See the commit msg if you're interested. In order to unify the naming on frontend level, I'll post another patch as a follow-up that unifies the naming also in the cluster view.

Comment 3 Frantisek Kobzik 2013-10-11 12:56:10 UTC
added patch for cluster views.

Comment 4 Pavel Novotny 2014-08-07 08:42:41 UTC
Verified in ovirt-engine-3.5.0-0.0.master.20140804172041.git23b558e.el6.noarch (rc1).

CPU name/family naming:
Before - host -> General sub-tab:
  CPU Name: Intel Nehalem Family
  CPU Type: Intel(R) Core(TM)2 Quad CPU    Q8400  @ 2.66GHz

Now - host -> Hardware Information (*) sub-tab:
  CPU Model: Intel(R) Core(TM)2 Quad CPU    Q8400  @ 2.66GHz
  CPU Type: Intel Nehalem Family
(*) CPU info have been moved to HW Info sub-tab as part of bug 894084

Also label in new/edit cluster dialog has been updated: CPU Name -> CPU Type.

Comment 5 Omer Frenkel 2015-02-17 08:29:30 UTC
RHEV-M 3.5.0 has been released

Note You need to log in before you can comment on or make changes to this bug.