Bug 772322

Summary: generated deployable.xml contains spaces, which will fail during launch
Product: [Retired] CloudForms Cloud Engine Reporter: dgao
Component: aeolus-conductorAssignee: Matt Wagner <matt.wagner>
Status: CLOSED CURRENTRELEASE QA Contact: wes hayutin <whayutin>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, athomas, deltacloud-maint, matt.wagner, 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-08-30 17:17:08 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
.
none
2
none
3
none
4
none
image_with_space
none
deployable_with_space none

Description dgao 2012-01-06 20:23:02 UTC
See attached screenshots.

Comment 1 dgao 2012-01-06 20:24:14 UTC
Created attachment 551252 [details]
.

Comment 2 dgao 2012-01-06 20:24:50 UTC
Created attachment 551253 [details]
2

Comment 3 dgao 2012-01-06 20:25:01 UTC
Created attachment 551254 [details]
3

Comment 4 dgao 2012-01-06 20:25:16 UTC
Created attachment 551255 [details]
4

Comment 5 dgao 2012-01-06 20:26:23 UTC
Expected Result: Conductor help convert the spaces into an acceptable character in the auto generated deployable xml

Comment 6 Matt Wagner 2012-01-06 21:47:16 UTC
From looking at this a little, it seems that you get here by letting Conductor build a Deployable XML object from an image, where the image name contains spaces.

I *think* that Deltacloud is objecting to spaces in the Assembly name. We could probably just replace with underscores.

Comment 8 Matt Wagner 2012-01-10 22:30:26 UTC
Pushed onto master:

commit 521b9b81611ec0c2244064e5bdab5f481f2ed0e3
Author: Matt Wagner <matt.wagner>
Date:   Mon Jan 9 11:02:34 2012 -0500

    Sanitize assembly name to prevent launch errors
    
    Resolves https://bugzilla.redhat.com/show_bug.cgi?id=772322

Comment 9 Shveta 2012-01-11 07:54:43 UTC
Created a auto generated deployable with image name containing spaces.

1) Environment -- > New Image .
2) From URL , provide a template 
3) Image name in template contains spaces.
4) Catalog -- > deployable is listed there
5) click on deployable and launch.
6) Image is launched .

Screenshot attached.

Comment 10 Shveta 2012-01-11 07:55:41 UTC
Created attachment 552028 [details]
image_with_space

Comment 11 Shveta 2012-01-11 08:00:01 UTC
Created attachment 552029 [details]
deployable_with_space

Comment 12 Shveta 2012-01-11 08:07:20 UTC
verified in 

rpm -qa|grep aeous
[root@pogolinux-1 yum.repos.d]# rpm -qa|grep aeolus
aeolus-conductor-0.8.0-4.el6.noarch
aeolus-configure-2.5.0-2.el6.noarch
aeolus-conductor-doc-0.8.0-4.el6.noarch
aeolus-all-0.8.0-4.el6.noarch
rubygem-aeolus-cli-0.3.0-3.el6.noarch
rubygem-aeolus-image-0.3.0-2.el6.noarch
aeolus-conductor-daemons-0.8.0-4.el6.noarch

Comment 13 wes hayutin 2012-01-12 16:16:33 UTC
bugs in verified or on_qa moving off tracker