-
Notifications
You must be signed in to change notification settings - Fork 167
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
fix(controller): make tag selector ignore references in an oci image index that don't look like they refer to an image #1268
fix(controller): make tag selector ignore references in an oci image index that don't look like they refer to an image #1268
Conversation
✅ Deploy Preview for docs-kargo-akuity-io ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1268 +/- ##
=======================================
Coverage 48.50% 48.50%
=======================================
Files 127 127
Lines 9675 9683 +8
=======================================
+ Hits 4693 4697 +4
- Misses 4823 4826 +3
- Partials 159 160 +1 ☔ View full report in Codecov by Sentry. |
const guestbookRepo = "ghcr.io/jessesuen/guestbook" | ||
const guestbookTag = "v0.0.47" |
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.
Hmm, I'd like the freedom to delete this repo without breaking tests. I've repushed this multi-arch image as: ghcr.io/akuity/kargo-test:unknown
. Can we change this to use that?
https://github.com/orgs/akuity/packages/container/kargo-test/156938778?tag=unknown
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.
Pre-emptive approval. As discussed, you can use ghcr.io/akuity/kargo-test:unknown
for testing the edge case.
Signed-off-by: Kent <kent.rancourt@gmail.com>
4ddadf4
to
8bd32c6
Compare
Fixes #1267