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

MultilineParser achieve an effect similar to embedding by using anonymous structs #1133

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

onecer
Copy link
Contributor

@onecer onecer commented Apr 17, 2024

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #1131 #1132

Does this PR introduced a user-facing change?

None

MultilineParserSpec achieve an effect similar to embedding by using anonymous structs
add MultilineParser resource role in helm chart

Additional documentation, usage docs, etc.:

@benjaminhuo
Copy link
Member

Thanks @onecer , you may need to run:
make manifests generate docs-update

@onecer onecer force-pushed the fix_MultilineParserSpec branch from 5da0901 to 457200b Compare April 17, 2024 05:40
@onecer
Copy link
Contributor Author

onecer commented Apr 17, 2024

Thanks @onecer , you may need to run: make manifests generate docs-update

done

@benjaminhuo benjaminhuo merged commit 8c05b8d into fluent:master Apr 17, 2024
9 of 10 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.

multilineparserspec should be embedding structs
2 participants