-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
KEP-2258: Promote NodeLogQuery to beta #45126
Conversation
👷 Deploy Preview for kubernetes-io-vnext-staging processing.
|
/milestone 1.30 |
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.
-
Do you also need to update
content/en/docs/reference/command-line-tools-reference/feature-gates/node-log-query.md
? -
These docs are only alpha quality; please update them in line with our style guide.
Specifically: avoid using “we” and make the table more accessible using a table shortcode with a caption. -
Don't call it an alpha feature
@@ -293,4 +293,4 @@ kubectl get --raw "/api/v1/nodes/node-1.example/proxy/logs/?query=kubelet&patter | |||
* Read about [Contextual Logging](https://github.com/kubernetes/enhancements/tree/master/keps/sig-instrumentation/3077-contextual-logging) | |||
* Read about [deprecation of klog flags](https://github.com/kubernetes/enhancements/tree/master/keps/sig-instrumentation/2845-deprecate-klog-specific-flags-in-k8s-components) | |||
* Read about the [Conventions for logging severity](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-instrumentation/logging.md) | |||
|
|||
* Read about [Log Query](https://github.com/kubernetes/enhancements/tree/master/keps/sig-windows/2258-node-log-query) |
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.
We link to https://kep.k8s.io/2256 instead.
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.
I assume you meant https://kep.k8s.io/2258. Fixed.
6049193
to
f5b5d6c
Compare
f5b5d6c
to
c3a6777
Compare
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 for the feedback, @sftim. I have addressed your comments to the best of my abilities. Please take a look.
@@ -293,4 +293,4 @@ kubectl get --raw "/api/v1/nodes/node-1.example/proxy/logs/?query=kubelet&patter | |||
* Read about [Contextual Logging](https://github.com/kubernetes/enhancements/tree/master/keps/sig-instrumentation/3077-contextual-logging) | |||
* Read about [deprecation of klog flags](https://github.com/kubernetes/enhancements/tree/master/keps/sig-instrumentation/2845-deprecate-klog-specific-flags-in-k8s-components) | |||
* Read about the [Conventions for logging severity](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-instrumentation/logging.md) | |||
|
|||
* Read about [Log Query](https://github.com/kubernetes/enhancements/tree/master/keps/sig-windows/2258-node-log-query) |
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.
I assume you meant https://kep.k8s.io/2258. Fixed.
c3a6777
to
5a55277
Compare
Hello @aravindhp 👋 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! |
@Princesso the PR is ready for review. Please let me know if I have missed something. |
/sig windows |
Hello @aravindhp 👋, it looks like you've already engaged feedback in review of this PR. Great job! Keep this up. 🚀 @marosset @LiGgit @thockin @marosset @Princesso Please help review this also. Here is a markdown document on the KEP and the kep.yaml which has information useful for review. You can also check a preview of the documentation with Netlify above. I appreciate everyone's help and so will our users! Thank you greatly. |
/assign @sftim. Hello Tim, can you give your final review on this PR? Hello @p0lyn0mial 👋, a friendly reminder that this PR needs a doc review by March 26th 2024 18:00 PT to get this into the release. Please reach out to the required SIGs to get their review. Thank you! |
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
/lgtm
/approve
LGTM label has been added. Git tree hash: bf7f9bc3eca189941977ba16406ca495aec236a7
|
[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 |
/milestone 1.30 |
No description provided.