[ARVADOS] updated: 7cdfe579a8ad314cf303a280b0de68b026244748
Git user
git at public.curoverse.com
Mon Jun 5 09:44:53 EDT 2017
Summary of changes:
sdk/cwl/arvados_cwl/arvjob.py | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
via 7cdfe579a8ad314cf303a280b0de68b026244748 (commit)
from f8f8b9030fd63fd6ccc10d02cd2e4cba6ea685b8 (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 7cdfe579a8ad314cf303a280b0de68b026244748
Author: Lucas Di Pentima <lucas at curoverse.com>
Date: Mon Jun 5 10:43:07 2017 -0300
11557: Don't assume log, output and components are always present on a job record.
This was causing some tests to fail.
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas at curoverse.com>
diff --git a/sdk/cwl/arvados_cwl/arvjob.py b/sdk/cwl/arvados_cwl/arvjob.py
index c3eb919..a620a20 100644
--- a/sdk/cwl/arvados_cwl/arvjob.py
+++ b/sdk/cwl/arvados_cwl/arvjob.py
@@ -293,7 +293,8 @@ class RunnerJob(Runner):
if self.enable_reuse:
# When reusing jobs, copy its output/log collection to the desired project
- reused_collections = [('Output', job['output']), ('Log', job['log'])]
+ reused_collections = [('Output', job.get('output', None)),
+ ('Log', job.get('log', None))]
for col_type, pdh in [(n, p) for n, p in reused_collections if p]:
c = arvados.collection.Collection(pdh,
api_client=self.arvrunner.api,
@@ -307,7 +308,7 @@ class RunnerJob(Runner):
col_type.lower(),
c.manifest_locator())
# Give read permission to the desired project on reused jobs
- for job_name, job_uuid in job['components'].items():
+ for job_name, job_uuid in job.get('components', {}).items():
self.arvrunner.api.links().create(body={
'link_class': 'can_read',
'tail_uuid': self.arvrunner.project_uuid,
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list