Bug 773768 - adding rhevm provider needs to validate rhevm data_center id
Summary: adding rhevm provider needs to validate rhevm data_center id
Keywords:
Status: CLOSED EOL
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-conductor
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: rc
Assignee: Jan Provaznik
QA Contact: Rehana
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-12 21:05 UTC by Dave Johnson
Modified: 2017-01-09 07:54 UTC (History)
8 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-01-09 07:54:02 UTC
Embargoed:


Attachments (Terms of Use)
ss (81.21 KB, image/png)
2012-01-12 21:05 UTC, Dave Johnson
no flags Details

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


Note You need to log in before you can comment on or make changes to this bug.