Bug 1288496 - When using an SSL enabled Openstack Infra provider the toolbar doesn't switch Hosts to Nodes
Summary: When using an SSL enabled Openstack Infra provider the toolbar doesn't switch...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.5.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.6.0
Assignee: Harpreet Kataria
QA Contact: Ola Pavlenko
URL:
Whiteboard: ui:explorer
Depends On:
Blocks: 1290554
TreeView+ depends on / blocked
 
Reported: 2015-12-04 12:08 UTC by Marius Cornea
Modified: 2017-08-30 03:49 UTC (History)
8 users (show)

Fixed In Version: 5.6.0.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1290554 (view as bug list)
Environment:
Last Closed: 2016-06-29 15:17:40 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
host-nodes.png (96.58 KB, image/png)
2015-12-04 12:08 UTC, Marius Cornea
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2016:1348 0 normal SHIPPED_LIVE CFME 5.6.0 bug fixes and enhancement update 2016-06-29 18:50:04 UTC

Description Marius Cornea 2015-12-04 12:08:24 UTC
Created attachment 1102246 [details]
host-nodes.png

Description of problem:
When using an SSL enabled Openstack Infra provider the toolbar doesn't switch Hosts to Nodes

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

How reproducible:
100%

Steps to Reproduce:
1. Add an SSL enabled Openstack Infra provider 
2. Refresh and wait for the nodes to show up


Actual results:
The toolbar and Infrastructure menu still show Hosts

Expected results:
I was expecting that Hosts would turn to nodes when having a single Openstack Infra provider.

Additional info:
Attaching screenshot.

Comment 4 Harpreet Kataria 2015-12-17 19:03:45 UTC
https://github.com/ManageIQ/manageiq/pull/5891

Comment 5 CFME Bot 2015-12-18 00:24:20 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/aadec88f7e406e8b43de9c31d6a72b79b60e4492

commit aadec88f7e406e8b43de9c31d6a72b79b60e4492
Author:     Harpreet Kataria <hkataria>
AuthorDate: Thu Dec 17 14:02:44 2015 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Thu Dec 17 17:59:23 2015 -0500

    Fixed model name passed in to hybrid_name method.
    
    - Incorrect model name was causing incorrect label to be displayed on the secondary navigation bar. To recreate this issue db should only contain openstack Hosts, and no openstack Clusters.
    - Added spec test to verify labels for Hosts & Clusters in different modes depending upon existence of openstack/non-openstack records in the db
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1288496
    https://bugzilla.redhat.com/show_bug.cgi?id=1290554

 app/presenters/menu/default_menu.rb       |  4 +-
 spec/presenters/menu/default_menu_spec.rb | 61 +++++++++++++++++++++++++++++++
 2 files changed, 63 insertions(+), 2 deletions(-)

Comment 6 CFME Bot 2015-12-18 20:37:50 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=c77451d21dc14dbbf864c0ef99ee017c60f5a11f

commit c77451d21dc14dbbf864c0ef99ee017c60f5a11f
Merge: 72e6f8d 21f80d3
Author:     Dan Clarizio <dclarizi>
AuthorDate: Fri Dec 18 15:35:32 2015 -0500
Commit:     Dan Clarizio <dclarizi>
CommitDate: Fri Dec 18 15:35:32 2015 -0500

    Merge branch '55z_pr_5891' into '5.5.z'
    
    Fixed model name passed in to hybrid_name method.
    
    - Incorrect model name was causing incorrect label to be displayed on the secondary navigation bar. To recreate this issue db should only contain openstack Hosts, and no openstack Clusters.
    - Added spec test to verify labels for Hosts & Clusters in different modes depending upon existence of openstack/non-openstack records in the db
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1288496
    https://bugzilla.redhat.com/show_bug.cgi?id=1290554
    (cherry picked from commit aadec88)
    
    @dclarizi this is a clean cherry-pick for https://github.com/ManageIQ/manageiq/pull/5891
    
    See merge request !640

 app/presenters/menu/default_menu.rb       |  4 +-
 spec/presenters/menu/default_menu_spec.rb | 61 +++++++++++++++++++++++++++++++
 2 files changed, 63 insertions(+), 2 deletions(-)

Comment 7 CFME Bot 2015-12-18 20:37:59 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=21f80d3b9a98b08129351d596928424171d1c7e8

commit 21f80d3b9a98b08129351d596928424171d1c7e8
Author:     Harpreet Kataria <hkataria>
AuthorDate: Thu Dec 17 14:02:44 2015 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri Dec 18 13:55:22 2015 -0500

    Fixed model name passed in to hybrid_name method.
    
    - Incorrect model name was causing incorrect label to be displayed on the secondary navigation bar. To recreate this issue db should only contain openstack Hosts, and no openstack Clusters.
    - Added spec test to verify labels for Hosts & Clusters in different modes depending upon existence of openstack/non-openstack records in the db
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1288496
    https://bugzilla.redhat.com/show_bug.cgi?id=1290554
    (cherry picked from commit aadec88)

 app/presenters/menu/default_menu.rb       |  4 +-
 spec/presenters/menu/default_menu_spec.rb | 61 +++++++++++++++++++++++++++++++
 2 files changed, 63 insertions(+), 2 deletions(-)

Comment 9 Dave Johnson 2016-02-23 15:20:56 UTC
No, this needs to stay open so we can make certain the same patch made it onto the CF 4.1 branch.  We did test this and it was verified working on 5.5 with its clone, bug 1290554 .

Comment 10 Ronnie Rasouli 2016-06-15 05:45:59 UTC
Adding some tech notes on how to reproduce it:
1. Need to deploy undercloud SSL based.
2. Add the undercloud as provider infra still hosts
3. deploy openstack overcloud - hosts switch to nodes
verified on 5.6.0.10 RC2.1

Comment 12 errata-xmlrpc 2016-06-29 15:17:40 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://access.redhat.com/errata/RHBA-2016:1348


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