-
Notifications
You must be signed in to change notification settings - Fork 9
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
[GEN-1164] add workflow to build docker image and push it to ghcr #556
Conversation
Remove tags section in docker/metadata-action since only push event will generate tag for branch. But we want branch based cache.
Disable provenance to avoid caching manifest
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.
Thanks for tackling this! Just a question
.github/workflows/ci.yml
Outdated
@@ -5,15 +5,18 @@ name: build | |||
|
|||
on: | |||
push: | |||
branches: | |||
- develop | |||
branches: [develop, 'GEN*', 'gen*'] |
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 a comment to note that test
and lint
will run for every push now to the feature branches regardless of whether there is a PR or not. Nothing to fix here.
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.
Yes, right now, push to develop, branches with gen/GEN prefix, all PRs and release will trigger the test and lint job.
.github/workflows/ci.yml
Outdated
@@ -5,15 +5,18 @@ name: build | |||
|
|||
on: | |||
push: | |||
branches: | |||
- develop | |||
branches: [develop, 'GEN*', 'gen*'] |
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.
Actually could you also add main
to the branches here? I don't know if your current setup for ghcr will always create a docker image that is based on the latest version of the repo like we have for dockerhub? Will this setup create a latest
tag by default?
Add main branch to push event branches.
Quality Gate passedIssues Measures |
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.
LGTM!
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.
🔥 thanks for the great work! Great review
Problem:
There are limited admin seats that we can have on dockerHub.
Solution:
Switch to pushing our docker images to ghcr
Output:
The docker image and cached image can be viewed here.