Bug 1375093

Summary: Compliance history is broken for a VM
Product: Red Hat CloudForms Management Engine Reporter: Dmitry Misharov <dmisharo>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED CURRENTRELEASE QA Contact: Dmitry Misharov <dmisharo>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.6.0CC: cpelland, hkataria, jhardy, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly
Target Release: 5.8.0   
Hardware: All   
OS: All   
Whiteboard: control
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1382845 1382847 (view as bug list) Environment:
Last Closed: 2017-06-12 17:03:37 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1382845, 1382847    
Attachments:
Description Flags
Screenshot of the bug
none
Details of last compliance check none

Description Dmitry Misharov 2016-09-12 07:36:13 UTC
Created attachment 1200076 [details]
Screenshot of the bug

Description of problem:
Compliance history link in a VM details screen doesn't work. Details of the last compliance check show whole compliance check history.

Version-Release number of selected component (if applicable):
5.7.0.0.20160906172503_12f65cb 

How reproducible:
Always

Steps to Reproduce:
1. Create any VM compliance policy.
2. Assign it to a policy profile.
3. Assign the policy profile to any VM.
4. Perform the compliance check for the VM.
5. Go to the VM details screen.
6. Click on "History" row in Compliance table and then on "Status" roe.

Actual results:
The same VM details screen will be opened and the toolbar will disappear if "History" row was clicked. Whole history of compliance check is shown if "Status" row was clicked.

Expected results:
Compliance history screen with last 10 checks should be opened for "History" and details of the last check should be shown for "Status".

Comment 2 Dmitry Misharov 2016-09-12 07:37:11 UTC
Created attachment 1200077 [details]
Details of last compliance check

Comment 4 Harpreet Kataria 2016-09-28 16:10:04 UTC
https://github.com/ManageIQ/manageiq/pull/11564

Comment 5 CFME Bot 2016-09-28 22:51:14 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/55161c7508091cd76fb91ce1444b841502155636

commit 55161c7508091cd76fb91ce1444b841502155636
Author:     Harpreet Kataria <hkataria>
AuthorDate: Tue Sep 27 17:36:55 2016 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Wed Sep 28 13:53:13 2016 -0400

    Fixed link to Compliance History/Status screen.
    
    Needed to set explorer key in the returning hash so AJAX url can be built for explorer screens.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1375093

 app/helpers/compliance_summary_helper.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comment 6 CFME Bot 2016-09-28 22:51:19 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/fdd2339cc30137ff6b50ddb6f9aae1b118546db5

commit fdd2339cc30137ff6b50ddb6f9aae1b118546db5
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed Sep 28 11:56:34 2016 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Wed Sep 28 16:52:55 2016 -0400

    spec test to verify classic/explorer screen links.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1375093

 spec/helpers/compliance_summary_helper_spec.rb | 64 ++++++++++++++++++++++++++
 1 file changed, 64 insertions(+)
 create mode 100644 spec/helpers/compliance_summary_helper_spec.rb

Comment 9 Dmitry Misharov 2017-03-08 09:20:10 UTC
Verified in 5.8.0.4.20170307183144_745a3d9, compliance history is opened correctly.