-
Notifications
You must be signed in to change notification settings - Fork 762
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
ci: push container images to ghcr.io as well #3658
ci: push container images to ghcr.io as well #3658
Conversation
d13fc9d
to
8a082f4
Compare
f29dedb
to
6148782
Compare
Signed-off-by: TakahiroTsuruda <[email protected]>
6148782
to
0804e02
Compare
.github/workflows/release.yaml
Outdated
- name: Build and push ghcr.io/${{ env.IMAGE_REPO }} | ||
if: steps.check-ghcr-image.outputs.exists == 'false' | ||
run: | | ||
make REPOSITORY=ghcr.io/${{ env.IMAGE_REPO }} docker-buildx-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.
can you integrate this into the existing release and pre-release flows so artifacts don't get rebuild and digests will be same
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.
Got it, I'll give it a try and update the PR.
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.
Signed-off-by: TakahiroTsuruda <[email protected]>
6a16cef
to
a7c5d95
Compare
Signed-off-by: Sertac Ozercan <[email protected]>
|
||
jobs: | ||
tagged-release: | ||
name: "Tagged Release" | ||
runs-on: "ubuntu-22.04" | ||
permissions: | ||
contents: write | ||
packages: write |
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.
@dulltz pushed a change to mainly move packages permission 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.
Thanks! LGTM
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3658 +/- ##
==========================================
- Coverage 54.49% 47.57% -6.93%
==========================================
Files 134 236 +102
Lines 12329 19786 +7457
==========================================
+ Hits 6719 9413 +2694
- Misses 5116 9487 +4371
- Partials 494 886 +392
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
What this PR does / why we need it:
Add GHCR as a destination container registry.
With this change we will be able to pull the container at the following URI:
Which issue(s) this PR fixes (optional, using
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when the PR gets merged):Fixes #3652
Special notes for your reviewer:
After merging this PR, container images will appear in Packages section.