Skip to content

Update documentation with filterGrep option for fluentbit #285

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

Merged
merged 1 commit into from
Apr 24, 2025

Conversation

joseacl
Copy link

@joseacl joseacl commented Apr 18, 2025

Update documentation with filterGrep option for fluentbit, related with kube-logging/logging-operator#2012.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Fluent Bit documentation to include details about the new filterGrep option.

  • Adds a filterGrep configuration snippet within the fluentbit.md example.
  • Updates CRD documentation in fluentbit_types.md to define filterGrep parameters and refines some existing documentation text.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
content/docs/logging-infrastructure/fluentbit.md Added filterGrep configuration example to the Fluent Bit setup snippet.
content/docs/configuration/crds/v1beta1/fluentbit_types.md Introduced filterGrep CRD documentation and adjusted formatting in related sections.
Comments suppressed due to low confidence (1)

content/docs/logging-infrastructure/fluentbit.md:129

  • The regex pattern in the filterGrep configuration may lead to YAML parsing issues due to the unquoted special characters. Consider enclosing the regex pattern in quotes to ensure it is interpreted correctly.
      - message ^."level"\:"(debug|DEBUG)".$

Copy link
Member

@csatib02 csatib02 left a comment

Choose a reason for hiding this comment

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

LGTM!

Let's wait for kube-logging/logging-operator#2012 before merging!

@csatib02 csatib02 merged commit c1aad42 into kube-logging:master Apr 24, 2025
2 checks passed
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.

2 participants