-
Notifications
You must be signed in to change notification settings - Fork 14.3k
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
Blog post for KEP-2896 and KEP-2885 #39844
Conversation
✅ Pull request preview available for checking
To edit notification comments on pull requests, go to your Netlify site settings. |
Hello, Comms Lead for the 1.27 release here. This feature blog is tracked for release, the deadline for submitting the draft is on 4th of April- the sooner the better since there's still editing to be done afterwards. Any doubt, the Comms team is here to help. |
Hello @Jefftree, we're doing a global reminder about submitting a draft for review for all opted-in feature blogs. If it's at all possible, it is very helpful for the release team to have drafts submitted for review before the hard deadline date, to better plan the release dates and avoid missing out. Thank you! |
@Jefftree |
@harshitasao Is there a guideline on how to set the date for the blog publishing date? |
/assign @apelisse |
Hello, Comms Lead for v1.27 here. The publication order and date for the Feature Blog series has been finalized and the tracking board is updated. The publication date for this article is 24-04-2023 (April 24). Thank you! |
content/en/blog/_posts/2023-xx-xx-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2023-xx-xx-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2023-xx-xx-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2023-xx-xx-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2023-xx-xx-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2023-xx-xx-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
|
||
kubectl will skip client side validation and will automatically use server side field validation in strict mode. Controllers by default use server side field validation in warn mode. | ||
|
||
With client side validation, we had to be extra lenient because some fields were missing from OpenAPI V2 and we didn’t want to reject possibly valid objects. This is all fixed in server side validation. Additional documentation may be found [here](https://kubernetes.io/docs/reference/using-api/api-concepts/#field-validation) |
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.
With client side validation, we had to be extra lenient because some fields were missing from OpenAPI V2 and we didn’t want to reject possibly valid objects. This is all fixed in server side validation. Additional documentation may be found [here](https://kubernetes.io/docs/reference/using-api/api-concepts/#field-validation) | |
With client side validation, we had to be extra lenient because some fields were missing | |
from OpenAPI V2 and we didn’t want to reject possibly valid objects. | |
This is all fixed in server side validation. Additional documentation may be found | |
[here](/reference/using-api/api-concepts/#field-validation) |
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.
Just want to confirm, this URL should be /docs/reference/using-api/api-concepts/#field-validation
instead of /reference/using-api/api-concepts/#field-validation
?
content/en/blog/_posts/2023-xx-xx-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2023-xx-xx-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2023-xx-xx-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
LGTM (after fixing comments), thanks for writing this! |
Thanks, @Jefftree is out right now but will be back in a few days and will make the updates. Thanks. |
content/en/blog/_posts/2023-xx-xx-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
/lgtm |
LGTM label has been added. Git tree hash: 7058d43f0ea13ae3715023536b76cf53f07ab4bc
|
/retest |
content/en/blog/_posts/2023-04-24-openapi-v3-field-validation-ga.md
Outdated
Show resolved
Hide resolved
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.
/lgtm
Some nits
|
||
#### Custom Resource Definitions | ||
|
||
In Kubernetes, Custom Resource Definitions use a structural OpenAPI V3 |
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)
In Kubernetes, Custom Resource Definitions use a structural OpenAPI V3 | |
In Kubernetes, CustomResourceDefinitions use a structural OpenAPI V3 |
Additional documentation may be found | ||
[here](/docs/reference/using-api/api-concepts/#field-validation) |
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)
Additional documentation may be found | |
[here](/docs/reference/using-api/api-concepts/#field-validation) | |
You can find [additional details](/docs/reference/using-api/api-concepts/#field-validation) | |
in the API Concepts reference page. |
Actually the final .
is the more important thing to add here, as missing it out cuts gravitas from the article.
|
||
## What's next? | ||
|
||
With Server Side Field Validation and OpenAPI V3 released as GA, we |
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) Clarify who “we” are in this paragraph.
|
||
## OpenAPI V3 | ||
|
||
### What does OpenAPI V3 offer over V2 |
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)
### What does OpenAPI V3 offer over V2 | |
### What does OpenAPI v3 offer over v2? |
OpenAPI V3 Golang client is available in | ||
[client-go](https://github.com/kubernetes/client-go/blob/release-1.27/openapi3/root.go). | ||
|
||
## Server Side Field Validation |
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)
## Server Side Field Validation | |
## Server side field validation |
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
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sftim 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 |
/lgtm |
LGTM label has been added. Git tree hash: e1511a1ddff0a887a96c95828d67a44372102219
|
Blog post for https://github.com/kubernetes/enhancements/blob/master/keps/sig-api-machinery/2896-openapi-v3/README.md and https://github.com/kubernetes/enhancements/blob/master/keps/sig-api-machinery/2885-server-side-unknown-field-validation/README.md
We will be combining them into one blog post.