Skip to content

Commit

Permalink
Merge pull request #44 from HumanCellAtlas/ajc-update-new-cromwell-to…
Browse files Browse the repository at this point in the history
…ols-api

updated cromwell-tools start_workflow parameters to comport with new cromwell-tools CLI.
  • Loading branch information
ambrosejcarr authored Dec 14, 2017
2 parents cfdd3d3 + b3503ca commit c81dadd
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions lira/api/notifications.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@
import json
import time
from flask import current_app
from pipeline_tools import gcs_utils
from cromwell_tools import cromwell_tools
import lira_utils


def post(body):
# Check authentication
logger = logging.getLogger('green-box')
Expand Down Expand Up @@ -46,9 +46,15 @@ def post(body):
wdl_deps_file = cromwell_tools.make_zip_in_memory(url_to_contents)

cromwell_response = cromwell_tools.start_workflow(
wdl_file, wdl_deps_file, cromwell_inputs_file,
wdl_default_inputs_file, options_file, green_config.cromwell_url,
green_config.cromwell_user, green_config.cromwell_password)
wdl_file=wdl_file,
zip_file=wdl_deps_file,
inputs_file=cromwell_inputs_file,
inputs_file2=wdl_default_inputs_file,
options_file=options_file,
url=green_config.cromwell_url,
user=green_config.cromwell_user,
password=green_config.cromwell_password
)

# Respond
if cromwell_response.status_code > 201:
Expand Down

0 comments on commit c81dadd

Please sign in to comment.