[ARVADOS] updated: e4a51416586f73593ac68bf0d2a74c53a4875f7e

Git user git at public.curoverse.com
Tue Nov 22 14:07:52 EST 2016


Summary of changes:
 .../app/controllers/projects_controller.rb         |   4 +-
 .../app/views/projects/_show_dashboard.html.erb    |  16 ++-
 .../app/views/projects/_show_processes.html.erb    |   5 +
 apps/workbench/app/views/projects/show.html.erb    |  16 ++-
 .../test/controllers/disabled_api_test.rb          |   4 +
 .../test/integration/application_layout_test.rb    |   2 +-
 .../test/integration/pipeline_instances_test.rb    |   2 +-
 apps/workbench/test/integration/projects_test.rb   |   2 +-
 apps/workbench/test/integration/work_units_test.rb |   2 +-
 apps/workbench/test/performance/browsing_test.rb   |   2 +-
 doc/_includes/_crunch1only_begin.liquid            |   2 +
 doc/_includes/_crunch1only_end.liquid              |   1 +
 doc/_includes/_notebox_begin_warning.liquid        |   2 +
 doc/_includes/_pipeline_deprecation_notice.liquid  |   2 +-
 doc/install/install-keep-web.html.textile.liquid   |   2 +-
 doc/install/install-keepproxy.html.textile.liquid  |   2 +-
 doc/user/cwl/cwl-runner.html.textile.liquid        |   8 +-
 doc/user/topics/arv-run.html.textile.liquid        |   4 +
 ...nning-pipeline-command-line.html.textile.liquid |   4 +
 ...tutorial-pipeline-workbench.html.textile.liquid |   4 +
 sdk/cli/bin/crunch-job                             |   6 +-
 sdk/cwl/arvados_cwl/__init__.py                    |  47 +++++---
 sdk/cwl/arvados_cwl/arvcontainer.py                |  11 +-
 sdk/cwl/arvados_cwl/arvdocker.py                   |   5 +-
 sdk/cwl/arvados_cwl/arvjob.py                      |  25 ++--
 sdk/cwl/arvados_cwl/arvworkflow.py                 |  12 +-
 sdk/cwl/tests/test_submit.py                       |  90 ++++++++++-----
 sdk/go/httpserver/id_generator.go                  |  31 +++++
 sdk/python/arvados/arvfile.py                      |   2 +-
 services/keepstore/azure_blob_volume.go            |   2 +-
 services/keepstore/azure_blob_volume_test.go       |   2 +-
 services/keepstore/bufferpool.go                   |   3 +-
 services/keepstore/config.go                       |  22 +++-
 services/keepstore/config_test.go                  |   2 +-
 services/keepstore/count.go                        |  44 +++++++
 services/keepstore/handler_test.go                 |   2 +-
 services/keepstore/handlers.go                     |  55 ++++++---
 services/keepstore/keepstore.go                    |   8 +-
 services/keepstore/logging_router.go               |  60 ++++++++--
 services/keepstore/pull_worker.go                  |   3 +-
 services/keepstore/s3_volume.go                    | 127 +++++++++++++++++++--
 services/keepstore/s3_volume_test.go               |  32 +++++-
 services/keepstore/trash_worker.go                 |   2 +-
 services/keepstore/usage.go                        |   4 +
 services/keepstore/volume.go                       |  45 ++++++--
 services/keepstore/volume_unix.go                  |  10 +-
 tools/arvbox/lib/arvbox/docker/common.sh           |   4 +
 47 files changed, 598 insertions(+), 144 deletions(-)
 create mode 100644 apps/workbench/app/views/projects/_show_processes.html.erb
 create mode 100644 doc/_includes/_crunch1only_begin.liquid
 create mode 100644 doc/_includes/_crunch1only_end.liquid
 create mode 100644 doc/_includes/_notebox_begin_warning.liquid
 create mode 100644 sdk/go/httpserver/id_generator.go
 create mode 100644 services/keepstore/count.go

       via  e4a51416586f73593ac68bf0d2a74c53a4875f7e (commit)
       via  81ccd9264d0742ba1bf0b9dfbc31ff4f15929f3e (commit)
       via  6f461f4d0a996da85140982846a5d5c10ccfaae4 (commit)
       via  9c3cc0f61751720cfdea62717934746d1aa32b72 (commit)
       via  4ed4b6554535849341673efb7f80392dd5fba946 (commit)
       via  7588bb6abd3886af9e1f3078db573a691a974771 (commit)
       via  55e693dc29608f9b9975bb40c2fe4cf9c51df3f1 (commit)
       via  726becee14c0f4aaa1b0f72fe33a16ee1de7ccd4 (commit)
       via  e41abf0cefe61a69a5dac27647066116eba5af96 (commit)
       via  5debe2b77df18d381d7d59976e76f4ea6c9d5ce0 (commit)
       via  6e38822705235d01fbb7d51626b073174a65e46e (commit)
       via  08d5c8073523b4c5d72413d8e2095f3b68b58190 (commit)
       via  7692a12d05d54f0e209dcc2ff9ae9152560c7d52 (commit)
       via  3afec6824121eea6aed8c2d25567a5e8974d3100 (commit)
       via  185bc33b57a164463a7b67fca84b7596f6f79ad3 (commit)
       via  e5fd39ad76ac8bcb5e7bfe0c6938fdeb7c8ab4e3 (commit)
       via  29356d0c02566aa33b3dbb9513dc701bad8fd16b (commit)
       via  b1e2f45d0a926617c991410feda842a5056ff5f0 (commit)
       via  bf08ad601e8c69e812dcfd5fd88cd711d35647ae (commit)
       via  8c716ec575b5f7679a2ab95ebea944a46ff756c1 (commit)
       via  d542de20d8617f5823ab8f675c114f78aaf4a924 (commit)
       via  515a58c0ef8634fca2397a8609f868524a42132c (commit)
       via  4568673894b4a752503ad403bd391767ac1805e5 (commit)
       via  b45d7c92a23390c8be246219a1c84b8736854581 (commit)
       via  78889e115e6fffd5eb82e54a541bd4858f804f91 (commit)
       via  c1ebef70f3b66080b51ef700383f44d70736f495 (commit)
       via  5977b70a38e7102a6a369074897af990944c8934 (commit)
       via  1071e1163f894c2a73df76cd400d102748e5281d (commit)
       via  322d784513c37abc8fa1d1c05dd5f41019868735 (commit)
       via  6f2ac70b70d4b64b728b815d5be429d5a165e2f0 (commit)
       via  c654baedb04251a741c840860041768ec661d3e7 (commit)
       via  329b27f1407d900f8de7872077e6c91ebb32107c (commit)
       via  a4e81f561cce966074a72ff6219826a89152451a (commit)
       via  6361d996c7a2c7d7648abfcf1699aa989e552f22 (commit)
       via  42f433ac4486c18fa6408d5f942dc394e5ff149e (commit)
      from  9f53e085c98249ba79d85ba59e6f1ca624fede10 (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 e4a51416586f73593ac68bf0d2a74c53a4875f7e
Author: radhika <radhika at curoverse.com>
Date:   Tue Nov 22 14:06:59 2016 -0500

    10293: invoke output_callback in a try/except block

diff --git a/sdk/cwl/arvados_cwl/arvcontainer.py b/sdk/cwl/arvados_cwl/arvcontainer.py
index b127f68..c349839 100644
--- a/sdk/cwl/arvados_cwl/arvcontainer.py
+++ b/sdk/cwl/arvados_cwl/arvcontainer.py
@@ -115,9 +115,9 @@ class ArvadosContainer(object):
                 body=container_request
             ).execute(num_retries=self.arvrunner.num_retries)
 
