Bug 1474096

Summary: In Utilisation graph for Pods and Containers the Rounding of metrics is inconsistant
Product: Red Hat CloudForms Management Engine Reporter: Loic Avenel <lavenel>
Component: UI - OPSAssignee: Robin Knaur <rknaur>
Status: CLOSED CURRENTRELEASE QA Contact: Einat Pacifici <epacific>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.8.0CC: dclarizi, gtanzill, hkataria, jhardy, lavenel, mpovolny, obarenbo, rknaur, simaishi
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1481436 (view as bug list) Environment:
Last Closed: 2018-03-06 15:49:15 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Container Management Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1481436    
Attachments:
Description Flags
screenshot none

Description Loic Avenel 2017-07-23 20:58:13 UTC
Created attachment 1303340 [details]
screenshot

Description of problem: In Utilisation graph for Pods and Containers the Rounding of metrics is inconsistant


Version-Release number of selected component (if applicable): 5.8.0.17.20170525183055_6317a22


How reproducible: always


Steps to Reproduce:
1. go to a pod or a container
2. look at utilisation
3.

Actual results: Number are display 3 times with 3 different rounding


Expected results: Rounding match..


Additional info: see Screenshots for details..

Comment 2 Dave Johnson 2017-07-26 00:28:10 UTC
Please assess the impact of this issue and update the severity accordingly.  Please refer to https://bugzilla.redhat.com/page.cgi?id=fields.html#bug_severity for a reminder on each severity's definition.

If it's something like a tracker bug where it doesn't matter, please set it to Low/Low.

Comment 3 Federico Simoncelli 2017-07-31 13:58:40 UTC
Gregg, what do you think, is this a UI or reporting issue?

Comment 4 Dan Clarizio 2017-07-31 16:42:19 UTC
Taking this as a UI issue.

Comment 5 Dan Clarizio 2017-07-31 16:44:08 UTC
Robin, I had thought we had consistency at least on the axis labels, to always show the same number of digits.  Perhaps we need a discussion regarding the report, axis, and hover text consistency.

Comment 6 Robin Knaur 2017-08-01 07:48:01 UTC
I am pretty sure, that this is issue for all CU charts, not only for pods and containers. Now we dynamically adjust precision for each chart based on data which are actually shown. When you hide/show data series it is also recalculated.

We don't have anchored Min or Max value, so there have been situations when you had chart like one at screenshot, but all labels were same, like 0.001. But chart look like a curve, so it was weird. Now labels should match what you see everywhere, but you can get this unnecessary precision like in this case.

Hover tooltips have one more floating point precision, than Y axis labels.

Comment 7 Dan Clarizio 2017-08-01 15:25:03 UTC
Robin, could we at least make the Y axis labels and the tooltip text have the same precision?  That would provide consistency at least within the chart itself.

Comment 8 Robin Knaur 2017-08-02 14:39:27 UTC
Yep, of course, that won't be a problem. Report table, on the other hand, would be much more problematic. If we want to keep chart responsibility, it would be very hard to keep consistency between charts and table.

Comment 10 Dave Johnson 2017-08-03 23:50:47 UTC
Please assess the impact of this issue and update the severity accordingly.  Please refer to https://bugzilla.redhat.com/page.cgi?id=fields.html#bug_severity for a reminder on each severity's definition.

If it's something like a tracker bug where it doesn't matter, please set it to Low/Low.

Comment 11 CFME Bot 2017-08-10 06:33:44 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/4de9579644a70c57d711ff13077da9d2433e2bf2

commit 4de9579644a70c57d711ff13077da9d2433e2bf2
Author:     PanSpagetka <rknaur>
AuthorDate: Wed Aug 2 16:43:46 2017 +0200
Commit:     PanSpagetka <rknaur>
CommitDate: Wed Aug 2 16:44:17 2017 +0200

    Make tooltip precision for CU chart constistent with Y axis labels
    https://bugzilla.redhat.com/show_bug.cgi?id=1474096

 app/assets/javascripts/miq_application.js | 1 -
 1 file changed, 1 deletion(-)