-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[filebeat/netflow]: set event normalisation to true #38780
[filebeat/netflow]: set event normalisation to true #38780
Conversation
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
This pull request is now in conflicts. Could you fix it? 🙏
|
db5cd03
to
a3d70d5
Compare
Pinging @elastic/sec-deployment-and-devices (Team:Security-Deployment and Devices) |
* fix(input/netflow): revert event normalisation to true for netflow input * doc: update CHANGELOG.next.asciidoc * fix: update fields.asciidoc (unrelated to this work) * doc: remove irrelevant comment (cherry picked from commit 2f5ee74)
…o true (#38802) * [filebeat/netflow]: set event normalisation to true (#38780) * fix(input/netflow): revert event normalisation to true for netflow input * doc: update CHANGELOG.next.asciidoc * fix: update fields.asciidoc (unrelated to this work) * doc: remove irrelevant comment (cherry picked from commit 2f5ee74) * doc: remove irrelevant changes from CHANGELOG.next.asciidoc --------- Co-authored-by: Panos Koutsovasilis <[email protected]>
Proposed commit message
This PR enables event normalisation for netflow input and solves this this issue. Specifically the netflow input in 8.13 had a refactor to transition to plugin.V2 and during this it was deemed as safe to disable event normalisation and thus improve the performance. However the produced events contain ip fields that cause similar errors to this following without normalisation enabled
Checklist
I have added tests that prove my fix is effective or that my feature worksCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
N/A
How to test this PR locally
Follow the steps in the issue #38703
Related issues
Use cases
Screenshots
N/A
Logs
N/A