Bug 773768

Summary: adding rhevm provider needs to validate rhevm data_center id
Product: [Retired] CloudForms Cloud Engine Reporter: Dave Johnson <dajohnso>
Component: aeolus-conductorAssignee: Jan Provaznik <jprovazn>
Status: CLOSED EOL QA Contact: Rehana <aeolus-qa-list>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, athomas, cpelland, deltacloud-maint, hbrock, mfojtik, ssachdev, tsedovic
Target Milestone: rcKeywords: Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-01-09 07:54:02 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
ss none

Description Dave Johnson 2012-01-12 21:05:09 UTC
Created attachment 552503 [details]
ss

Description of problem:
================================
Add a rhevm provider manually though the web ui, the user now needs to append the datacenter id within the X-Deltacloud-provider field.  At the very least, that id should be verified as a valid data center object on the rhevm server you are hitting.

Comment 1 Angus Thomas 2012-01-13 14:28:35 UTC
Jan,

Please check whether DC will allow this verification. If not, it will need to be implemented after 1.0

Comment 2 Michal Fojtik 2012-01-19 11:13:51 UTC
DC indeed allows this verification. The list of all data centers is available here:

/api/drivers/rhevm

What I'm not entirely sure is, if Ruby client is able to fetch this informations.

Comment 3 Jan Provaznik 2012-01-25 10:33:26 UTC
Michal is going to add "valid_provider?" method into dc-client lib, so waiting on implementation of this feature.

Comment 4 Michal Fojtik 2012-01-25 11:41:05 UTC
This feature is already in master:

http://mail-archives.apache.org/mod_mbox/deltacloud-dev/201201.mbox/%3C1327404674-85464-5-git-send-email-mfojtik%40redhat.com%3E

Now it's waiting for being packaged.

Comment 5 Hugh Brock 2012-02-08 16:23:24 UTC
Michal, did the fix above make it into Deltacloud-0.5.0? If so, Jan, can we go ahead and implement based on that?

Comment 6 Michal Fojtik 2012-02-08 17:09:42 UTC
Yes, this fix is included in latest DC brew build (0.5.0-5)

Comment 7 Jan Provaznik 2012-02-09 11:03:23 UTC
Could you send me a link where I can get it? In brew I see only el6 builds (0.5.0-2 newest)

Comment 8 Michal Fojtik 2012-02-10 09:39:04 UTC
https://brewweb.devel.redhat.com/buildinfo?buildID=197651

Should be available by usual channel now.

Comment 9 Michal Fojtik 2012-02-10 09:41:25 UTC
Sorry, the client you mean. Yes, the latest in brew is 0.5.0-2 and it should have the methods mentioned in Comment #4 :-)

Comment 10 Jan Provaznik 2012-08-23 10:38:26 UTC
ATM valid_provider? method doesn't work properly, we need a fix on dc-core side at first: https://issues.apache.org/jira/browse/DTACLOUD-307

Because of above, I doubt this BZ could be fixed in 1.1 release.

Comment 11 Angus Thomas 2012-08-30 12:53:37 UTC
This is being descoped for 1.1 because it depends on deltacloud work which is being tracked here: https://issues.apache.org/jira/browse/DTACLOUD-307