Bug 1326499

Summary: Trusted Forest Settings table disappears when switched between authentication modes.
Product: Red Hat CloudForms Management Engine Reporter: amogh <amavinag>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED ERRATA QA Contact: amogh <amavinag>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.5.0CC: cpelland, dajohnso, hkataria, jhardy, jprause, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: ZStream
Target Release: 5.6.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ldap
Fixed In Version: 5.6.0.7 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1335876 (view as bug list) Environment:
Last Closed: 2016-06-29 15:48:48 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1335876    

Description amogh 2016-04-12 20:05:36 UTC
Description of problem:
Trusted Forest Settings table disappears when switched between authentication modes.

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

How reproducible:
always.

Steps to Reproduce:
1. Login as "admin", navigate to configuration->configure->authentication
2. change the authentication mode to "LDAP", specify all the details for LDAP settings.
3. Check "Get User Groups from LDAP", specify all the details.
4. specify the details for "Trusted Forest Settings", click "add" trusted forest  and click save.
5. change the authentication mode to other than LDAP/LDAPS (e.g. Amazon), then switch back to LDAP. observe that "Trusted Forest Settings" table disappeared. 

However, after manual refresh or (uncheck and check) the option "Get User Groups from LDAP" the "Trusted Forest Settings" is displayed.

Actual results:
Trusted Forest Settings table disappears when switched between authentication mode.

Expected results:
the data table "Trusted Forest Settings" is not expected to disappear when switched between authentication modes.

Comment 3 CFME Bot 2016-05-12 18:51:02 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/4e403662a8b664caab5a36496d316311688327ab

commit 4e403662a8b664caab5a36496d316311688327ab
Author:     Harpreet Kataria <hkataria>
AuthorDate: Thu May 12 12:18:43 2016 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Thu May 12 12:40:14 2016 -0400

    Display Forest Entries box when switching mode back to LDAP/LDAPS
    
    - Fixed code to show Forest Entries box when switching back to LDAP/LDAPS mode
    - Added spec test to verify fix
    - Fixed alignment of icons in Forest Entries box
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1326499

 app/controllers/ops_controller/settings/common.rb  |  2 ++
 app/views/ops/_ldap_forest_entries.html.haml       |  4 +--
 .../ops_controller/settings/common_spec.rb         | 41 ++++++++++++++++++++++
 3 files changed, 44 insertions(+), 3 deletions(-)

Comment 6 amogh 2016-05-17 20:44:57 UTC
looks good. verified in 5.6.0.7-beta2.6.20160516171555_b1be31f

Comment 7 CFME Bot 2016-06-13 18:39:21 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=313d8c2ae35a9dbc4f1920a2e486a734d0e44430

commit 313d8c2ae35a9dbc4f1920a2e486a734d0e44430
Author:     Harpreet Kataria <hkataria>
AuthorDate: Thu May 12 12:18:43 2016 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri May 13 09:10:04 2016 -0400

    Display Forest Entries box when switching mode back to LDAP/LDAPS
    
    - Fixed code to show Forest Entries box when switching back to LDAP/LDAPS mode
    - Added spec test to verify fix
    - Fixed alignment of icons in Forest Entries box
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1326499
    https://bugzilla.redhat.com/show_bug.cgi?id=1335876
    (cherry picked from commit 4e40366)

 app/controllers/ops_controller/settings/common.rb  |  2 ++
 app/views/ops/_ldap_forest_entries.html.haml       |  4 +--
 .../ops_controller/settings/common_spec.rb         | 41 ++++++++++++++++++++++
 3 files changed, 44 insertions(+), 3 deletions(-)

Comment 8 CFME Bot 2016-06-13 18:39:33 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=a2ec6c596b5ff1b2ccd6ca51b42a92c15f2b9d7f

commit a2ec6c596b5ff1b2ccd6ca51b42a92c15f2b9d7f
Merge: 3e03d5c 313d8c2
Author:     Dan Clarizio <dclarizi>
AuthorDate: Mon Jun 13 13:30:46 2016 -0400
Commit:     Dan Clarizio <dclarizi>
CommitDate: Mon Jun 13 13:30:46 2016 -0400

    Merge branch '55z_pr_8658' into '5.5.z'
    
    Display Forest Entries box when switching mode back to LDAP/LDAPS
    
    - Fixed code to show Forest Entries box when switching back to LDAP/LDAPS mode
    - Added spec test to verify fix
    - Fixed alignment of icons in Forest Entries box
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1326499
    https://bugzilla.redhat.com/show_bug.cgi?id=1335876
    (cherry picked from commit 4e40366)
    
    @dclarizi please review, this is clean cherry-pick for https://github.com/ManageIQ/manageiq/pull/8658
    
    See merge request !936

 app/controllers/ops_controller/settings/common.rb  |  2 ++
 app/views/ops/_ldap_forest_entries.html.haml       |  4 +--
 .../ops_controller/settings/common_spec.rb         | 41 ++++++++++++++++++++++
 3 files changed, 44 insertions(+), 3 deletions(-)

Comment 10 errata-xmlrpc 2016-06-29 15:48:48 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