-            self.arvrunner.processes[response["uuid"]] = response["uuid"]
+            self.arvrunner.processes[response["uuid"]] = self
 
-            logger.info("Container request %s (%s) state is %s with container %s %s", self.name, response["uuid"], response["state"])
+            logger.info("Container request %s (%s) state is %s", self.name, response["uuid"], response["state"])
 
             if response["state"] == "Final":
                 self.done(response)
@@ -146,9 +146,14 @@ class ArvadosContainer(object):
                 processStatus = "permanentFail"
 
             outputs = {}
+
             if container["output"]:
                 outputs = done.done_outputs(self, container, "/tmp", self.outdir, "/keep")
-            self.output_callback(outputs, processStatus)
+            try:
+                self.output_callback(outputs, processStatus)
+            except Exception as e:
+                logger.error("Got error %s" % str(e))
+                self.output_callback({}, "permanentFail")
         finally:
             del self.arvrunner.processes[record["uuid"]]
 

commit 81ccd9264d0742ba1bf0b9dfbc31ff4f15929f3e
Merge: 9f53e08 6f461f4
Author: radhika <radhika at curoverse.com>
Date:   Tue Nov 22 13:38:27 2016 -0500

    Merge branch 'master' into 10293-cwl-cr-output


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


hooks/post-receive
-- 




More information about the arvados-commits mailing list