-
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
Fix detect_sequence_reset flag in netflow module #24270
Fix detect_sequence_reset flag in netflow module #24270
Conversation
This flag was not passed to the input unless set to true, which is the default. It was impossible to turn it off. Fixes elastic#24268
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
/test |
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪💚 Flaky test reportTests succeeded. Expand to view the summary
Test stats 🧪
|
This flag was not passed to the input unless set to true, which is the default. It was impossible to turn it off. Fixes elastic#24268 (cherry picked from commit 22e20a1)
This flag was not passed to the input unless set to true, which is the default. It was impossible to turn it off. Fixes elastic#24268 (cherry picked from commit 22e20a1)
💔 Build Failed
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪Steps errors
Expand to view the steps failures
|
Test | Results |
---|---|
Failed | 0 |
Passed | 5375 |
Skipped | 738 |
Total | 6113 |
…stic#24459) This flag was not passed to the input unless set to true, which is the default. It was impossible to turn it off. Fixes elastic#24268 (cherry picked from commit 73b1fc1)
This flag was not passed to the input unless set to true, which is the
default. It was impossible to turn it off.
What does this PR do?
Fixes the netflow module so that it passes the value in
var.detect_sequence_reset
to the netflow input.Why is it important?
It was impossible to override the default for detect_sequence_reset (true).
Checklist
[ ] My code follows the style guidelines of this project[ ] I have commented my code, particularly in hard-to-understand areas[ ] I have made corresponding changes to the documentation[ ] I have made corresponding change to the default configuration files[ ] I have added tests that prove my fix is effective or that my feature worksCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Related issues