Bug 1474096 - In Utilisation graph for Pods and Containers the Rounding of metrics is inconsistant
Summary: In Utilisation graph for Pods and Containers the Rounding of metrics is incon...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.9.0
Assignee: Robin Knaur
QA Contact: Einat Pacifici
URL:
Whiteboard:
Depends On:
Blocks: 1481436
TreeView+ depends on / blocked
 
Reported: 2017-07-23 20:58 UTC by Loic Avenel
Modified: 2018-06-06 08:42 UTC (History)
9 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1481436 (view as bug list)
Environment:
Last Closed: 2018-03-06 15:49:15 UTC
Category: Bug
Cloudforms Team: Container Management
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
screenshot (82.33 KB, image/png)
2017-07-23 20:58 UTC, Loic Avenel
no flags Details

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(-)


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