docs: Improve information about how each daemonset pod watches ALL pods logs in the cluster#5359
Merged
clayton-cornell merged 8 commits intomainfrom Feb 5, 2026
Merged
Conversation
Contributor
|
💻 Deploy preview available (Improve information about how each daemonset pod watches ALL pods logs in the cluster): |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR improves the discovery.kubernetes documentation with a focus on clarifying DaemonSet deployment behavior, language clarity, and readability. The changes address issue #4793 by adding prominent admonitions about performance considerations when running in DaemonSet mode.
Changes:
- Added a caution admonition highlighting that each DaemonSet Pod watches all cluster resources by default, which can cause significant API server load
- General documentation cleanup including grammar improvements, active voice conversion, and better readability
- Fixed typos and improved consistency in formatting (bullet lists, table formatting)
Contributor
|
💻 Deploy preview deleted (docs: Improve information about how each daemonset pod watches ALL pods logs in the cluster). |
blewis12
approved these changes
Feb 3, 2026
Member
blewis12
left a comment
There was a problem hiding this comment.
lgtm - thank you including the caution regarding daemonsets, this has come up in escalations i've seen already
clayton-cornell
added a commit
that referenced
this pull request
Feb 5, 2026
…ds logs in the cluster [backport] (#5452) ## Backport of #5359 This PR backports #5359 to release/v1.13. ### Original PR Title docs: Improve information about how each daemonset pod watches ALL pods logs in the cluster ### Original PR Author @clayton-cornell ### Description Fix admonitions around info explaining that each daemonset pod watches ALL pods logs in the cluster. General topic cleanup, with a focus on language, readability, active voice, and grammar. Fixes #4793 --- *This backport was created automatically.* Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
clayton-cornell
added a commit
that referenced
this pull request
Feb 5, 2026
…ds logs in the cluster [backport] (#5451) ## Backport of #5359 This PR backports #5359 to release/v1.12. ### Original PR Author @clayton-cornell ### Description Fix admonitions around info explaining that each daemonset pod watches ALL pods logs in the cluster. General topic cleanup, with a focus on language, readability, active voice, and grammar. Fixes #4793 --- *This backport was created automatically.* Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix admonitions around info explaining that each daemonset pod watches ALL pods logs in the cluster.
General topic cleanup, with a focus on language, readability, active voice, and grammar.
Fixes #4793