-
-
Notifications
You must be signed in to change notification settings - Fork 152
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
Add arm builds to docker images #128
Comments
cc @jacobtomlinson for when he gets back. @holdenk I'm curious, how often do you see people using docker on arm for dask-like workloads today? I'm curious how much we should be prioritizing this. |
I’m not sure, for my use case it’s for trying to use dask on kubernetes with Jetsons which are ARM based. |
This sounds reasonable. As Dask is pure Python there is no reason why ARM shouldn't just work. Although it is worth noting that we are not currently testing against ARM. We currently use the It looks like |
Enabling building for ARM has been completed in #166 thanks to @holdenk! In #149 we updated the CI/CD to use GitHub Actions to build and push our images. Due to a limitation in This isn't possible during an unprivileged CI run as there are no credentials to push to a registry. To work around this we load the image into Docker and use the dask-docker/.github/workflows/build.yml Line 68 in 2f81f71
dask-docker/.github/workflows/build.yml Line 33 in 2f81f71
However the @holdenk do you have any suggestions on how we could work around this? Can the image cache be exported/imported in a different way? Or perhaps we need to set up some local temporary registry and push the interim image there? |
I'm going to reopen this until we get CI builds running. If you have any thoughts @holdenk I'd be keen to hear as I'm a bit stuck. |
Sure one option is using qemu to cross build in the CI. Let me see what I can figure out this weekend :) |
Hello! |
It would be really useful if the default builds supported arm, docker buildx makes this a lot easier than it used to be.
The text was updated successfully, but these errors were encountered: