Bug 1389278 - Container Providers details remain empty even after validation succeeds
Summary: Container Providers details remain empty even after validation succeeds
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.8.0
Assignee: Yaacov Zamir
QA Contact: Einat Pacifici
URL:
Whiteboard: container
: 1390145 (view as bug list)
Depends On:
Blocks: 1395856
TreeView+ depends on / blocked
 
Reported: 2016-10-27 10:55 UTC by Einat Pacifici
Modified: 2018-04-17 08:23 UTC (History)
6 users (show)

Fixed In Version: 5.8.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1395856 (view as bug list)
Environment:
Last Closed: 2017-06-12 17:02:34 UTC
Category: ---
Cloudforms Team: Container Management
Target Upstream Version:
Embargoed:
epacific: automate_bug+


Attachments (Terms of Use)
evm.log attached. (900.58 KB, application/x-gzip)
2016-10-27 10:55 UTC, Einat Pacifici
no flags Details

Description Einat Pacifici 2016-10-27 10:55:27 UTC
Created attachment 1214573 [details]
evm.log attached.

Description of problem:
Add a new container provider in CFME and pass validation successfully. Expecting CFME to connect to provider, retrieve details and show in Providers summary page. 
Actual result: No details are available and all entity values remain = 0 


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

How reproducible:
Always

Steps to Reproduce:
1.Add a container provider (openshift) to CFME
2.Validate Token
3.Save new provider
4.Wait 1-2 minutes 
5. Go to providers summary page

Actual results:
1. All provider entities show as 0
2. Bearer Authentication show as None

Expected results:
1. All provider entities should show actual values. 
2. Bearer Authentication should show: "Valid x minutes ago"

Additional info:
Workaround is available - 
1. After entering the provider details+validating token+saving --> In providers summary screen select: Authentication --> Recheck Authentication status
2. Return to edit provider details - Press Validate again and Save
3. Wait approx. 1-2 minutes --> Provider summary details should now be available. 
4. If still unavailable - redo steps 1-3.

Comment 4 CFME Bot 2016-11-01 19:06:10 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/062f09401268ee52919091914c361a484c75294f

commit 062f09401268ee52919091914c361a484c75294f
Author:     Oleg Barenboim <chessbyte>
AuthorDate: Tue Nov 1 15:03:11 2016 -0400
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Tue Nov 1 15:03:44 2016 -0400

    Merge pull request #11504 from yaacov/remove-extra-default-authentication-for-ems-containers
    
    Remove the extra "default" authentication created when editing a container provider.
    (cherry picked from commit bfe87e83aaaa2c5318f524a788a8fcab4a946424)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1361012
    https://bugzilla.redhat.com/show_bug.cgi?id=1389278

 app/controllers/mixins/ems_common_angular.rb        | 21 +++++++++++----------
 .../providers/kubernetes/container_manager_mixin.rb |  2 +-
 app/models/mixins/authentication_mixin.rb           |  6 +++---
 spec/controllers/ems_common_controller_spec.rb      |  6 ++++++
 4 files changed, 21 insertions(+), 14 deletions(-)

Comment 5 CFME Bot 2016-11-01 19:06:20 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/a1147c93451a834c6476151f7a30e82297ee3f5b

commit a1147c93451a834c6476151f7a30e82297ee3f5b
Author:     yaacov <kobi.zamir>
AuthorDate: Sun Sep 25 15:38:27 2016 +0300
Commit:     yaacov <kobi.zamir>
CommitDate: Tue Nov 1 11:21:16 2016 +0200

    Reove the extra "default" authentication created when editing a container provider.
    
    PR #9947 solved the removal of the extra "default" authentications only in cases where the
    params[:default_password] was not blank (this is not always true), this PR fixes that,
    and remove the extra authenication in all the cases the where the external manegment system
    is container manager.
    
    Bugzilla:
    https://bugzilla.redhat.com/show_bug.cgi?id=1370576
    https://bugzilla.redhat.com/show_bug.cgi?id=1389278

 app/controllers/mixins/ems_common_angular.rb        | 21 +++++++++++----------
 .../providers/kubernetes/container_manager_mixin.rb |  2 +-
 app/models/mixins/authentication_mixin.rb           |  6 +++---
 spec/controllers/ems_common_controller_spec.rb      |  6 ++++++
 4 files changed, 21 insertions(+), 14 deletions(-)

Comment 6 Yaacov Zamir 2016-11-02 05:49:32 UTC
merged
https://github.com/ManageIQ/manageiq/pull/11504

Comment 7 Yaacov Zamir 2016-11-09 15:13:31 UTC
*** Bug 1390145 has been marked as a duplicate of this bug. ***

Comment 9 Tony 2017-03-05 10:30:21 UTC
Verified on 5.8.0.3


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