Bug 683874

Summary:	Provider Accounts: "test" connection w/o public and private pem files succeeds when it should not
Product:	[Retired] CloudForms Cloud Engine	Reporter:	wes hayutin <whayutin>
Component:	aeolus-conductor	Assignee:	Richard Su <rwsu>
Status:	CLOSED CURRENTRELEASE	QA Contact:	wes hayutin <whayutin>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	0.3.1	CC:	akarol, cpelland, dajohnso, deltacloud-maint, dgao, morazi, ssachdev
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
URL:	http://nec-em15.rhts.eng.bos.redhat.com:3000/conductor/admin/provider_accounts
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2011-03-22 19:38:16 UTC	Type:	---
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:	669238, 684278

Description wes hayutin 2011-03-10 14:55:37 UTC

Recreate:
1. create a valid provider
2. enter in provider account details that are valid w/o a public/private x509 key
3. click test

test succeeds.  

Is this by design?  I would assume that the test should include all the required fields.  If the test to ec2 works w/o the x509 certs, we should at least require the field to be filled out for test to succeed.

Comment 1 wes hayutin 2011-03-11 16:58:38 UTC

adding to the ce-beta blocking tracker

Comment 2 wes hayutin 2011-03-16 15:16:57 UTC

aeolus-conductor-debuginfo-0.0.3-0.fc1420110311212550gitab7a7ba.x86_64
aeolus-conductor-doc-0.0.3-0.fc1420110311212550gitab7a7ba.x86_64
aeolus-conductor-daemons-0.0.3-0.fc1420110311212550gitab7a7ba.x86_64
aeolus-conductor-0.0.3-0.fc1420110311212550gitab7a7ba.x86_64
aeolus-configure-2.0.0-4.fc1420110311170652gite6ac5c7.noarch

Comment 3 wes hayutin 2011-03-22 19:38:16 UTC

It seems I thought this worked ok whether the test worked or not w/ the keys. w/o a clear design this boils down to opinion.

closing as not a bug

Comment 4 wes hayutin 2011-12-08 13:46:35 UTC

perm close