docs(collector): add Kubernetes troubleshooting guidance#8884
docs(collector): add Kubernetes troubleshooting guidance#8884tiffany76 merged 5 commits intoopen-telemetry:mainfrom
Conversation
cc8af89 to
6147c8f
Compare
|
Thanks, @MUSKANNISHAD! @open-telemetry/collector-approvers and @atoulme, PTAL. |
|
While this information can indeed be useful, it is also overly generic. Attaching onto a running container is far from being otel-specific. |
b2f5a1d to
35206a1
Compare
|
Thanks for the feedback! That makes sense. I can update the section to make the OpenTelemetry-specific context clearer, or adjust the location if there’s a more appropriate place for this content. Happy to revise based on your preference. |
There was a problem hiding this comment.
I wonder if instead of adding that we can just add a link to https://kubernetes.io/docs/tasks/debug/debug-application/debug-running-pod/#ephemeral-container
I agree with @dmathieu that the info is overly generic, maybe adding that link is enough and we leave it to Kubernetes to document this
|
/fix:all |
|
✅ |
| ) | ||
| return loggerProvider, nil | ||
| } | ||
|
|
There was a problem hiding this comment.
Can you undo this unrelated change? Otherwise LGTM
There was a problem hiding this comment.
Done — the unrelated Go getting-started change has been reverted.
Thanks for catching it!
There was a problem hiding this comment.
Thanks for the review and the guidance. I’ve reduced the Kubernetes section to a minimal, Collector-specific note, fixed the formatting issues. Please let me know if this looks good now.
There was a problem hiding this comment.
The line is still there, please take another look.
@MUSKANNISHAD I also wanted to ask you if you are using an LLM to write the comments on this PR. Is that the case?
There was a problem hiding this comment.
Hi @MUSKANNISHAD - did you have a chance to see @mx-psi's comment above?
Also, I've updated the PR description to include the questions from our latest template. Could you please fill it out and check the boxes to indicate if you've used any AI-generated content?
Thanks!
a25a670 to
96c0437
Compare
96c0437 to
e1b96f9
Compare
|
/fix:format |
|
✅ |
|
Thanks, @MUSKANNISHAD. As the Collector docs refactoring is moving forward, we need to clean up some of the outstanding PRs to avoid conflicts. I made one additional change, which I think now addresses all the concerns of the Collector folks. I'm going to merge. If there are any lingering issues I missed, comment here and I'll raise follow-up issues. Thanks. |
|
Thank you for your contribution @MUSKANNISHAD! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. |
This PR adds Kubernetes-specific troubleshooting guidance for the OpenTelemetry
Collector, including an example using
kubectl debugwith an ephemeralcontainer to diagnose networking and runtime issues.
Fixes #6129
Footnotes
Yes, I can answer maintainer questions about the content of this PR, without using AI. ↩