Bug 1384969

Summary: Middleware- Error shows in UI while generating Middleware Reports
Product: Red Hat CloudForms Management Engine Reporter: Prachi <pyadav>
Component: ProvidersAssignee: Lucas Ponce <lponce>
Status: CLOSED CURRENTRELEASE QA Contact: Prachi <pyadav>
Severity: high Docs Contact:
Priority: high    
Version: 5.7.0CC: cpelland, gblomqui, jfrey, jhardy, mmahoney, 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:
: 1388992 (view as bug list) Environment:
Last Closed: 2017-06-12 17:26:24 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:
Embargoed:
Bug Depends On:    
Bug Blocks: 1388992    
Attachments:
Description Flags
UI-screenshot none

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