Bug 770574 - Build should be allowed only for that provider for which deployable is created .
Summary: Build should be allowed only for that provider for which deployable is created .
Keywords:
Status: CLOSED DUPLICATE of bug 783133
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-conductor
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
Assignee: Matt Wagner
QA Contact: wes hayutin
URL:
Whiteboard:
Depends On:
Blocks: ce-sprint
TreeView+ depends on / blocked
 
Reported: 2011-12-27 14:41 UTC by Shveta
Modified: 2012-01-25 14:58 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-01-25 14:58:30 UTC


Attachments (Terms of Use)
err (223.60 KB, image/png)
2011-12-27 14:41 UTC, Shveta
no flags Details
server_error (183.38 KB, image/png)
2011-12-27 14:42 UTC, Shveta
no flags Details

Description Shveta 2011-12-27 14:41:48 UTC
Created attachment 549703 [details]
err

Description of problem:


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


How reproducible:


Steps to Reproduce:
1. Environment --> Images --> Import Image 
2. Imported an image for ec2 provider.
3. Created a deployable (ec2_deployable) from the imported image named 
4. Catalog --> click on ec2 deployable (catalog_entry) and click 
on build (start) 

Message displayed : "Failed. Response code = 500. Response message = Internal Server Error. "


  
Actual results:


Expected results:


Additional info:

rpm -qa|grep aeolus
aeolus-conductor-0.8.0-0.20111222233342gitd98cb57.el6.noarch
aeolus-all-0.8.0-0.20111222233342gitd98cb57.el6.noarch
aeolus-conductor-daemons-0.8.0-0.20111222233342gitd98cb57.el6.noarch
rubygem-aeolus-image-0.3.0-0.20111222173411gitc13b654.el6.noarch
rubygem-aeolus-cli-0.3.0-0.20111222173356git3cd6277.el6.noarch
aeolus-configure-2.5.0-0.20111222173430git17b704a.el6.noarch
aeolus-conductor-doc-0.8.0-0.20111222233342gitd98cb57.el6.noarch

Comment 1 Shveta 2011-12-27 14:42:22 UTC
Created attachment 549704 [details]
server_error

Comment 2 wes hayutin 2012-01-10 17:11:10 UTC
adding to ce-sprint-next

Comment 3 wes hayutin 2012-01-10 17:14:17 UTC
adding to ce-sprint-next

Comment 4 wes hayutin 2012-01-12 16:35:59 UTC
adding to ce-sprint

Comment 5 wes hayutin 2012-01-12 16:42:11 UTC
removing ce-sprint-next tracker

Comment 6 wes hayutin 2012-01-12 16:44:17 UTC
taking off ce-sprint-next..

Comment 7 Matt Wagner 2012-01-13 15:30:45 UTC
Jan points out that, since a deployable will be launched on only one provider, if an image is imported, that's the only provider anything else can sensibly be built for.

It sounds like the following is in order:

1.) If all images are imported, hide the build/push buttons, as they're inapplicable. Only show the provider in question.

2.) If *some* images are imported:
 a.) The build_missing method should silently skip over imported images, rather than raising an exception.
 b.) The build section should only build for the provider of the imported image

Comment 8 Matt Wagner 2012-01-13 15:40:59 UTC
Scott points out that we may (post-1.0) eventually need to support imported images for multiple provider accounts here. So I think the most-correct thing to do here is to fetch an array of "all" provider accounts supported by all imported images. (In the event that they don't all share one or more provider accounts, an exception should be raised, since you'll never be able to launch the thing successfully.)

Comment 9 Matt Wagner 2012-01-18 22:06:01 UTC
After an inordinately long time, this is on the list: http://lists.fedorahosted.org/pipermail/aeolus-devel/2012-January/008190.html and children

Comment 10 Matt Wagner 2012-01-25 14:58:30 UTC
Unfortunately, as Jan points out on list, this is now rendered obsolete by  https://bugzilla.redhat.com/show_bug.cgi?id=783133 -- see http://lists.fedorahosted.org/pipermail/aeolus-devel/2012-January/008243.html

Closing as a dupe.

*** This bug has been marked as a duplicate of bug 783133 ***


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