Bug 772996 - disable "launch" button if no images are built/pushed
Summary: disable "launch" button if no images are built/pushed
Keywords:
Status: CLOSED ERRATA
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: Scott Seago
QA Contact: wes hayutin
URL: https://brewweb.devel.redhat.com/buil...
Whiteboard:
: 770091 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-10 15:55 UTC by wes hayutin
Modified: 2012-05-15 21:33 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-05-15 21:33:42 UTC


Attachments (Terms of Use)
ss1 (109.37 KB, image/png)
2012-01-10 15:56 UTC, wes hayutin
no flags Details
ss2 (103.64 KB, image/png)
2012-01-10 15:56 UTC, wes hayutin
no flags Details
ss3 (104.57 KB, image/png)
2012-01-10 15:57 UTC, wes hayutin
no flags Details
ss4 (89.69 KB, image/png)
2012-01-10 15:57 UTC, wes hayutin
no flags Details
disabled_launch_btn (222.84 KB, image/png)
2012-01-19 09:08 UTC, Shveta
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2012:0583 0 normal SHIPPED_LIVE new packages: aeolus-conductor 2012-05-15 22:31:59 UTC

Description wes hayutin 2012-01-10 15:55:58 UTC
Description of problem:

1. disable "launch" button if no images are built/pushed
2. images ready icon should change to something to indicate "images are NOT ready"


recreate:
1. upload a template
2. generate a deployable
3. add deployable to catalog

see screenshots

Comment 1 wes hayutin 2012-01-10 15:56:32 UTC
Created attachment 551875 [details]
ss1

Comment 2 wes hayutin 2012-01-10 15:56:52 UTC
Created attachment 551876 [details]
ss2

Comment 3 wes hayutin 2012-01-10 15:57:17 UTC
Created attachment 551877 [details]
ss3

Comment 4 wes hayutin 2012-01-10 15:57:54 UTC
Created attachment 551878 [details]
ss4

Comment 5 Scott Seago 2012-01-10 20:51:18 UTC
Actually, 'images ready' is a misnomer, what that really means is 'images valid' -- i.e. the referenced image exists in IWHD, if images aren't "ready" in this sense, the user has to edit the deployable XML to provide a valid image UUID.

So we should really change 'images ready' to 'images  valid' -- i.e. if images aren't ready/valid, we can't even build -- we have to fix the deployable. Once images are valid, we can build/push.

So I'd change 2) in the above problem description to show this as 'images are valid' -- since that's the output of Deployable XML validation rather than image build status -- image build status (i.e. ready vs. not) is in the per-provider-account status fields.

The need to disable the 'launch' button is, of course, still a valid problem.

Comment 6 wes hayutin 2012-01-12 16:34:09 UTC
adding to ce-sprint

Comment 7 wes hayutin 2012-01-12 16:40:29 UTC
removing ce-sprint-next tracker

Comment 8 Scott Seago 2012-01-13 01:18:56 UTC
Patch here:
https://fedorahosted.org/pipermail/aeolus-devel/2012-January/008065.html

'images ready' is now 'images valid', and launch button is disabled unless all
images are pushed to at least one provider account.

Comment 9 wes hayutin 2012-01-16 20:29:45 UTC
*** Bug 770091 has been marked as a duplicate of this bug. ***

Comment 10 Steve Linabery 2012-01-18 20:38:10 UTC
d24dcbe780e127a7054cc10187df0b0f3e93198a in aeolus-conductor-0.8.0-8.el6.src.rpm

Comment 11 Shveta 2012-01-19 09:08:17 UTC
Created attachment 556219 [details]
disabled_launch_btn

Comment 12 Shveta 2012-01-19 09:09:54 UTC
"Image Ready" is replaced by "Image Valid"
Launch button is disabled if images are not built and pushed.


Verified in
rpm -qa|grep aeolus
aeolus-conductor-0.8.0-8.el6.noarch
aeolus-conductor-daemons-0.8.0-8.el6.noarch
rubygem-aeolus-image-0.3.0-3.el6.noarch
rubygem-aeolus-cli-0.3.0-4.el6.noarch
aeolus-all-0.8.0-8.el6.noarch
aeolus-configure-2.5.0-5.el6.noarch
aeolus-conductor-doc-0.8.0-8.el6.noarch

Comment 14 errata-xmlrpc 2012-05-15 21:33:42 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2012-0583.html


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