[ARVADOS] updated: 8b7d63024652c112973d4dd82f9a5d89cc624fc7

[Git user]

Summary of changes:
 sdk/cwl/arvados_cwl/__init__.py    | 4 ++--
 sdk/cwl/arvados_cwl/arvworkflow.py | 6 ++++--
 2 files changed, 6 insertions(+), 4 deletions(-)

       via  8b7d63024652c112973d4dd82f9a5d89cc624fc7 (commit)
      from  0ff7b94edaaaa07932ae757c0a2b7ba3fde026cb (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 8b7d63024652c112973d4dd82f9a5d89cc624fc7
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Fri Sep 23 14:27:35 2016 -0400

    10081: Change logical identifier for Arvados schema to http://arvados.org/cwl.
    Improve exception text when checking that locations are keep references.

diff --git a/sdk/cwl/arvados_cwl/__init__.py b/sdk/cwl/arvados_cwl/__init__.py
index f52c4a5..cd38003 100644
--- a/sdk/cwl/arvados_cwl/__init__.py
+++ b/sdk/cwl/arvados_cwl/__init__.py
@@ -368,10 +368,10 @@ def arg_parser():  # type: () -> argparse.ArgumentParser
 def add_arv_hints():
     cache = {}
     res = pkg_resources.resource_stream(__name__, 'arv-cwl-schema.yml')
-    cache["https://w3id.org/cwl/arv-cwl-schema.yml"] = res.read()
+    cache["http://arvados.org/cwl"] = res.read()
     res.close()
     _, cwlnames, _, _ = cwltool.process.get_schema("v1.0")
-    _, extnames, _, _ = schema_salad.schema.load_schema("https://w3id.org/cwl/arv-cwl-schema.yml", cache=cache)
+    _, extnames, _, _ = schema_salad.schema.load_schema("http://arvados.org/cwl", cache=cache)
     for n in extnames.names:
         if not cwlnames.has_name("http://arvados.org/cwl#"+n, ""):
             cwlnames.add_name("http://arvados.org/cwl#"+n, "", extnames.get_name(n, ""))
diff --git a/sdk/cwl/arvados_cwl/arvworkflow.py b/sdk/cwl/arvados_cwl/arvworkflow.py
index 6087648..ab8ad03 100644
--- a/sdk/cwl/arvados_cwl/arvworkflow.py
+++ b/sdk/cwl/arvados_cwl/arvworkflow.py
@@ -6,7 +6,7 @@ import logging
 from cwltool.pack import pack
 from cwltool.load_tool import fetch_document
 from cwltool.process import shortname
-from cwltool.workflow import Workflow
+from cwltool.workflow import Workflow, WorkflowException
 from cwltool.pathmapper import adjustFileObjs, adjustDirObjs
 
 import ruamel.yaml as yaml
@@ -83,8 +83,10 @@ class ArvadosWorkflow(Workflow):
             def keepmount(obj):
                 if obj["location"].startswith("keep:"):
                     obj["location"] = "/keep/" + obj["location"][5:]
+                elif obj["location"].startswith("_:"):
+                    pass
                 else:
-                    raise Exception("Uh oh %s" % obj["location"])
+                    raise WorkflowException("Location is not a keep reference or a literal: '%s'" % obj["location"])
                 if "listing" in obj:
                     del obj["listing"]
             adjustFileObjs(joborder_keepmount, keepmount)

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


hooks/post-receive
--