This ended up taking a while to chase down and debug, yet it has a really simple fix. The bug happens when you attempt to build an XML deployable from an image in Conductor, and the image name contains spaces.
This should fix https://bugzilla.redhat.com/show_bug.cgi?id=772322
Resolves https://bugzilla.redhat.com/show_bug.cgi?id=772322 --- src/app/models/deployable.rb | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/app/models/deployable.rb b/src/app/models/deployable.rb index 54d6129..5720617 100644 --- a/src/app/models/deployable.rb +++ b/src/app/models/deployable.rb @@ -104,7 +104,7 @@ class Deployable < ActiveRecord::Base doc.root << description assemblies = doc.create_element('assemblies') doc.root << assemblies - assembly = doc.create_element('assembly', :name => image.name, :hwp => hw_profile.name) + assembly = doc.create_element('assembly', :name => image.name.gsub(/[^a-zA-Z0-9]+/, '-'), :hwp => hw_profile.name) assemblies << assembly img = doc.create_element('image', :id => image.uuid) assembly << img
On Mon, Jan 09, 2012 at 11:13:23AM -0500, Matt Wagner wrote:
Resolves https://bugzilla.redhat.com/show_bug.cgi?id=772322
src/app/models/deployable.rb | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/app/models/deployable.rb b/src/app/models/deployable.rb index 54d6129..5720617 100644 --- a/src/app/models/deployable.rb +++ b/src/app/models/deployable.rb @@ -104,7 +104,7 @@ class Deployable < ActiveRecord::Base doc.root << description assemblies = doc.create_element('assemblies') doc.root << assemblies
- assembly = doc.create_element('assembly', :name => image.name, :hwp => hw_profile.name)
- assembly = doc.create_element('assembly', :name => image.name.gsub(/[^a-zA-Z0-9]+/, '-'), :hwp => hw_profile.name) assemblies << assembly img = doc.create_element('image', :id => image.uuid) assembly << img
-- 1.7.6.5
Trivial enough that I'm self-ACKing this. Pushing momentarily.
-- Matt
aeolus-devel@lists.fedorahosted.org
-
Matt Wagner