-
Notifications
You must be signed in to change notification settings - Fork 92
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
Switch to buildx for github actions #394
Conversation
scottyhq
commented
Oct 17, 2022
- Switch to docker github actions such as (docker/build-push-action) that are more flexible for newer buildkit options (Docker BuildX for parallel builds and conda package cache #357)
- Use only docker, not docker and podman for quay.io to speed up CI (fixes New DockerHub Image retention policies will delete unused images after 6 months #121 (comment)
- Cancel concurrent Binder and Test workflows for PRs if multiple commits are made
* use docker buildx in Test workflow
${{env.DOCKER_ORG}}/${{ env.IMAGE }}:master | ||
${{env.DOCKER_ORG}}/${{ env.IMAGE }}:${{env.DOCKER_TAG}} | ||
quay.io/${{env.DOCKER_ORG}}/${{ env.IMAGE }}:master |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just flagging the use of master
as the docker image tag... I suppose this is just following the git repo's master branch right now, and latest needs to be explicitly tagged (i.e. latest
is semi-stable)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exactly. explained here https://github.com/pangeo-data/pangeo-docker-images#image-tagging-and-continuous-building
Nice work @scottyhq! This seems like a huge improvement. Thanks for your maintenance work! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great to me! I think you should merge this yourself @scottyhq to test the push part of the workflow!