Skip to content

Conversation

@kaitlynmichael
Copy link
Contributor

  • Replace brief option mentions with complete options table
  • Include all 12 available command-line options with descriptions
  • Add CLI configuration prerequisite step
  • Add RBAC requirements section
  • Follows same documentation pattern as other CLI tools

- Replace brief option mentions with complete options table
- Include all 12 available command-line options with descriptions
- Add CLI configuration prerequisite step
- Add RBAC requirements section
- Follows same documentation pattern as other CLI tools
@github-actions
Copy link
Contributor

@kaitlynmichael kaitlynmichael self-assigned this Jul 24, 2025
@kaitlynmichael kaitlynmichael marked this pull request as ready for review July 24, 2025 19:47
@kaitlynmichael kaitlynmichael requested a review from a team July 24, 2025 19:47
Copy link
Collaborator

@dwdougherty dwdougherty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one nitpick. Otherwise, LGTM. Approving.

@kaitlynmichael kaitlynmichael merged commit 55d1bdd into main Jul 24, 2025
5 checks passed
@kaitlynmichael kaitlynmichael deleted the DOC-5485-clean branch July 24, 2025 21:09
kaitlynmichael added a commit that referenced this pull request Jul 29, 2025
* K8s: Add comprehensive log collector options documentation

- Replace brief option mentions with complete options table
- Include all 12 available command-line options with descriptions
- Add CLI configuration prerequisite step
- Add RBAC requirements section
- Follows same documentation pattern as other CLI tools

* Apply suggestions from code review

* remove 6.2.18 mention

* Update content/operate/kubernetes/logs/collect-logs.md

Co-authored-by: David Dougherty <[email protected]>

---------

Co-authored-by: David Dougherty <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants