Bug 1348796

Summary: No gtl view buttons in cloud key pair page
Product: Red Hat CloudForms Management Engine Reporter: Aziza Karol <akarol>
Component: UI - OPSAssignee: Eric Hayes <ehayes>
Status: CLOSED ERRATA QA Contact: Aziza Karol <akarol>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.6.0CC: ehayes, hkataria, jhardy, jprause, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: ZStream
Target Release: 5.7.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: view:ui
Fixed In Version: 5.7.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1353280 (view as bug list) Environment:
Last Closed: 2017-01-04 12:56:01 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1353280    
Attachments:
Description Flags
snapshot none

Description Aziza Karol 2016-06-22 06:17:02 UTC
Created attachment 1170598 [details]
snapshot

Description of problem:


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

How reproducible:
100%

Steps to Reproduce:
In Default view settings there is an option to set gtl view for key pairs, but in 
cloud key pairs page gtl view buttons are missing.
see attached screenshot

Actual results:


Expected results:


Additional info:

Comment 1 Harpreet Kataria 2016-06-22 13:37:04 UTC
Eric,

Looks like we need to add @layout for Cloud Key Pairs screen to the list here https://github.com/ManageIQ/manageiq/blob/master/app/helpers/application_helper.rb#L1060

Let me know if you have questions.

~Harpreet

Comment 4 CFME Bot 2016-06-23 20:06:44 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/e3db9d1d079d2849f25a742d0e37a3622df2ea36

commit e3db9d1d079d2849f25a742d0e37a3622df2ea36
Author:     Eric Hayes <eric>
AuthorDate: Wed Jun 22 12:13:21 2016 -0700
Commit:     Eric Hayes <eric>
CommitDate: Thu Jun 23 07:59:26 2016 -0700

    Allow gtl view for Cloud Key Pairs
    
    * Adds `auth_key_pair_cloud` to layout configuration constant
    * Adds kludge in `quadicon_helper` for detecting Authentication class and setting correct controller. (I’m working on refactoring this.)
    * Cleans up some style and adds a note for future refactoring
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1348796

 app/helpers/application_helper.rb     | 18 +++++++++---------
 app/helpers/quadicon_helper.rb        |  3 +++
 app/models/authentication.rb          |  5 +++++
 app/views/layouts/gtl/_list.html.haml |  2 ++
 4 files changed, 19 insertions(+), 9 deletions(-)

Comment 6 CFME Bot 2016-07-06 16:51:24 UTC
New commit detected on ManageIQ/manageiq/darga:
https://github.com/ManageIQ/manageiq/commit/0e179d7f9e0ce9b0cb5ed29326978f43f81ac4a4

commit 0e179d7f9e0ce9b0cb5ed29326978f43f81ac4a4
Author:     Eric Hayes <eric>
AuthorDate: Tue Jul 5 11:50:27 2016 -0700
Commit:     Eric Hayes <eric>
CommitDate: Tue Jul 5 13:29:11 2016 -0700

    Allow gtl view for Cloud Key Pairs
    
    * Adds `auth_key_pair_cloud` to layout configuration constant
    * Adds kludge in `quadicon_helper` for detecting Authentication class and setting correct controller. (I’m working on refactoring this.)
    * Cleans up some style and adds a note for future refactoring
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1348796
    (cherry picked from commit e3db9d1d079d2849f25a742d0e37a3622df2ea36)

 app/helpers/application_helper.rb     | 18 +++++++++---------
 app/helpers/quadicon_helper.rb        |  3 +++
 app/models/authentication.rb          |  5 +++++
 app/views/layouts/gtl/_list.html.haml |  1 +
 4 files changed, 18 insertions(+), 9 deletions(-)

Comment 11 Aziza Karol 2016-11-04 05:12:38 UTC
GTL view buttons added in cloud key pair page.

Verified:5.7.0.9-beta2.1

Comment 12 Aziza Karol 2016-11-04 05:14:09 UTC
However, updating the gtl view settings does not work.I have logged a separate bz https://bugzilla.redhat.com/show_bug.cgi?id=1391792 to track this issue

Comment 14 errata-xmlrpc 2017-01-04 12:56:01 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2017-0012.html