Skip to content

Commit 56c8582

Browse files
committed
Maintain consistency with parameter list when invoking cwl_runner.
1 parent 20f27c0 commit 56c8582

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

wes_service/arvados_wes.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ def ListRuns(self, page_size=None, page_token=None, state_search=None):
109109
}
110110

111111
def invoke_cwl_runner(self, cr_uuid, workflow_url, workflow_params,
112-
env, workflow_attachment_file, project_uuid,
112+
env, project_uuid,
113113
tempdir):
114114
api = arvados.api_from_config(version="v1", apiconfig={
115115
"ARVADOS_API_HOST": env["ARVADOS_API_HOST"],
@@ -150,9 +150,6 @@ def invoke_cwl_runner(self, cr_uuid, workflow_url, workflow_params,
150150
except subprocess.CalledProcessError as e:
151151
api.container_requests().update(uuid=cr_uuid, body={"priority": 0,
152152
"properties": {"arvados-cwl-runner-log": str(e)}}).execute()
153-
finally:
154-
if workflow_attachment_file is not None:
155-
workflow_attachment_file.close()
156153

157154
@catch_exceptions
158155
def RunWorkflow(self, **args):

0 commit comments

Comments
 (0)