Bug 1384969 - Middleware- Error shows in UI while generating Middleware Reports
Summary: Middleware- Error shows in UI while generating Middleware Reports
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.8.0
Assignee: Lucas Ponce
QA Contact: Prachi
URL:
Whiteboard: hawkular
Depends On:
Blocks: 1388992
TreeView+ depends on / blocked
 
Reported: 2016-10-14 12:46 UTC by Prachi
Modified: 2019-08-06 20:06 UTC (History)
7 users (show)

Fixed In Version: 5.8.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1388992 (view as bug list)
Environment:
Last Closed: 2017-06-12 17:26:24 UTC
Category: ---
Cloudforms Team: Middleware
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
UI-screenshot (192.88 KB, image/png)
2016-10-14 12:46 UTC, Prachi
no flags Details

Description Prachi 2016-10-14 12:46:22 UTC
Created attachment 1210519 [details]
UI-screenshot

Description of problem:
Unable to generate a Report for middleware providers


Version-Release number of selected component (if applicable): CFME DR05


How reproducible:


Steps to Reproduce:
1. Login to CME
2. Navigate to Cloud Intel >> Reports
3. Navigate to Middleware Provider and select JVM garbage Collection/Heap usage/Non Heap usage
4. Click on "queue" tab	
5. it should generate a report


Actual results: It is not generating a report and displaying Error on UI (Attached Screen-shot)


Expected results: It should generate a Report and display


Additional info:

Comment 2 Lucas Ponce 2016-10-18 17:50:14 UTC
Yes. I confirmed this is a regression introduced in a previous refactoring.
Working on a fix.

Comment 3 Lucas Ponce 2016-10-19 10:21:45 UTC
Fixed on this PR:

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

Comment 4 CFME Bot 2016-10-24 19:06:29 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/23de1796aca18083f173076a2bcfa301f2fe8a79

commit 23de1796aca18083f173076a2bcfa301f2fe8a79
Author:     Lucas Ponce <ponce.ballesteros>
AuthorDate: Wed Oct 19 12:18:03 2016 +0200
Commit:     Lucas Ponce <ponce.ballesteros>
CommitDate: Thu Oct 20 13:06:23 2016 +0200

    Fix missing collect_stats_metrics
    https://bugzilla.redhat.com/show_bug.cgi?id=1384969

 .../middleware_manager/live_metrics_capture.rb      | 12 +++++++++++-
 app/models/middleware_performance.rb                | 21 ++++++++++++++-------
 app/models/mixins/live_metrics_mixin.rb             |  2 +-
 3 files changed, 26 insertions(+), 9 deletions(-)

Comment 5 CFME Bot 2016-10-24 20:00:57 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/c4af1bc975e3a0bb98712755c767bd2f42cdc344

commit c4af1bc975e3a0bb98712755c767bd2f42cdc344
Author:     Greg Blomquist <blomquisg>
AuthorDate: Mon Oct 24 15:01:56 2016 -0400
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Mon Oct 24 21:57:03 2016 +0200

    Merge pull request #12044 from lucasponce/bz-1384969
    
    Fix missing collect_stats_metrics
    (cherry picked from commit 97cba90b759b4ac50f48ec65c9122e9ecf48c8f8)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1384969

 .../middleware_manager/live_metrics_capture.rb      | 12 +++++++++++-
 app/models/middleware_performance.rb                | 21 ++++++++++++++-------
 app/models/mixins/live_metrics_mixin.rb             |  2 +-
 .../middleware_datasource_spec.rb                   | 15 +++++++++++++++
 .../middleware_manager/middleware_messaging_spec.rb | 15 +++++++++++++++
 .../middleware_manager/middleware_server_spec.rb    | 15 +++++++++++++++
 6 files changed, 71 insertions(+), 9 deletions(-)


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