-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Image policy is resolving images on replica sets by default #15867
Image policy is resolving images on replica sets by default #15867
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: smarterclayton The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
6e8b9d7
to
c2613d2
Compare
/retest |
@smarterclayton pls update this PR with changes from #15868 so we can ship it |
It is |
/retest |
// value is DoNotAttempt. This overrides the global image policy for a matching resource. If there | ||
// are multiple rules matching a resource, any rule that requests resolution or rewrite will | ||
// result in rewrite. | ||
Type ImageResolutionType |
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.
@smarterclayton isn't this duplicate to Policy
? seems like rebase going wrong
c2613d2
to
66fe69b
Compare
Even when local names are not requested, the image policy plugin is deciding to rewrite image references in replica sets that point to the integrated registry (with tags) to use digests. This causes the deployment controller that created them to get wedged (because it detects a change to the template) and become unable to update status on that replica set. https://bugzilla.redhat.com/show_bug.cgi?id=1481801
66fe69b
to
4f99fe7
Compare
Updated |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest |
/test all [submit-queue is verifying that this PR is safe to merge] |
/retest Please review the full test history for this PR and help us cut down flakes. |
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue |
Even when local names are not requested, the image policy plugin is
deciding to rewrite image references in replica sets that point to the
integrated registry (with tags) to use digests. This causes the
deployment controller that created them to get wedged (because it
detects a change to the template) and become unable to update status on
that replica set.
https://bugzilla.redhat.com/show_bug.cgi?id=1481801