[ARVADOS] updated: 8aeac20eed5c6eb2cc9f41ada60632265cbdd556

[git at public.curoverse.com]

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

       via  8aeac20eed5c6eb2cc9f41ada60632265cbdd556 (commit)
      from  c53fb9e0fab0ce043602c50e25d7fe8b94c15b79 (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 8aeac20eed5c6eb2cc9f41ada60632265cbdd556
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Thu Dec 10 16:45:41 2015 -0500

    6429: Rename request_finalize to handle_completed

diff --git a/services/api/app/models/container.rb b/services/api/app/models/container.rb
index 33607af..d777c0c 100644
--- a/services/api/app/models/container.rb
+++ b/services/api/app/models/container.rb
@@ -16,7 +16,7 @@ class Container < ArvadosModel
   validates :command, :container_image, :output_path, :cwd, :priority, :presence => true
   validate :validate_state_change
   validate :validate_change
-  after_save :request_finalize
+  after_save :handle_completed
 
   has_many :container_requests, :foreign_key => :container_uuid, :class_name => 'ContainerRequest', :primary_key => :uuid
 
@@ -147,7 +147,7 @@ class Container < ArvadosModel
     check_update_whitelist permitted
   end
 
-  def request_finalize
+  def handle_completed
     # This container is finished so finalize any associated container requests
     # that are associated with this container.
     if self.state_changed? and [Complete, Cancelled].include? self.state

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


hooks/post-receive
--