search

nextflow-io / nextflow

A DSL for data-driven computational pipelines
http://nextflow.io
Apache License 2.0
2.61k stars 605 forks source link

Re-implement some changes that were dropped when #5042 was merged #5063

Closed adamrtalbot closed 2 weeks ago

adamrtalbot commented 2 weeks ago

Some changes were dropped when @swampie's PR was merged. This PR re-adds them from scratch, will try and trim it down so we retain the relevant lines.

netlify[bot] commented 2 weeks ago

Deploy Preview for nextflow-docs-staging canceled.

Name Link
Latest commit 2e03135a3f1ec58dbefe5295853d56a62bde83bf
Latest deploy log https://app.netlify.com/sites/nextflow-docs-staging/deploys/666b1eb1e1e1150008974ce2
pditommaso commented 2 weeks ago

Can be added an integration tests here

https://github.com/nextflow-io/nextflow/blob/658566e83a009c36be3591ac077f650bc1eefc95/validation/azure.sh#L57-L65

adamrtalbot commented 2 weeks ago

Can be added an integration tests here

https://github.com/nextflow-io/nextflow/blob/658566e83a009c36be3591ac077f650bc1eefc95/validation/azure.sh#L57-L65

I think you have to run on a machine with a Managed Identity, which basically means on an Azure VM. I can expand some integration tests there though.

pditommaso commented 2 weeks ago

Merge manually 306814e7