-
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
Add wired off code for Validation rules for Custom Resource Definitions using the CEL expression language #106234
Conversation
/priority important-soon |
/kind feature |
Unknown CLA label state. Rechecking for CLA labels. Send feedback to sig-contributor-experience at kubernetes/community. /check-cla |
staging/src/k8s.io/apiextensions-apiserver/third_party/forked/celopenapi/model/README.md
Show resolved
Hide resolved
staging/src/k8s.io/apiextensions-apiserver/third_party/forked/celopenapi/model/README.md
Show resolved
Hide resolved
@@ -17,6 +17,7 @@ limitations under the License. | |||
package validation | |||
|
|||
import ( | |||
"k8s.io/apiextensions-apiserver/pkg/apiserver/schema" |
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.
nit: group with below imports
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.
still outstanding?
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.
oops, fixed
staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/validation/validation.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/validation/validation.go
Show resolved
Hide resolved
7d5bd55
to
b41290c
Compare
Thanks for the review @liggitt, feedback applied. |
staging/src/k8s.io/apiextensions-apiserver/third_party/forked/celopenapi/model/escaping.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/apiextensions-apiserver/third_party/forked/celopenapi/model/schemas.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/apiextensions-apiserver/third_party/forked/celopenapi/model/schemas.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/apiextensions-apiserver/third_party/forked/celopenapi/model/schemas.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/apiextensions-apiserver/third_party/forked/celopenapi/model/escaping_test.go
Show resolved
Hide resolved
staging/src/k8s.io/apiextensions-apiserver/third_party/forked/celopenapi/model/schemas.go
Show resolved
Hide resolved
{
- "directDependencies": 163,
- "transitiveDependencies": 300,
- "totalDependencies": 359,
- "maxDepthOfDependencies": 29
+ "directDependencies": 164,
+ "transitiveDependencies": 306,
+ "totalDependencies": 366,
+ "maxDepthOfDependencies": 30
}
|
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jpbetz, 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 |
This adds the dependencies and third_party/forked code import that is needed for #106051.
This PR purely intended to simplify the code review of #106051. Once this PR merges, #106051 will have about 6k lines to review instead of 50k.
/kind feature
/sig api-machinery
cc @cici37 @sttts @liggitt