-
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
Docs for OpenAPI v3 #30534
Docs for OpenAPI v3 #30534
Conversation
👷 Deploy Preview for kubernetes-io-vnext-staging processing. 🔨 Explore the source changes: 4154709 🔍 Inspect the deploy log: https://app.netlify.com/sites/kubernetes-io-vnext-staging/deploys/61a663f95b412b00098158fc |
/assign |
@Jefftree Reminder that tomorrow Nov 23rd is the deadline for 1.23 Docs Ready for Review. Please push your docs content for this enhancement and remove the work-in-progress label when ready for review. We have a tight turnaround to getting docs reviewed and approved for the release, and we're hoping to have all docs in for review by EOB tomorrow. Thank you! |
/sig api-machinery |
@Jefftree would you be willing to squash commits? One of your commit messages is “Placeholder”. |
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.
This'll need tech review from SIG API Machinery (which, I hope, will be a formality only).
Also I spotted some potential improvements.
@Jefftree The 'docs ready' deadline for 1.23 is tomorrow Nov 30. Can you please check out the recent comments from Tim, as well as land a tech reviewer for this PR? Thanks! |
LGTM for SIG Docs @kubernetes/sig-api-machinery-pr-reviews happy with technical accuracy for this PR? |
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.
Small clarification, but lgtm otherwise.
</tbody> | ||
</table> | ||
|
||
A discovery endpoint `/openapi/v3` is provided to see a list of all |
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.
is this accepting the same content-type as above? Or only json?
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! Updated to state that it only returns JSON.
/lgtm |
LGTM label has been added. Git tree hash: 2c24c5017260472fdbd0f7e8066729e8ae665ab7
|
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jlbutler 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 |
Add section for OpenAPI V3