[ARVADOS] updated: 9706665da1dd8f3ccb8129933f02f4b7e21bc878

git at public.curoverse.com git at public.curoverse.com
Thu May 29 10:08:57 EDT 2014


Summary of changes:
 services/api/app/models/pipeline_instance.rb | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

       via  9706665da1dd8f3ccb8129933f02f4b7e21bc878 (commit)
      from  e2a7fe50d3b6e98b7f549c78c4f91dbe4066ddbb (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 9706665da1dd8f3ccb8129933f02f4b7e21bc878
Author: radhika <radhika at curoverse.com>
Date:   Thu May 29 10:08:02 2014 -0400

    2903: during validation methods, set state instead of success

diff --git a/services/api/app/models/pipeline_instance.rb b/services/api/app/models/pipeline_instance.rb
index 185bf02..f83d436 100644
--- a/services/api/app/models/pipeline_instance.rb
+++ b/services/api/app/models/pipeline_instance.rb
@@ -8,7 +8,7 @@ class PipelineInstance < ArvadosModel
   belongs_to :pipeline_template, :foreign_key => :pipeline_template_uuid, :primary_key => :uuid
 
   before_validation :bootstrap_components
-  before_validation :update_success
+  before_validation :update_status
   before_validation :verify_status
   before_create :set_state_before_save
   before_save :set_state_before_save
@@ -78,7 +78,7 @@ class PipelineInstance < ArvadosModel
         else
           row << 0.0
           if step['failed']
-            self.success = false
+            self.state = Failed
           end
         end
         row << (step['warehousejob']['id'] rescue nil)
@@ -109,9 +109,9 @@ class PipelineInstance < ArvadosModel
     end
   end
 
-  def update_success
+  def update_status
     if components and progress_ratio == 1.0
-      self.success = true
+      self.state = Complete
     end
   end
 

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list