-
Notifications
You must be signed in to change notification settings - Fork 622
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
Updating the default SKU for Azure Batch #2868
Conversation
Marked as draft 😄 Discussing with Venkat how we can avoid having the same image deprecation issue again. Thinking of implementing a pre-flight check for confirming that the specified image exists. |
859008a
to
bde6df0
Compare
@pditommaso , I've opened up this PR for a review so that this patches the current default settings. I will follow up with another PR to introduce a pre-flight check to see if the image specified exists or has been retired, essentially the
|
This commit update the default SKU used by Batch to `batch.node.ubuntu 20.04` in place of `batch.node.centos 8`
This PR addresses the deprecation of CentOS-8 Azure/Batch#119 , the defaults for a Nextflow pool have become a bit problematic.
From experience with some clients the straight-forward solution seems to be to move to
Ubuntu
based base image rather thanCentOS 7.8
and therefore I've gone ahead and changed those settings.Happy to accommodate any feedback 😊