-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Kubelet OpenTelemetry Tracing #2831
Comments
@sallyom can you please add the mandatory discussion link to this issue? |
/sig node instrumentation |
/assign |
/cc |
2 similar comments
/cc |
/cc |
Hello @sallyom v1.24 Enhancements team here. Just checking in as we approach enhancements freeze on 18:00pm PT on Thursday Feb 3rd, 2022. This enhancement is targeting Here’s where this enhancement currently stands:
The status of this enhancement is marked as |
@encodeflush thanks for the check-in - |
@sallyom Thanks for the update. Those criterion should be fulfilled form your side. From enhancements team point of view, we go through them and check if your KEP meets the requirements. |
Hi @sallyom 👋 1.24 Docs shadow here. This enhancement is marked as 'Needs Docs' for the 1.24 release. Please follow the steps detailed in the documentation to open a PR against the dev-1.24 branch in the k/website repo. This PR can be just a placeholder at this time and must be created before Thu March 31, 11:59 PM PDT. Also, if needed take a look at Documenting for a release to familiarize yourself with the docs requirement for the release. Thanks! |
Hi @sallyom 👋 1.24 Release Comms team here. We have an opt-in process for the feature blog delivery. If you would like to publish a feature blog for this issue in this cycle, then please opt in on this tracking sheet. The deadline for submissions and the feature blog freeze is scheduled for 01:00 UTC Wednesday 23rd March 2022 / 18:00 PDT Tuesday 22nd March 2022. Other important dates for delivery and review are listed here: https://github.com/kubernetes/sig-release/tree/master/releases/release-1.24#timeline. For reference, here is the blog for 1.23. Please feel free to reach out any time to me or on the #release-comms channel with questions or comments. Thanks! |
Hello @sallyom I'm just checking in once more as we approach the 1.24 Code Freeze on 18:00 PDT, Tuesday, March 29th 2022 Please ensure the following items are completed:
For note, the status of this enhancement is currently marked as Could you please confirm if following PR is part of the implementation for this enhancement?
Kindly please let me know if I'm missing any related PRs other than the ones I linked above. Thank you so much! |
#3087 is not directly related to the overall OpenTelemetry enhancements. |
kubernetes/kubernetes#105126 is the initial implementation for this enhancement |
Hey @sallyom, can you please add a placeholder PR for this feature in https://github.com/kubernetes/website by March 30, that is our extended deadline for feature blog freeze? Example of a feature blog PR: kubernetes/website#30538 |
Hi, 1.24 Enhancements Lead here 👋. With code freeze now in effect, this enhancement has not met the criteria for the freeze and has been removed from the milestone. As a reminder, the criteria for code freeze is: All PRs to the kubernetes/kubernetes repo have merged by the code freeze deadline Thanks! |
I went through the kubelet codebase, and found the following places we could potentially add tracing:
I don't think any of those should block GA. The only remaining step is to graduate the configuration resource (staging/src/k8s.io/apiserver/pkg/apis/apiserver/v1beta1/types.go). |
I agree. Would that also mean that we graduate to GA in the first place and are free to add tracing later on per demand? |
Yes, I think that is the right approach. |
Hello @somalley, @dashpole👋, v1.33 Enhancements team here. Just checking in as we approach enhancements freeze on 02:00 UTC Friday 14th February 2025 / 19:00 PDT Thursday 13th February 2025. This enhancement is targeting stage Here's where this enhancement currently stands:
For this KEP, we would just need to update the following:
The status of this enhancement is marked as If you anticipate missing enhancements freeze, you can file an exception request in advance. Thank you! |
@dashpole howdy! do we want to move this forward in 1.33? |
Yes, I plan to graduate this to stable in 1.33 |
Hi @somalley, @dashpole 👋, 1.33 Enhancements team here, Just a quick friendly reminder as we approach the enhancements freeze later this week, at 02:00 UTC Friday 14th February 2025 / 19:00 PDT Thursday 13th February 2025. The current status of this enhancement is marked as If you anticipate missing enhancements freeze, you can file an exception request in advance. Thank you! |
Hello @somalley, @dashpole 👋, 1.33 Enhancements team here, It looks like PR #5134 covers most of the updates we need for inclusion of this KEP in v1.33. To ensure it’s tracked for the release, we also need to ensure KEP README.md readme using the latest template. Currently I can see in the README the below question under Troubleshooting section is missing.
The current status of this enhancement is still marked as If you anticipate missing enhancements freeze, you can file an exception request in advance. Thank you! |
@somalley @dashpole Now that PRs #5170 #5134 (please do mention both PRs in issue description too) have been merged, all the KEP requirements are in place and merged into k/enhancements, this enhancement is all good for the upcoming enhancements freeze. 🚀 The status of this enhancement is marked as (cc: @ArkaSaha30) /label tracked/yes |
Hello @dashpole and @logicalhan 👋, v1.33 Docs Shadow here. |
Enhancement Description
k/enhancements
) update PR(s):k/k
) update PR(s):k/website
) update PR(s):k/enhancements
) update PR(s):k/k
) update PR(s):KubeletTracing
feature to beta kubernetes#115750k/website
) update(s):k/enhancements
) update PR(s): KEP-2831: update kubelet tracing KEP to target GA in 1.33 #5134k/k
) update PR(s):k/website
) update(s):Please keep this description up to date. This will help the Enhancement Team to track the evolution of the enhancement efficiently.
The text was updated successfully, but these errors were encountered: