Bug 1352914 - Key Pairs: wrong quadicon displayed
Summary: Key Pairs: wrong quadicon displayed
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.6.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.8.0
Assignee: Harpreet Kataria
QA Contact: Matouš Mojžíš
URL:
Whiteboard: ui:cloud
Depends On:
Blocks: 1353228 1385170 1385173
TreeView+ depends on / blocked
 
Reported: 2016-07-05 13:18 UTC by Harpreet Kataria
Modified: 2017-06-12 16:14 UTC (History)
8 users (show)

Fixed In Version: 5.8.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1353228 1385170 1385173 (view as bug list)
Environment:
Last Closed: 2017-06-12 16:14:30 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
key pairs list view (90.02 KB, image/png)
2016-07-05 13:18 UTC, Harpreet Kataria
no flags Details

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 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.


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