[ARVADOS] updated: 1.1.4-552-g7cebf3bba

Git user git at public.curoverse.com
Fri Jul 6 13:50:50 EDT 2018


Summary of changes:
 sdk/cwl/arvados_cwl/arvworkflow.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

       via  7cebf3bba60a98897ebddf5e613db6337e5f1513 (commit)
      from  334c069f4da11278844f4276b5c254e8cbf06412 (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 7cebf3bba60a98897ebddf5e613db6337e5f1513
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Fri Jul 6 13:48:34 2018 -0400

    13681: Revert to yaml.round_trip_dump instead of json.dumps().
    
    Also style tweaks.
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/sdk/cwl/arvados_cwl/arvworkflow.py b/sdk/cwl/arvados_cwl/arvworkflow.py
index f5afd387e..0706ef7db 100644
--- a/sdk/cwl/arvados_cwl/arvworkflow.py
+++ b/sdk/cwl/arvados_cwl/arvworkflow.py
@@ -52,8 +52,8 @@ def upload_workflow(arvRunner, tool, job_order, project_uuid, uuid=None,
     upload_dependencies(arvRunner, name, tool.doc_loader,
                         packed, tool.tool["id"], False)
 
-    hints = main["hints"] if "hints" in main else []
     if submit_runner_ram:
+        hints = main.get("hints", [])
         found = False
         for h in hints:
             if h["class"] == "http://arvados.org/cwl#WorkflowRunnerResources":
@@ -63,13 +63,13 @@ def upload_workflow(arvRunner, tool, job_order, project_uuid, uuid=None,
         if not found:
             hints.append({"class": "http://arvados.org/cwl#WorkflowRunnerResources",
                           "ramMin": submit_runner_ram})
-    main["hints"] = hints
+        main["hints"] = hints
 
     body = {
         "workflow": {
             "name": name,
             "description": tool.tool.get("doc", ""),
-            "definition": json.dumps(packed, indent=2, sort_keys=True)
+            "definition":yaml.round_trip_dump(packed)
         }}
     if project_uuid:
         body["workflow"]["owner_uuid"] = project_uuid

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list