broadinstitute / viral-ngs

Viral genomics analysis pipelines
Other
186 stars 66 forks source link

update and test Cromwell 32 with Google PAPI v2 #843

Open dpark01 opened 5 years ago

dpark01 commented 5 years ago

Cromwell version 32 starts the transition from PAPIv1 to PAPIv2 and signals the future deprecation some of the old "JES" terminology (the predecessor to PAPI). Our current template Cromwell config file uses this old terminology (and PAPI v1).

We should update our Cromwell template to utilize PAPI v2, keeping in mind that:

Cromwell version 32 in viral-ngs is pending the merge of #838 .

tomkinsc commented 5 years ago

It appears that with WDL 1.0 we can specify the bootsize as part of the runtime block:

runtime {
  bootDiskSizeGb: 50
}
dpark01 commented 5 years ago

Hm... although dxWDL does not yet support WDL 1.0, I think it ignores runtime attributes it doesn't recognize, so maybe we can use this...