GCP Batch-friendly Cromwell root detection [VS-1532] (#9069) #9102
Conversation
| CROMWELL_ROOT=/cromwell_root | ||
| elif [[ -e /mnt/disks/cromwell_root/gcs_delocalization.sh ]] | ||
| then | ||
| CROMWELL_ROOT=/mnt/disks/cromwell_root |
There was a problem hiding this comment.
I know that AoU uses individual instances of Cromwell instead of Terra's central instance. Does this assignment account for this?
There was a problem hiding this comment.
This code should be fine to run on GCP Batch or PAPI regardless of where the Cromwell instance is deployed.
But you will need to update the configs for your Cromwells to point to GCP Batch. Did the Cromwell team help you with this before?
There was a problem hiding this comment.
And also upgrade your version of Cromwell to get all the GCP Batch fixes...
There was a problem hiding this comment.
They may have, but this is the first time that I have dealt with this, so I will look into it.
Thanks for your help with all this!
Probably not but easy enough to pick over the commit that fixes that too... |
AoU RWB Echo version of a GCP Batch compatibility fix already on ah_var_store.