Merge branch 'master' into 10081-cwl-run-same-job

[arvados.git] / sdk / cwl / arvados_cwl / arvworkflow.py

diff --git a/sdk/cwl/arvados_cwl/arvworkflow.py b/sdk/cwl/arvados_cwl/arvworkflow.py
index ab3940998bd669c3f5bc389866b19effb9a708d3..ab8ad035fd6c8d13d14b3d226135993e6c0792ce 100644 (file)
--- 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)
@@ -106,12 +108,12 @@ class ArvadosWorkflow(Workflow):
                             "entryname": "workflow.cwl",
                             "entry": yaml.safe_dump(packed).replace("\\", "\\\\").replace('$(', '\$(').replace('${', '\${')
                         }, {
-                            "entryname": "cwl.input.json",
+                            "entryname": "cwl.input.yml",
                             "entry": yaml.safe_dump(joborder_keepmount).replace("\\", "\\\\").replace('$(', '\$(').replace('${', '\${')
                         }]
                 }],
                 "hints": workflowobj["hints"],
-                "arguments": ["--no-container", "--move-outputs", "workflow.cwl#main", "cwl.input.json"]
+                "arguments": ["--no-container", "--move-outputs", "workflow.cwl#main", "cwl.input.yml"]
             }
             kwargs["loader"] = self.doc_loader
             kwargs["avsc_names"] = self.doc_schema