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
WIP: Document KEP 3157 (watch-list) for milestone 1.30 #45283
WIP: Document KEP 3157 (watch-list) for milestone 1.30 #45283
Conversation
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
--- | ||
Enable support for [streaming initial state of objects in watch requests](/docs/reference/using-api/api-concepts/#streaming-lists). | ||
Enable support for [streaming initial state of objects in watch requests](/content/en/docs/reference/using-api/api-concepts.md/#streaming-lists). |
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.
the link was broken.
7f434cd
to
4c986ed
Compare
4c986ed
to
db02e3a
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
👷 Deploy Preview for kubernetes-io-vnext-staging processing.
|
--- | ||
Enable support for [streaming initial state of objects in watch requests](/docs/reference/using-api/api-concepts/#streaming-lists). | ||
Enable support for [streaming initial state of objects in watch requests](/content/en/docs/reference/using-api/api-concepts.md/#streaming-lists). |
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.
The suggested link update seems unnecessary and incorrect. The existing link works as expected when rendered on the website. I'm not sure if this update is needed.
Please check it out here: https://kubernetes.io/docs/reference/command-line-tools-reference/feature-gates/ (check "WatchList" under the "List of feature gates" section).
The final redirection location is https://kubernetes.io/docs/reference/using-api/api-concepts/#streaming-lists.
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.
Enable support for [streaming initial state of objects in watch requests](/content/en/docs/reference/using-api/api-concepts.md/#streaming-lists). | |
Enable support for [streaming initial state of objects in watch requests](/docs/reference/using-api/api-concepts/#streaming-lists). |
/remove-language zh |
@@ -9,5 +9,8 @@ stages: | |||
- stage: alpha | |||
defaultValue: false | |||
fromVersion: "1.27" |
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.
fromVersion: "1.27" | |
fromVersion: "1.27" | |
toVersion: "1.29" |
The website does not work out toVersion
for you, you need to be explicit.
Hello @p0lyn0mial 👋 please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday March 12th 2024 18:00 PST. Thank you! |
/hold /sig api-machinery |
The streaming feature didn't make it to 1.30. Hopefully, the feature will be completed in the next release, and I will be able to update this PR. |
/remove-area blog I'll close the pull request. You are very welcome to open a new PR against the development branch for a future release (and to reuse the work you did here). /close |
@sftim: Closed this PR. In response to this:
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. |
Heads up that |
TODO: Document
WatchListClient
as well.