Bug 1436706

Summary: Reports - Number of Nodes per CPU cores - Wrong Name of report
Product: Red Hat CloudForms Management Engine Reporter: Gilad Shefer <gshefer>
Component: ProvidersAssignee: Nimrod Shneor <nshneor>
Status: CLOSED CURRENTRELEASE QA Contact: Gilad Shefer <gshefer>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.8.0CC: cpelland, dajohnso, epacific, fsimonce, gshefer, jfrey, jhardy, obarenbo, simaishi, yzamir
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: container:ui:report
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1461541 (view as bug list) Environment:
Last Closed: 2018-03-06 14:47:34 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Container Management Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1461541    

Description Gilad Shefer 2017-03-28 13:19:23 UTC 
Description of problem:
The report should be called "Nodes by Number of CPU Cores" since it presenting "Node Name" (column 1) and its "Hardware Number of CPU Cores" (column 2) it uses.

Version-Release number of selected component (if applicable):
5.7 / 5.8

How reproducible:
100%

Steps to Reproduce:
1. Could Intel --> Reports -(accordion)-> Reports --> Configuration Management --> Containers --> Number of Nodes per CPU cores
2.
3.

Actual results:
"Number of Nodes per CPU cores"

Expected results:
"Nodes by Number of CPU Cores"

Additional info:
Comment 2 Nimrod Shneor 2017-04-05 08:30:30 UTC 
Fixed int the following PR: https://github.com/ManageIQ/manageiq/pull/14646
Comment 3 Yaacov Zamir 2017-04-05 08:58:58 UTC 
The report **chart** show

Y: Number of nodes
X: Hardware number of CPU cores

So the currect name should be:
"Number of Nodes per Hardware number of CPU cores"

What is the currect title ?
Comment 4 Nimrod Shneor 2017-04-05 09:10:48 UTC 
see comment in PR
Comment 5 Yaacov Zamir 2017-04-05 09:27:07 UTC 
Gilad Hi,

The chart in this report has no title or legend,
No one can know what the x and y really represent ... 

We should not have "mystery" charts in our reports, can you open a bug about that ?
Comment 6 Nimrod Shneor 2017-04-18 08:27:09 UTC 
After discussing this with Gilad and Federico we have decided to take the route Federico had suggested, meaning, the report will show:

Number of Nodes	|Number of CPU Cores
4	        |8
32	        |4

As the original title suggested.
Comment 8 Nimrod Shneor 2017-05-27 14:53:58 UTC 
After further consideration reverting back to changing the name of the report in the following: 
https://github.com/ManageIQ/manageiq/pull/14646