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
Promote apiserver.config.k8s.io/v1, kind=ResourceQuotaConfiguration #85099
Conversation
3e457d1
to
11416a4
Compare
/retest |
11416a4
to
ac6eb7e
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: liggitt 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 |
ac6eb7e
to
c650f69
Compare
c650f69
to
7d3012f
Compare
/sig api-machinery related to #76310 |
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 @liggitt for working on this :)
/lgtm
/hold for @deads2k ack |
}}, | ||
}, | ||
{ | ||
name: "valid v1 config", |
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.
thanks
Since this is logically generic, moving the group makes sense. It does raise some questions about introducing a v2, but sparse versions should work. /lgtm |
/retest |
the pull-kubernetes-node-e2e-containerd failure is known, not a flake, non-blocking, and being worked on - #85040 |
/retest Review the full test history for this PR. Silence the bot with an |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
resourcequota.admission.k8s.io/v1beta1, kind=Configuration
toapiserver.config.k8s.io/v1, kind=ResourceQuotaConfiguration
apiserver.config.k8s.io
group for generic API server configuration was determined after the configuration type was introducedConfiguration
suffix is used consistently for top-level configuration typesDoes this PR introduce a user-facing change?:
/cc @derekwaynecarr @deads2k @ahg-g @ravisantoshgudimetla