[PATCH conductor 1/3] BZ #795794 - Display a saner error message if launch fails.

Matt Wagner matt.wagner at redhat.com
Tue Feb 21 21:08:21 UTC 2012


The messages after the first line appear to be more of a dump of the
request than helpful error text, so only relay the first line to the
user -- this is the one that describes the problem.

Resolves https://bugzilla.redhat.com/show_bug.cgi?id=795794
---
 src/app/models/deployment.rb |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/app/models/deployment.rb b/src/app/models/deployment.rb
index 9593645..a23080c 100644
--- a/src/app/models/deployment.rb
+++ b/src/app/models/deployment.rb
@@ -226,7 +226,7 @@ class Deployment < ActiveRecord::Base
       rescue
         logger.error $!.message
         logger.error $!.backtrace.join("\n    ")
-        status[:errors][assembly.name] = $!.message
+        status[:errors][assembly.name] = $!.message.to_s.split("\n").first
       end
     end
 
@@ -286,7 +286,7 @@ class Deployment < ActiveRecord::Base
       rescue
         logger.error $!.message
         logger.error $!.backtrace.join("\n    ")
-        status[:errors][assembly_name] = $!.message
+        status[:errors][assembly_name] = $!.message.to_s.split("\n").first
       end
     end
     status
-- 
1.7.6.5




More information about the aeolus-devel mailing list