-
Notifications
You must be signed in to change notification settings - Fork 642
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
Support not using a container in Azure Batch #5536
base: master
Are you sure you want to change the base?
Support not using a container in Azure Batch #5536
Conversation
Signed-off-by: adamrtalbot <[email protected]>
…Batch Signed-off-by: adamrtalbot <[email protected]>
Signed-off-by: adamrtalbot <[email protected]>
Signed-off-by: adamrtalbot <[email protected]>
✅ Deploy Preview for nextflow-docs-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Given this, would it be worth keeping the strict validation on by default and instead providing an override config option to turn it off (like |
That's a great idea! |
Signed-off-by: adamrtalbot <[email protected]>
b327beb
to
ad07555
Compare
Signed-off-by: adamrtalbot <[email protected]>
Signed-off-by: adamrtalbot <[email protected]>
Signed-off-by: adamrtalbot <[email protected]>
Signed-off-by: adamrtalbot <[email protected]>
Signed-off-by: adamrtalbot <[email protected]>
@tom-seqera implemented now as I'm still not sure this is a wise thing to do, but it unblocks some very specific use cases and it's a relatively buried feature. |
My understanding is that some tasks my require a container, some other may not, therefore a global setting is sub-optimal. Also, could not be Dragen accessed by the container anyway? this is for example how it works with AWS Batch (the dragen path in the host is made accessible in the container context) |
Our implementation only works on AWS using a custom AMI. The AMI provided by Illumina uses CentOS 7, which is EOL. And obviously, it's AWS only. The official AMI provided by Illumina for Azure is not compatible with Docker and they do not want to update it. So we end up either using a Docker compatible image which does not work on Azure Batch, or a specific Dragen based image which does not use Docker. We spent a long time playing around with this and this was the most parsimonious solution we could make work. |
What this means exactly? |
Illumina provide a machine image for Dragen, it’s not compatible with Docker. |
There are rare cases where a user needs to run a process on a bare VM on Azure Batch. Most notably, when using the Dragen which requires you to use a specific machine image that does not support a Docker container.
Previously, Nextflow prevented this, but this PR removes that safeguard and replaces it with a warning. It's not a good idea, but it can unblock users if they need to use a bare VM.
If nothing else, this PR functions as proof it can be done if anyone else needs to use it.