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
kube-controller-manager: unit tests WatchListClient feature for client-go #122608
kube-controller-manager: unit tests WatchListClient feature for client-go #122608
Conversation
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
a034fd2
to
f0298b3
Compare
f0298b3
to
d478c15
Compare
d478c15
to
5d3fdc9
Compare
Just additional tests for #122738 ? No objection, but next time you could get Ben to include them. |
…te for command line options
5d3fdc9
to
8668c7f
Compare
Yes, just additional tests, no problem. Although, I prefer to open smaller PRs for easier review. |
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
/approve
LGTM label has been added. Git tree hash: a22d4a605396d5048626cbd8f8e96e9c1f4c680e
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: p0lyn0mial, soltysh The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/triage accepted |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Tests whether the
WatchListClient
(defined in client-go) is exposed as a command line option forKCM
with its default value set tofalse
.We specificity test for
KCM
because this will be the only component using the watch-list feature for this release (targeted for1.30
).Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
requires:
xref: kubernetes/enhancements#3157
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: