Bug 1310555

Summary:	docker compute resource pre-fills and overrides username when empty value is stored
Product:	Red Hat Satellite	Reporter:	Pavel Moravec <pmoravec>
Component:	Container Management	Assignee:	Daniel Lobato Garcia <dlobatog>
Status:	CLOSED WONTFIX	QA Contact:	Katello QA List <katello-qa-list>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	6.1.6	CC:	tomckay
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-08-04 13:19:08 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:

Description Pavel Moravec 2016-02-22 08:40:38 UTC

Description of problem:
When editing a docker compute resource, if it has emtpy credentials (username/password), WebUI automatically - everytime - fills admin/****** there.

Please get rid of the automatic pre-filling of credentials when editing a docker compute resource. It tries to override what is really stored, confuses a user and attempts to store data a user does not want.


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

How reproducible:
100%


Steps to Reproduce:
1. Create a docker compute resource with empty credentials. Test connection it is OK.
2. Edit the docker compute resource.
3. Test Connection.
4. Save it.


Actual results:
2. shows admin/****** credentials, despite _emtpy_ were inserted.
3. Test fails with Docker::Error::AuthenticationError - despite it was working before.
4. The username is really saved (you can check this by running:

su - postgres -c "psql foreman -c \"select * from compute_resources;\""

before and after the save)


Expected results:
2. to show whatever I stored, not overriding anything
3. test connection to pass
4. not storing the username


Additional info:

Comment 2 Pavel Moravec 2016-02-22 08:53:57 UTC

Yet another scenario:

5. when editing the compute resource, remove "admin" from the username and click to Test connection.
6. check "admin" has been filled there again 

That means, a user clicking to Test connection (that passes) and then to Submit, "admin" username is stored instead of empty one. Such confusing.

Comment 3 Bryan Kearney 2016-07-26 15:25:24 UTC

Moving 6.2 bugs out to sat-backlog.

Comment 4 Bryan Kearney 2016-07-26 15:35:31 UTC

Moving 6.2 bugs out to sat-backlog.

Comment 6 Bryan Kearney 2017-08-04 13:19:08 UTC

Thank you for your interest in Satellite 6. We have evaluated this request, and we do not expect this to be implemented in product in the forseeable future. We are therefore closing this out as WONTFIX. If you have any concerns about this, please feel free to contact Rich Jerrido or Bryan Kearney. Thank you.