Bug 1375093 - Compliance history is broken for a VM
Summary: Compliance history is broken for a VM
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.6.0
Hardware: All
OS: All
medium
medium
Target Milestone: GA
: 5.8.0
Assignee: Harpreet Kataria
QA Contact: Dmitry Misharov
URL:
Whiteboard: control
Depends On:
Blocks: 1382845 1382847
TreeView+ depends on / blocked
 
Reported: 2016-09-12 07:36 UTC by Dmitry Misharov
Modified: 2017-06-12 17:03 UTC (History)
6 users (show)

Fixed In Version: 5.8.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1382845 1382847 (view as bug list)
Environment:
Last Closed: 2017-06-12 17:03:37 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Screenshot of the bug (203.17 KB, image/png)
2016-09-12 07:36 UTC, Dmitry Misharov
no flags Details
Details of last compliance check (94.92 KB, image/png)
2016-09-12 07:37 UTC, Dmitry Misharov
no flags Details

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.


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