-
Notifications
You must be signed in to change notification settings - Fork 582
ci(ghcr): add deploy on GitHub #441
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
base: master
Are you sure you want to change the base?
Conversation
context: . | ||
push: true | ||
tags: | | ||
ghcr.io/${{ github.repository_owner }}/redis:latest |
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.
I think that we shouldn't set the tag to latest due to versioning scheme expectations which might cause confusion.
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.
Lets use the specific commit SHA only for the time being
|
||
on: | ||
push: | ||
branches: |
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.
We should also trigger for release/*
branches
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.
Ok, what we want ?
- Trigger on tags ?
- Trigger on master ?
- Trigger on release/* ?
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.
I think that we should trigger on pushes to master and release/* and tag with the commit SHA for the time being.
- name: Set up Docker Buildx | ||
uses: docker/setup-buildx-action@v3 | ||
|
||
- name: Build and push Docker images |
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.
We provide various platforms for our images, and this action seems like it would only push the amd64
architecture. What do you think about adding the rest of the archs, like we do on Dockerhub?
Simple question why you don't use tags ? 🤔 |
@TartanLeGrand |
This pull request introduces a new GitHub Actions workflow to automate the publishing of Docker images to the GitHub Container Registry (GHCR). The workflow is triggered on pushes to the
main
branch and includes steps for building and pushing Docker images.New GitHub Actions Workflow:
.github/workflows/publish-to-ghcr.yml
: Added a workflow named "Publish Docker Images to GHCR" that builds and pushes Docker images to GHCR. It includes steps for checking out the repository, logging into GHCR, setting up QEMU and Docker Buildx, and building and pushing Docker images with appropriate tags.