[arvados] updated: 2.1.0-3013-g88d70b6d8

git repository hosting git at public.arvados.org
Fri Nov 11 14:53:49 UTC 2022


Summary of changes:
 sdk/cwl/arvados_cwl/arvworkflow.py | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

       via  88d70b6d852d2347a958b7e76de3f47df2699e70 (commit)
      from  9f39acdc1d65d8b2714ffccd51a0311e6df9ec4a (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 88d70b6d852d2347a958b7e76de3f47df2699e70
Author: Peter Amstutz <peter.amstutz at curii.com>
Date:   Fri Nov 11 09:53:18 2022 -0500

    19688: Make sure CommandLineTool gets wrapped properly
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>

diff --git a/sdk/cwl/arvados_cwl/arvworkflow.py b/sdk/cwl/arvados_cwl/arvworkflow.py
index c0b479c37..7f9c3c348 100644
--- a/sdk/cwl/arvados_cwl/arvworkflow.py
+++ b/sdk/cwl/arvados_cwl/arvworkflow.py
@@ -61,10 +61,26 @@ def make_wrapper_workflow(arvRunner, main, packed, project_uuid, name):
         "run": "keep:%s/workflow.json#main" % pdh
     }
 
+    newinputs = []
+    for i in main["inputs"]:
+        inp = {}
+        # Make sure to only copy known fields that are meaningful at
+        # the workflow level. In practice this ensures that if we're
+        # wrapping a CommandLineTool we don't grab inputBinding.
+        # Right now also excludes extension fields, which is fine,
+        # Arvados doesn't currently look for any extension fields on
+        # input parameters.
+        for f in ("type", "label", "secondaryFiles", "streamable",
+                  "doc", "id", "format", "loadContents",
+                  "loadListing", "default"):
+            if f in i:
+                inp[f] = i[f]
+        newinputs.append(inp)
+
     wrapper = {
         "class": "Workflow",
         "id": "#main",
-        "inputs": main["inputs"],
+        "inputs": newinputs,
         "outputs": [],
         "steps": [step]
     }

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list