Skip to content
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

in_kubernetes_events: add k8s permission details #1396

Merged
merged 5 commits into from
Aug 16, 2024

Conversation

ryanohnemus
Copy link
Contributor

Documentation for fluent/fluent-bit#8351

fluent/fluent-bit#8351

Signed-off-by: ryanohnemus <ryanohnemus@gmail.com>
Copy link
Contributor

@lockewritesdocs lockewritesdocs left a comment

Choose a reason for hiding this comment

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

Thanks for submitting this change request! I suggested a few editorial changes for capitalization of product names and other consistency changes.

pipeline/inputs/kubernetes-events.md Outdated Show resolved Hide resolved
pipeline/inputs/kubernetes-events.md Outdated Show resolved Hide resolved
ryanohnemus and others added 2 commits June 21, 2024 15:55
Co-authored-by: Adam Locke <adam.locke@chronosphere.io>
Signed-off-by: Ryan Ohnemus <ryanohnemus@users.noreply.github.com>
Co-authored-by: Adam Locke <adam.locke@chronosphere.io>
Signed-off-by: Ryan Ohnemus <ryanohnemus@users.noreply.github.com>
Signed-off-by: Adam Locke <adam.locke@chronosphere.io>
Copy link
Contributor

@lockewritesdocs lockewritesdocs left a comment

Choose a reason for hiding this comment

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

Changes look good to me -- thanks for your submission! 🦖

@lecaros lecaros merged commit 3057b50 into fluent:master Aug 16, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants