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

userguide: document packet alert queue - v1 #7212

Closed
wants to merge 3 commits into from

Conversation

jufajardini
Copy link
Contributor

This work goes with

Link to redmine ticket:

Describe changes:

  • Add a section about packet alert max config
  • Add subsection about packet alert queue behavior (especially when an overflow happens)
  • Add subsection about discarded alerts in case of packet alert queue overflow

Added a subsection along packet-alert-max config section explaining
packet alert queue overflow (when Suri reached packet alert max), how
are alerts replaced/discarded etc.

Task OISF#5178
@jufajardini jufajardini requested a review from norg as a code owner April 6, 2022 20:35
@codecov
Copy link

codecov bot commented Apr 6, 2022

Codecov Report

Merging #7212 (38ed975) into master (2b5eeab) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7212      +/-   ##
==========================================
- Coverage   78.14%   78.05%   -0.10%     
==========================================
  Files         628      628              
  Lines      184586   184585       -1     
==========================================
- Hits       144247   144069     -178     
- Misses      40339    40516     +177     
Flag Coverage Δ
fuzzcorpus 60.22% <ø> (-0.25%) ⬇️
suricata-verify 54.65% <ø> (-0.02%) ⬇️
unittests 62.95% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@jufajardini
Copy link
Contributor Author

Brought into #7343

@jufajardini jufajardini deleted the packet-alert-doc/v1 branch May 2, 2022 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant