-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
af-packet: Remove support for rollover #9123
Conversation
This commit updates the summary message when Suricata terminates. Without this commit, "nan" was displayed if there were no drops/packets
Issue: 6128 cluster_rollover is no longer permitted; using it will generate a warning message and it'll be replaced with cluster_flow
Issue: 6128 No longer permit rollover/cluster_rollover to be used. Usage will generate a warning message and cluster_flow will be used instead.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #9123 +/- ##
==========================================
- Coverage 82.31% 82.30% -0.01%
==========================================
Files 969 969
Lines 273741 273740 -1
==========================================
- Hits 225329 225314 -15
- Misses 48412 48426 +14
Flags with carried forward coverage won't be shown. Click here to find out more. |
WARNING:
Pipeline 14954 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This still leaves the regular rollover
option. I think that should just turn into a no-op with a warning.
Continued in #9148 |
Continuation of #9069
This MR removes support for AF-Packet rollover.
Rollover support is defined here.
If
cluster-type: cluster_rollover
is configured, a warning message will be displayed and the clustering used will becluster_flow
.The warning message:
Link to redmine ticket: 6128
Describe changes:
cluster-type: cluster_rollover
) and usecluster_flow
insteadUpdates
rollover
handling per review feedback.Provide values to any of the below to override the defaults.
To use a pull request use a branch name like
pr/N
whereN
is the pull request number.