Bug 1388532

Summary: Chart name changes when we change "interval" on chart options
Product: Red Hat CloudForms Management Engine Reporter: Jeeva Kandasamy <jkandasa>
Component: UI - OPSAssignee: Lucas Ponce <lponce>
Status: CLOSED CURRENTRELEASE QA Contact: Jeeva Kandasamy <jkandasa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: hkataria, jhardy, lponce, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly, Triaged
Target Release: 5.8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: Hawkular
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1391983 (view as bug list) Environment:
Last Closed: 2017-06-12 17:33:43 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Middleware Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1391983    
Attachments:
Description Flags
chart title none

Description Jeeva Kandasamy 2016-10-25 15:01:03 UTC
Created attachment 1213947 [details]
chart title

Description of problem:
When we change interval from "Hourly" to "Daily", "JVM Non Heap Usage (Bytes)" chart name change to "Non Heap Usage (Bytes)"


Version-Release number of selected component (if applicable):
Version: 5.7.0.4-alpha1.20161005153002_cfc8a23
Browser: Chrome
Browser Version: 49
Browser OS: Linux

Hawkular Services:	Running	0.0.16.Final-redhat-1 (commit 3d920b87b2fce2c326f6e9ae9190cb39d43852c1)	
Hawkular Metrics:	Running	0.20.1.Final	
Hawkular Alerts:	Running	1.2.2.Final	
Hawkular Inventory:	Running	0.20.0.Final

How reproducible:
100%

Steps to Reproduce:
1. Go CFME-middleware provider page
2. Select any server
3. Go to 'Utilization >> Monitoring'
4. Note "Non heap usage" chart name and change chart interval from "Hourly" to "Daily"

Actual results:
Chart name changes when we change interval option

Expected results:
Chart name never change.

Additional info:
Screen shot is attached.

Comment 2 Lucas Ponce 2016-10-26 11:09:47 UTC
Fixed on this PR:

https://github.com/ManageIQ/manageiq/pull/12196

Comment 3 CFME Bot 2016-10-26 21:11:27 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/e2b29d7ff23ea7e2eb14bac2cbc596ff64b83a42

commit e2b29d7ff23ea7e2eb14bac2cbc596ff64b83a42
Author:     Lucas Ponce <ponce.ballesteros>
AuthorDate: Wed Oct 26 13:06:33 2016 +0200
Commit:     Lucas Ponce <ponce.ballesteros>
CommitDate: Wed Oct 26 13:06:33 2016 +0200

    Fix wrong chart title on middleware server graph
    https://bugzilla.redhat.com/show_bug.cgi?id=1388532

 product/charts/layouts/daily_perf_charts/MiddlewareServer.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 4 CFME Bot 2016-11-01 15:41:02 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/1b9dff63c632902625a1871102adaf219e427563

commit 1b9dff63c632902625a1871102adaf219e427563
Author:     Jason Frey <fryguy9>
AuthorDate: Wed Oct 26 17:07:08 2016 -0400
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Tue Nov 1 11:36:16 2016 -0400

    Merge pull request #12196 from lucasponce/bz-1388532
    
    Fix wrong chart title on middleware server graph
    (cherry picked from commit 1031c5c2cd358cc1a2ffe395541df785c6ebb212)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1388532

 product/charts/layouts/daily_perf_charts/MiddlewareServer.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)