Bug 1352914

Summary: Key Pairs: wrong quadicon displayed
Product: Red Hat CloudForms Management Engine Reporter: Harpreet Kataria <hkataria>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED CURRENTRELEASE QA Contact: Matouš Mojžíš <mmojzis>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.6.0CC: cpelland, dclarizi, hkataria, jhardy, mmojzis, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly
Target Release: 5.8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ui:cloud
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1353228 1385170 1385173 (view as bug list) Environment:
Last Closed: 2017-06-12 16:14:30 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: 1353228, 1385170, 1385173    
Attachments:
Description Flags
key pairs list view none

Description Harpreet Kataria 2016-07-05 13:18:20 UTC 
Created attachment 1176429 [details]
key pairs list view

Description of problem: On the Key Pairs List View "auth_key_pair.png" its display. However, on the Summary Screen "authentication.png" is displayed in the quad icon.


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


How reproducible:


Steps to Reproduce:
1. On the Key Pairs List View "auth_key_pair.png" its display. However, on the Summary Screen "authentication.png" is displayed in the quad icon.
2.
3.

Actual results:


Expected results:


Additional info: Github issue https://github.com/ManageIQ/manageiq/issues/8987
Comment 3 Satoe Imaishi 2016-08-23 13:49:02 UTC 
PR: https://github.com/ManageIQ/manageiq/pull/9062
Comment 4 Matouš Mojžíš 2016-09-30 14:17:27 UTC 
I can still see authentication.png on the summary screen in 5.7.0.3.
Comment 5 CFME Bot 2016-10-12 16:21:11 UTC 
https://github.com/ManageIQ/manageiq/pull/11881
Comment 8 CFME Bot 2016-10-17 16:21:30 UTC 
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/7e89b5a2f83f6e6d84b0ba9450426703245612c2

commit 7e89b5a2f83f6e6d84b0ba9450426703245612c2
Author:     Martin Povolny <mpovolny>
AuthorDate: Thu Oct 13 11:56:07 2016 +0200
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Thu Oct 13 17:16:52 2016 -0400

    Merge pull request #11881 from h-kataria/added_missing_key_pair_decorators
    
    Added missing key_pair decorators.
    (cherry picked from commit 25991b7f21c0f51b0375ccf64aa16eeeb83a7177)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1352914

 .../google/cloud_manager/auth_key_pair_decorator.rb    | 18 ++++++++++++++++++
 .../openstack/cloud_manager/auth_key_pair_decorator.rb | 18 ++++++++++++++++++
 .../openstack/infra_manager/auth_key_pair_decorator.rb | 18 ++++++++++++++++++
 3 files changed, 54 insertions(+)
 create mode 100644 app/decorators/manageiq/providers/google/cloud_manager/auth_key_pair_decorator.rb
 create mode 100644 app/decorators/manageiq/providers/openstack/cloud_manager/auth_key_pair_decorator.rb
 create mode 100644 app/decorators/manageiq/providers/openstack/infra_manager/auth_key_pair_decorator.rb
Comment 9 CFME Bot 2016-10-17 16:22:41 UTC 
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/029ada27316a01773c5a43a44aeba24ea015d910

commit 029ada27316a01773c5a43a44aeba24ea015d910
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed Oct 12 12:06:44 2016 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Wed Oct 12 12:08:41 2016 -0400

    Added missing key_pair decorators.
    
    This fixes quadicons on different types of key pair summary screens. This was partially fixed by https://github.com/ManageIQ/manageiq/pull/9062
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1352914

 .../google/cloud_manager/auth_key_pair_decorator.rb    | 18 ++++++++++++++++++
 .../openstack/cloud_manager/auth_key_pair_decorator.rb | 18 ++++++++++++++++++
 .../openstack/infra_manager/auth_key_pair_decorator.rb | 18 ++++++++++++++++++
 3 files changed, 54 insertions(+)
 create mode 100644 app/decorators/manageiq/providers/google/cloud_manager/auth_key_pair_decorator.rb
 create mode 100644 app/decorators/manageiq/providers/openstack/cloud_manager/auth_key_pair_decorator.rb
 create mode 100644 app/decorators/manageiq/providers/openstack/infra_manager/auth_key_pair_decorator.rb
Comment 11 Matouš Mojžíš 2017-03-14 21:22:24 UTC 
Verified in 5.8.0.4. "Two keys" icon is displayed in summary.