Bug 1361045

Summary: Unable to add configuration management provider - Add button not clickable
Product: Red Hat CloudForms Management Engine Reporter: Pavol Kotvan <pakotvan>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED CURRENTRELEASE QA Contact: Leo Khomenko <lkhomenk>
Severity: urgent Docs Contact:
Priority: high    
Version: 5.6.0CC: cpelland, hkataria, jhardy, jkrocil, lkhomenk, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.7.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ui:tower:foreman
Fixed In Version: 5.7.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1361308 (view as bug list) Environment:
Last Closed: 2017-01-11 19:55:30 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1349991, 1361308    
Attachments:
Description Flags
Ansible was added successfully none

Comment 5 CFME Bot 2016-07-28 20:56:07 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/7da0a0c70debaa1a8ff423fae858032b0b3f2be6

commit 7da0a0c70debaa1a8ff423fae858032b0b3f2be6
Author:     Harpreet Kataria <hkataria>
AuthorDate: Thu Jul 28 14:42:17 2016 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Thu Jul 28 15:13:05 2016 -0400

    Populate read only Zone text field when there is only 1 zone in region
    
    Fixed code path that deals with showing zone name as readonly text field when user has only 1 zone in their region, text field was not getting populated and was marked as required field which was causing Add button to remain disabled. Adjusted/fixed failing spec test.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1361045
    https://bugzilla.redhat.com/show_bug.cgi?id=1361308

 .../provider_foreman_form_controller.js            | 28 +++++----
 app/controllers/provider_foreman_controller.rb     |  5 ++
 .../provider_foreman_form_controller_spec.js       | 70 ++++++++++------------
 3 files changed, 52 insertions(+), 51 deletions(-)

Comment 11 Harpreet Kataria 2016-08-03 20:42:33 UTC
Pavol,

I am sending this BZ to POST as originally reported issue in this BZ has already been fixed. Support to enforce validation of credentials before adding a provider has not been added for Tower providers and middleware providers. Please open an RFE to address that in a separate BZ.

Thanks,
~Harpreet

Comment 12 Leo Khomenko 2016-09-16 08:04:38 UTC
Created attachment 1201555 [details]
Ansible was added successfully

Comment 13 Leo Khomenko 2016-09-16 08:15:24 UTC
It's possible to add provider and flash message on wrong credentials appears (https://bugzilla.redhat.com/show_bug.cgi?id=1368162) but it's possible to add with wrong creds - opening new BZ