-
Notifications
You must be signed in to change notification settings - Fork 38.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
remove ListMetaFor #43802
remove ListMetaFor #43802
Conversation
@k8s-bot non-cri e2e test this |
@deads2k Change looks like a nop and cleaner. However, I'm finding it hard to see how it helps us towards a manageable reststorage path. Is this part of a series of changes that help us towards that? |
Yeah, I'm push on our generic RESTStorage and finding places where it doesn't work on generic runtime.Objects. This is one of those fixes and I thought it could stand on its own. |
/lgtm Thanks for explaining the intent here. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, foxish
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
@deads2k Can you please fix the release-note label? |
(sorry, changed the wrong PR) |
Automatic merge from submit-queue (batch tested with PRs 44119, 42538, 43802, 42336, 43396) |
Finish replacing
ListMetaFor
withListAccessor
. This gives more flexibility for list handling as intended.UnstructuredList
matches now.@kubernetes/sig-api-machinery-pr-reviews