Skip to content

K8s: add log collector options #1871

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

Closed
wants to merge 9 commits into from
Closed

K8s: add log collector options #1871

wants to merge 9 commits into from

Conversation

kaitlynmichael
Copy link
Contributor

@kaitlynmichael kaitlynmichael requested a review from a team July 24, 2025 17:53
@kaitlynmichael kaitlynmichael self-assigned this Jul 24, 2025
@@ -25,17 +25,31 @@ As of version 6.2.18-3, the log collector tool has two modes:
1. Download the latest [`log_collector.py`](https://github.com/RedisLabs/redis-enterprise-k8s-docs/blob/master/log_collector/log_collector.py) file.

1. Have a K8s administrator run the script on the system that runs your `kubectl` or `oc` commands.
Copy link
Contributor

Choose a reason for hiding this comment

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

It could be useful to also mention that the k8s CLI (kubectl / oc) needs to be configured to access the k8s cluster to be collected.

Copy link
Contributor

Choose a reason for hiding this comment

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

Can also be useful to mention the required RBAC permissions for the user executing the log collector?

Copy link
Contributor

Choose a reason for hiding this comment

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

Can also be useful to mention the required RBAC permissions for the user executing the log collector?

Actually we have DOC-3998 for this.

- Added comprehensive log collector RBAC page with restricted and all mode configurations
- Updated logs section index to include log collector RBAC documentation
- Added direct links from collect-logs.md to specific YAML examples
- Configured proper navigation order (collect logs before RBAC)
- Each YAML example is in its own linkable subsection for direct referencing
- Includes complete RBAC configurations using existing embed files from content/embeds/k8s/
- Provides security considerations and troubleshooting guidance
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.

Language LGTM.

kaitlynmichael and others added 6 commits July 24, 2025 14:29
* SITE: enable use of 'latest' tag for TCEs (part 1 of 2)

* Apply a suggestion from issue #1694.

* A bit more cleanup.

* Final tweaks.

* Update description in update_cmds.py
* DEV: update RQE tags doc

* Apply suggestions from peer review
* remvoe 7.2.8-6 mentions

* revise index pages

* cleanup

* reference custom resource info
@kaitlynmichael
Copy link
Contributor Author

kaitlynmichael commented Jul 24, 2025

Augment made this branch messy, creating a new clean one (#1873)

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