Bug 773402 - Grey out launch button if deployment name is > 50 chars
Summary: Grey out launch button if deployment name is > 50 chars
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: Tzu-Mainn Chen
QA Contact: wes hayutin
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-11 18:36 UTC by wes hayutin
Modified: 2012-05-15 21:39 UTC (History)
4 users (show)

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

Attachments (Terms of Use)
ss (93.52 KB, image/png)
2012-01-11 18:36 UTC, wes hayutin 		no flags Details
ss (242.16 KB, image/png)
2012-01-25 08:34 UTC, Aziza Karol 		no flags Details
View All


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-11 18:36:43 UTC 
Created attachment 552188 [details]
ss

Description of problem:


We should grey out the launch button if the deployment name is > 50 chars. There is no reason to create a failed deployment card.

See screenshot




[root@qeblade32 yum.repos.d]# rpm -qa | grep aeolus
aeolus-configure-2.5.0-3.el6.noarch
rubygem-aeolus-image-0.3.0-2.el6.noarch
aeolus-conductor-0.8.0-5.el6.noarch
rubygem-aeolus-cli-0.3.0-3.el6.noarch
aeolus-conductor-doc-0.8.0-5.el6.noarch
aeolus-conductor-daemons-0.8.0-5.el6.noarch
aeolus-all-0.8.0-5.el6.noarch
Comment 1 wes hayutin 2012-01-12 16:34:15 UTC 
adding to ce-sprint
Comment 2 wes hayutin 2012-01-12 16:40:34 UTC 
removing ce-sprint-next tracker
Comment 3 Tzu-Mainn Chen 2012-01-19 21:52:29 UTC 
Patch submitted:

https://fedorahosted.org/pipermail/aeolus-devel/2012-January/008211.html

It doesn't grey out the launch button, but it does give a warning and it does *not* create the deployment.  This is more consistent with how other forms work.
Comment 4 Tzu-Mainn Chen 2012-01-24 06:41:52 UTC 
Pushed to 1.0-staging:

commit c208e29c4c5648ec65e251babc65a8335f171489
BZ 773402 enforce maximum length of 50 chars for names, as longer lengths result in creation errors
Comment 5 Tzu-Mainn Chen 2012-01-24 16:13:10 UTC 
Additional commit and push:

commit 1f47b6e77445a39f2a26cd7a3280f487a49712ed
BZ 773402 fixed deployment name test to limit length to 50
Comment 6 Steve Linabery 2012-01-25 00:11:02 UTC 
1f47b6e  c208e29 in aeolus-conductor-0.8.0-11.el6
Comment 7 Aziza Karol 2012-01-25 08:33:37 UTC 
If Deployment name is more the 50 characters message "Name is too long (maximum is 50 characters)" is displayed.

see attached screenshot.

verified on:
# rpm -qa | grep aeolus
rubygem-aeolus-image-0.3.0-3.el6.noarch
aeolus-conductor-doc-0.8.0-11.el6.noarch
rubygem-aeolus-cli-0.3.0-5.el6.noarch
aeolus-configure-2.5.0-7.el6.noarch
aeolus-conductor-daemons-0.8.0-11.el6.noarch
aeolus-conductor-0.8.0-11.el6.noarch
aeolus-all-0.8.0-11.el6.noarch
Comment 8 Aziza Karol 2012-01-25 08:34:23 UTC 
Created attachment 557397 [details]
ss
Comment 10 errata-xmlrpc 2012-05-15 21:39:25 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.