-
Notifications
You must be signed in to change notification settings - Fork 109
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
feat: use scarf.sh registry for all container images #507
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #507 +/- ##
==========================================
+ Coverage 48.84% 48.90% +0.05%
==========================================
Files 55 55
Lines 5143 5143
==========================================
+ Hits 2512 2515 +3
+ Misses 2525 2523 -2
+ Partials 106 105 -1
Flags with carried forward coverage won't be shown. Click here to find out more. |
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
0959360
to
546770f
Compare
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Kudos, SonarCloud Quality Gate passed!
|
Any experience with the scarf service, as it might "break" the download, if not working correctly, can scarf cope with the reliability of the typical public registries? |
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
@hwinkel good question, we don't have any meaningful experience yet, but we aim to get it by trying scarf out for now. |
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.
should we change also this: https://github.com/keptn/lifecycle-toolkit/blob/main/.github/workflows/release.yml#L57 ?
thanks for the feeedback.. as we are looking for a similar service |
@thisthat that occurrence should stay as it is. We still push to ghcr directly without going though scarf, therefore also the image tag needs to still point to ghcr. |
This PR
THIS IS A BREAKING CHANGE
Part of #195