From 846a2728f3cfc80efc220a60536dea16b5f0fcbd Mon Sep 17 00:00:00 2001
From: Peter Amstutz <peter.amstutz@curoverse.com>
Date: Fri, 11 Mar 2016 16:54:59 -0500
Subject: [PATCH] 8654: Passes in local testing.

---
 crunch_scripts/cwl-runner       | 5 +++--
 docker/jobs/Dockerfile          | 4 +---
 sdk/cwl/arvados_cwl/__init__.py | 3 ++-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/crunch_scripts/cwl-runner b/crunch_scripts/cwl-runner
index 8cbf4d876e..59f6817336 100755
--- a/crunch_scripts/cwl-runner
+++ b/crunch_scripts/cwl-runner
@@ -8,6 +8,7 @@ from cwltool.process import shortname
 import cwltool.main
 import logging
 import os
+import json
 from arvados.api import OrderedJsonModel
 
 api = arvados.api("v1")
@@ -28,8 +29,8 @@ try:
     outputObj = runner.arvExecutor(t, job_order_object, "", None)
 
     c = arvados.collection.Collection()
-    with c.open("cwl.output.json") as f:
-        json.dump(f, indent=4)
+    with c.open("cwl.output.json", "w") as f:
+        json.dump(outputObj, f, indent=4)
 
     api.job_tasks().update(uuid=arvados.current_task()['uuid'],
                                          body={
diff --git a/docker/jobs/Dockerfile b/docker/jobs/Dockerfile
index 836634909b..bbe7844b7d 100644
--- a/docker/jobs/Dockerfile
+++ b/docker/jobs/Dockerfile
@@ -6,10 +6,8 @@ ENV DEBIAN_FRONTEND noninteractive
 
 ADD apt.arvados.org.list /etc/apt/sources.list.d/
 RUN apt-key adv --keyserver pool.sks-keyservers.net --recv 1078ECD7
-RUN apt-get update -q
 
-RUN apt-get install -qy git python-pip python-virtualenv python-arvados-python-client python-dev libcurl4-gnutls-dev nodejs
-RUN pip install arvados-cwl-runner
+RUN apt-get update -q && apt-get install -qy git python-pip python-virtualenv python-arvados-python-client python-dev libcurl4-gnutls-dev nodejs python-arvados-cwl-runner
 
 RUN gpg --keyserver pool.sks-keyservers.net --recv-keys D39DC0E3
 
diff --git a/sdk/cwl/arvados_cwl/__init__.py b/sdk/cwl/arvados_cwl/__init__.py
index 885497171e..67a324ac85 100644
--- a/sdk/cwl/arvados_cwl/__init__.py
+++ b/sdk/cwl/arvados_cwl/__init__.py
@@ -362,7 +362,8 @@ class ArvCwlRunner(object):
         self.fs_access = CollectionFsAccess(input_basedir)
 
         kwargs["fs_access"] = self.fs_access
-        kwargs["enable_reuse"] = args.enable_reuse
+        if args:
+            kwargs["enable_reuse"] = args.enable_reuse
 
         kwargs["outdir"] = "$(task.outdir)"
         kwargs["tmpdir"] = "$(task.tmpdir)"
-- 
2.30.2