[ARVADOS] updated: 4f03b29b3efb42a48c3f397942d5b618b89390eb

[Git user]

Summary of changes:
 sdk/cwl/arvados_cwl/arvjob.py                |   9 +-
 sdk/cwl/arvados_cwl/crunch_script.py         |   1 +
 sdk/cwl/tests/test_job.py                    |  25 +++---
 sdk/cwl/tests/test_submit.py                 |   9 +-
 sdk/dev-jobs.dockerfile                      |   2 +-
 services/api/app/models/pipeline_instance.rb |   6 +-
 services/crunch-run/crunchrun.go             |   6 +-
 services/keepstore/azure_blob_volume.go      | 125 +++++++++++++++++++++++----
 services/keepstore/azure_blob_volume_test.go |  64 ++++++++++++++
 9 files changed, 205 insertions(+), 42 deletions(-)

       via  4f03b29b3efb42a48c3f397942d5b618b89390eb (commit)
       via  2eb7a28fa7900a005bf48dc40dd1af16d0bc455d (commit)
       via  cafa08eae78c6e29898164f8b5b2fc0127d69f48 (commit)
       via  b96bff759bb71ebb80d7ea33a7b7944eb7c0269f (commit)
       via  ebb2559b3a09636ff687316bbe512e0e8a86b168 (commit)
       via  3beb006fd2c533a192379c9730a3a730c5493b90 (commit)
       via  c1a8e01539932e2f0153cfb2ffc4eaa2c3dc00f1 (commit)
       via  53468cf5e609c4456d1bfd24aeb74c326ff6a8b7 (commit)
       via  5fe2a0ab78c82b67b92434596d554bedda1d4259 (commit)
       via  a092ab09fab941057e5cc44d93886676d542bdd6 (commit)
       via  d0dc273718a2f657643a2b2800d984c7a3a62f78 (commit)
      from  4115524460e89f36b99f2fb5de1adebd86daaf4a (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 4f03b29b3efb42a48c3f397942d5b618b89390eb
Merge: 2eb7a28 cafa08e
Author: Lucas Di Pentima <lucas at curoverse.com>
Date:   Wed Dec 21 14:28:41 2016 -0300

    10671: Merge branch 'master' into 10671-pipeline-instance-finish-time


commit 2eb7a28fa7900a005bf48dc40dd1af16d0bc455d
Author: Lucas Di Pentima <lucas at curoverse.com>
Date:   Wed Dec 21 14:26:25 2016 -0300

    10671: Changed default assigments to be before validation. Also make sure started_at is not nil when pipeline instance is finished.

diff --git a/services/api/app/models/pipeline_instance.rb b/services/api/app/models/pipeline_instance.rb
index dada5d6..f84c4a3 100644
--- a/services/api/app/models/pipeline_instance.rb
+++ b/services/api/app/models/pipeline_instance.rb
@@ -10,9 +10,9 @@ class PipelineInstance < ArvadosModel
   before_validation :bootstrap_components
   before_validation :update_state
   before_validation :verify_status
+  before_validation :update_timestamps_when_state_changes
   before_create :set_state_before_save
   before_save :set_state_before_save
-  before_save :update_timestamps_when_state_changes
 
   api_accessible :user, extend: :common do |t|
     t.add :pipeline_template_uuid
@@ -144,7 +144,9 @@ class PipelineInstance < ArvadosModel
     when RunningOnServer, RunningOnClient
       self.started_at ||= db_current_time
     when Failed, Complete
-      self.finished_at ||= db_current_time
+      current_time = db_current_time
+      self.started_at ||= current_time
+      self.finished_at ||= current_time
     end
   end
 

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


hooks/post-receive
--