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
Graduate APIListChunking to GA #119503
Graduate APIListChunking to GA #119503
Conversation
/hold Wait for kubernetes/enhancements#3274 to be approved first. |
c231a25
to
cdce100
Compare
/retest |
1 similar comment
/retest |
The only thing I wanted to point out before LGTM is that the conformance test randomizes the "limit" used to paginate. Are we OK with having a non-deterministic conformance test like this? When run repeatedly it will find various edge cases, but it won't find them on each run. |
It's a fair question - I think we should avoid non-determinism. I just changed the test to be deterministic now. PTAL |
/lgtm Thank you! |
LGTM label has been added. Git tree hash: b8813e518ddbef102262b28d7a52f094bebc4dc1
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jpbetz, wojtek-t 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 |
Please update the docs around this for v1.29 |
Yup - on my list. Just we have quite a bit of time still for it. |
Opened kubernetes/website#42800 |
Ref kubernetes/enhancements#365
Ref #108003
/sig api-machinery
/kind feature
/priority important-longterm