-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add config and/or modify implementation of execution paths #112
Comments
My suggestion:
|
This was referenced May 10, 2023
Merged
ashanhol
added a commit
that referenced
this issue
May 15, 2023
Fixes #112 This PR further aims to untangle CoA from TES by separating cromwell's execution files from TES's batch files. The majority of this work is done in BatchScheduler.cs's ConvertTesTaskToBatchTaskAsync function. Created a new container in our storage account called tes-internal which is the new location for the upload, download, and batch scripts as well as metrics.txt. New storage path convention is /tes-internal/task.id/[file] Got rid of Tes's executions container. Create tes-internal container in CoA deployer: Create tes-internal container during deploy CromwellOnAzure#658 On the batch node, truncated the long paths that were previously something like cromwell-executions/assemble_refbased/0fb40b27-dffe-4d3f-b6da-02ea0b82aedd/call-align_to_ref/shard-0/execution/__batch/batch_script to wd/batch_script. This required some finagling with the mounting directories. Updated any tests that were relying on the previous way of doing things. --------- Co-authored-by: Blair L Murri <BMurri@users.noreply.github.com>
ashanhol
added a commit
to microsoft/CromwellOnAzure
that referenced
this issue
May 15, 2023
Sister PR to microsoft/ga4gh-tes#222 Issue: microsoft/ga4gh-tes#112 Creates the tes-internal container for convenience to TES.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Investigate and propose the changes required to make the batch script have the desired path schema for Terra:
Current:
Desired:
The text was updated successfully, but these errors were encountered: