Bug 782097 - Aeolus::Image::Factory::Builder.find_active_build doesn't respect target
Summary: Aeolus::Image::Factory::Builder.find_active_build doesn't respect target
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: rubygem-aeolus-image
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
Assignee: Jan Provaznik
QA Contact: wes hayutin
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-16 14:13 UTC by Jan Provaznik
Modified: 2012-08-30 17:12 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-30 17:12:36 UTC

Attachments (Terms of Use)
image show page (179.42 KB, image/png)
2012-01-17 12:30 UTC, Jan Provaznik 		no flags Details
build_status (230.74 KB, image/png)
2012-02-07 17:25 UTC, Shveta 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Description Jan Provaznik 2012-01-16 14:13:59 UTC 
When checking if an image is being built in factory for specified target type (for example 'ec2'), current builds are not filtered by this target type.
Comment 1 wes hayutin 2012-01-16 15:24:00 UTC 
more detail about the bug would be helpful here..
Thanks!
Comment 2 Jan Provaznik 2012-01-17 12:30:10 UTC 
Created attachment 555748 [details]
image show page
Comment 3 Jan Provaznik 2012-01-17 12:47:22 UTC 
If there are multiple provider types and image building for one of the provider types takes more time, all builds on the image show page will be marked as "pending" (including the ones which are already built).

For most provders image build process is quick/immediate, so it could occur only for provider types where building takes more time (probably rhevm).

(On the screenshot above are used mock and ec2 providers, but I updated imagefactory's mock builder for this test to use 60s delay - it will not work for you on these 2 provider types)
Comment 4 Jan Provaznik 2012-01-17 14:41:23 UTC 
patch is on the list: https://fedorahosted.org/pipermail/aeolus-devel/2012-January/008133.html
Comment 5 Jan Provaznik 2012-01-18 15:25:27 UTC 
pushed, commit 0d31a376173163a0b7d2a16ca2b035e2714510e7
Comment 6 Steve Linabery 2012-01-18 21:10:03 UTC 
rubygem-aeolus-image-0.3.0-3.el6.src.rpm
62d8e10c1391e4cf4da52dbc61aae04cfa72ae34
Comment 7 Shveta 2012-02-07 17:25:02 UTC 
Created attachment 560017 [details]
build_status
Comment 8 Shveta 2012-02-07 17:26:10 UTC 
Individual different build status are shown .

Verified in
rpm -qa|grep aeolus
aeolus-conductor-daemons-0.8.0-21.el6.noarch
aeolus-configure-2.5.0-12.el6.noarch
aeolus-conductor-doc-0.8.0-21.el6.noarch
rubygem-aeolus-image-0.3.0-7.el6.noarch
rubygem-aeolus-cli-0.3.0-8.el6.noarch
aeolus-conductor-0.8.0-21.el6.noarch
aeolus-all-0.8.0-21.el6.noarch
Note You need to log in before you can comment on or make changes to this bug.