Bug 1388928

Summary:	Unable to add Configuration manager (Ansible, Satellite 6) "Endpoints.url has already been taken"
Product:	Red Hat CloudForms Management Engine	Reporter:	Josh Carter <jocarter>
Component:	Providers	Assignee:	Brandon Dunne <bdunne>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Pavol Kotvan <pakotvan>
Severity:	high	Docs Contact:
Priority:	high
Version:	5.7.0	CC:	cpelland, dajohnso, jfrey, jhardy, jkrocil, jocarter, kmorey, obarenbo, pakotvan, saali
Target Milestone:	GA	Keywords:	Reopened, TestOnly
Target Release:	5.8.0
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:	5.8.0.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1393987 (view as bug list)		Environment:
Last Closed:	2017-06-12 16:18:14 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:
Embargoed:
Bug Depends On:
Bug Blocks:	1387708, 1389789, 1390731, 1393987

Description Josh Carter 2016-10-26 13:35:31 UTC

Description of problem:

When adding an Ansible Tower provider after user validation when clicking add you are presented with a error "Endpoints.url has already been taken"

[----] E, [2016-10-26T09:27:51.486022 #13758:132d830] ERROR -- : MIQ(provider_foreman_controller-edit): Endpoints.url has already been taken

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

5.7.0.7-beta1.20161025153249_9376fbd 

This error does not happen on the alpha 3 build 

How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 4 Brandon Dunne 2016-10-27 16:09:30 UTC

It's a feature, not a bug... see https://bugzilla.redhat.com/show_bug.cgi?id=1382627

Comment 7 Dave Johnson 2016-10-31 18:19:33 UTC

*** Bug 1390133 has been marked as a duplicate of this bug. ***

Comment 8 CFME Bot 2016-11-04 20:36:43 UTC

https://github.com/ManageIQ/manageiq/pull/12449

Comment 9 Brandon Dunne 2016-11-04 21:08:27 UTC

*** Bug 1389457 has been marked as a duplicate of this bug. ***

Comment 10 CFME Bot 2016-11-09 00:06:15 UTC

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/1b0d107009d473a45c482547d0bca9e4bf4204f8

commit 1b0d107009d473a45c482547d0bca9e4bf4204f8
Author:     Brandon Dunne <bdunne>
AuthorDate: Tue Nov 8 17:26:32 2016 -0500
Commit:     Brandon Dunne <bdunne>
CommitDate: Tue Nov 8 17:35:08 2016 -0500

    Allow the :autosave => true update the endpoint
    
    Delegate verify_ssl= to the endpoint and leverage the relation to autosave
    the default_endpoint.  update_attributes was creating a disconnected
    endpoint record, so verify_credentials would create an endpoint then save
    would raise a validation error.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1388928

 app/models/provider.rb | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comment 11 CFME Bot 2016-11-09 22:51:03 UTC

New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/aca061e269b9eefa4176f42a1120a2bc97dfed82

commit aca061e269b9eefa4176f42a1120a2bc97dfed82
Author:     Greg McCullough <gmccullo>
AuthorDate: Tue Nov 8 19:05:46 2016 -0500
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Wed Nov 9 17:49:28 2016 -0500

    Merge pull request #12449 from bdunne/endpoint_url
    
    Let the relation autosave the verify_ssl value
    (cherry picked from commit 31921db11f5445908f8c9ab101d5ae8495d1c231)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1388928

 app/models/provider.rb | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comment 12 Jan Krocil 2016-11-10 14:04:58 UTC

This affects not only Ansible but also Satellite 6.

Comment 14 Brandon Dunne 2016-11-21 18:00:31 UTC

*** Bug 1396764 has been marked as a duplicate of this bug. ***