Bug 773402

Summary: Grey out launch button if deployment name is > 50 chars
Product: [Retired] CloudForms Cloud Engine Reporter: wes hayutin <whayutin>
Component: aeolus-conductorAssignee: Tzu-Mainn Chen <tzumainn>
Status: CLOSED ERRATA QA Contact: wes hayutin <whayutin>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, deltacloud-maint, slinaber, ssachdev
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-05-15 21:39:25 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
ss
none
ss none

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