Bug 772322 - generated deployable.xml contains spaces, which will fail during launch
Summary: generated deployable.xml contains spaces, which will fail during launch
Keywords:
Status: CLOSED CURRENTRELEASE
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: Matt Wagner
QA Contact: wes hayutin
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-06 20:23 UTC by dgao
Modified: 2012-08-30 17:17 UTC (History)
5 users (show)

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


Attachments (Terms of Use)
. (168.68 KB, image/png)
2012-01-06 20:24 UTC, dgao
no flags Details
2 (229.82 KB, image/png)
2012-01-06 20:24 UTC, dgao
no flags Details
3 (224.98 KB, image/png)
2012-01-06 20:25 UTC, dgao
no flags Details
4 (244.78 KB, image/png)
2012-01-06 20:25 UTC, dgao
no flags Details
image_with_space (221.64 KB, image/png)
2012-01-11 07:55 UTC, Shveta
no flags Details
deployable_with_space (222.28 KB, image/png)
2012-01-11 08:00 UTC, Shveta
no flags Details

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


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