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

Custom MULTILINE_PARSER support #1028

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Custom MULTILINE_PARSER support #1028

wants to merge 1 commit into from

Conversation

elnurm
Copy link

@elnurm elnurm commented Nov 28, 2023

Custom MULTILINE_PARSER

The current implementation of the AWS Fluent Bit Helm chart lacks support for custom MULTILINE_PARSER configurations in the extraParsers section of the values.yaml file. When attempting to define a custom multiline parser in this section, an error is encountered, specifically: [multiline_parser] multiline_logs not registered.

It's possible that there might be nuances or specific configurations required to enable the MULTILINE_PARSER functionality within the extraParsers section that I might not be aware of, if so please correct me.

Description of changes

I added support for the MULTILINE_PARSER in the extraParsers section of the values.yaml file in the AWS Fluent Bit Helm chart. This enhancement allowed me to define custom multiline parsers.

Checklist

  • Added/modified documentation as required (such as the README.md for modified charts)
  • Incremented the chart version in Chart.yaml for the modified chart(s)
  • Manually tested. Describe what testing was done in the testing section below
  • Make sure the title of the PR is a good description that can go into the release notes

Testing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@cdhesse
Copy link

cdhesse commented Dec 15, 2023

I also need support for custom multiline parsers.

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