Bug 1389278

Summary: Container Providers details remain empty even after validation succeeds
Product: Red Hat CloudForms Management Engine Reporter: Einat Pacifici <epacific>
Component: UI - OPSAssignee: Yaacov Zamir <yzamir>
Status: CLOSED CURRENTRELEASE QA Contact: Einat Pacifici <epacific>
Severity: high Docs Contact:
Priority: high    
Version: 5.7.0CC: cpelland, hkataria, jhardy, jkrocil, mpovolny, obarenbo
Target Milestone: GAKeywords: TestOnly
Target Release: 5.8.0Flags: epacific: automate_bug+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: container
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1395856 (view as bug list) Environment:
Last Closed: 2017-06-12 17:02:34 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Container Management Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1395856    
Attachments:
Description Flags
evm.log attached. none

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