Skip to content

📖 docs: add comprehensive XValidation CEL documentation and examples#1341

Open
camilamacedo86 wants to merge 1 commit intokubernetes-sigs:mainfrom
camilamacedo86:fix-cel
Open

📖 docs: add comprehensive XValidation CEL documentation and examples#1341
camilamacedo86 wants to merge 1 commit intokubernetes-sigs:mainfrom
camilamacedo86:fix-cel

Conversation

@camilamacedo86
Copy link
Member

Document the correct CEL variables (self/oldSelf) for CRD validation rules, add field-level documentation for all XValidation struct fields, and provide practical examples covering basic validation, custom reasons, immutability checks, and field path usage.

This clarifies the distinction between CRD validation CEL variables (self/oldSelf) and ValidatingAdmissionPolicy CEL variables (object/oldObject).

Generated-by: Cursor/Claude

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Feb 15, 2026
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: camilamacedo86
Once this PR has been reviewed and has the lgtm label, please assign sbueringer for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 15, 2026
@camilamacedo86
Copy link
Member Author

Hi @JoelSpeed

Thank you for the nice review.
Could you please give a look and see if that is now accurate?

@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 19, 2026
Document the correct CEL variables (self/oldSelf) for CRD validation rules,
add field-level documentation for all XValidation struct fields, and provide
practical examples covering basic validation, custom reasons, immutability
checks, and field path usage.

This clarifies the distinction between CRD validation CEL variables
(self/oldSelf) and ValidatingAdmissionPolicy CEL variables (object/oldObject).

Generated-by: Cursor/Claude
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants