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 feature CustomResourceValidationExpressions to beta #111158
Conversation
Skipping CI for Draft Pull Request. |
/sig api-machinery |
Don't forget to update pkg/features/kube_features.go too |
/cc |
updated the description to a bulleted list of the PRs, so we can see they all merged before merging this one |
The e2e test associated with this feature will be handled in separate PR: #111448 |
I'm diffing against the list in https://github.com/kubernetes/enhancements/blob/master/keps/sig-api-machinery/2876-crd-validation-expression-language/README.md#beta I'm assessing "Demonstrate adoption and successful feature usage in the community" based on use-cases contained in https://docs.google.com/document/d/1KatwZJ3O0lTThs1Lg8YqY-PipbNM8aaUDDdPCLd3lic/edit?resourcekey=0-YeuQhCZlstVjL61WVqrz_w# that have specific reference and relating them back to the unit tests that ensure they work. I'll also come back to the rest of the criteria and check them as well. |
note to self, int-or-string evaluation is confirmed here: kubernetes/staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/schema/cel/validation_test.go Lines 793 to 842 in 631a5a8
|
/lgtm holding for the comment update. Feel free to re-lgtm and unhold after that. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cici37, deads2k 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 |
New changes are detected. LGTM label has been removed. |
@cici37: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. I understand the commands that are listed here. |
The |
Fix for |
This PR got accidentally closed and cannot be reopen due to a force-push. Open #111524 as a duplication |
What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR is to promote feature CustomResourceValidationExpressions to beta as it's proposed in KEP.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
The feature is in alpha since v1.23. For v1.25 cycle, we focus on performance optimization, test coverage, bug fix, metrics adding, dependency bumping, etc. The related PR FYI:
Graduation to beta criteria from the KEP
kubernetes/staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/schema/cel/validation_test.go
Lines 1701 to 1753 in 631a5a8
kubernetes/staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/schema/cel/validation_test.go
Lines 793 to 842 in 631a5a8
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: