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
Vendor k-sigs/controller-runtime@v0.5.10 #2797
Vendor k-sigs/controller-runtime@v0.5.10 #2797
Conversation
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.
Thank you for this PR!
I see that with kubernetes-sigs/controller-runtime#1061 there is already an Or
predicate. I guess we can now remove our custom or predicate (https://github.com/gardener/gardener/blob/v1.9.0/extensions/pkg/predicate/predicate.go#L92-L123) and use the one from upstream, right?
Yes, indeed. Originally I wanted to do it seperately, but now that you also mentioned it, I just did it in this PR. Rather than removing our |
Makes sense. Thanks. |
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
How to categorize this PR?
/area open-source
/kind task
/priority normal
What this PR does / why we need it:
Vendor k-sigs/controller-runtime@v0.5.10, which contains several bug fixes.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
v0.6 will be vendored as part of #2489, see @timuthy's progress.
Release note: