Skip to content

Add commandline option to simulate alert queue expand failure - v4#7656

Closed
jufajardini wants to merge 2 commits intoOISF:masterfrom
jufajardini:alert-expand-fail-5319/v4
Closed

Add commandline option to simulate alert queue expand failure - v4#7656
jufajardini wants to merge 2 commits intoOISF:masterfrom
jufajardini:alert-expand-fail-5319/v4

Conversation

@jufajardini
Copy link
Contributor

@jufajardini jufajardini commented Jul 28, 2022

Previous PR: #7649

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/5319

Describe changes:

  • rebased
  • possibly fixed author name and email

@codecov
Copy link

codecov bot commented Jul 28, 2022

Codecov Report

Merging #7656 (4e8868e) into master (61f9f0d) will increase coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #7656      +/-   ##
==========================================
+ Coverage   75.78%   75.81%   +0.02%     
==========================================
  Files         659      659              
  Lines      185660   185662       +2     
==========================================
+ Hits       140706   140757      +51     
+ Misses      44954    44905      -49     
Flag Coverage Δ
fuzzcorpus 60.23% <0.00%> (+0.07%) ⬆️
suricata-verify 52.54% <ø> (+<0.01%) ⬆️
unittests 60.71% <ø> (-0.02%) ⬇️

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

@suricata-qa
Copy link

ERROR:

ERROR: QA failed on ips_afp_drop_chk.

Pipeline 8383

For testing purposes. Meant to simulate a reallocation failure when
dynamically growing the alert queue in DetectEngineThreadCtx, so we can
check that Suri's behavior doesn't break under such circumstances.

Task OISF#5319
@jufajardini jufajardini force-pushed the alert-expand-fail-5319/v4 branch from 5cf2927 to 4e8868e Compare July 29, 2022 13:24
@jufajardini
Copy link
Contributor Author

Force pushed sneaky command author info changes

@suricata-qa
Copy link

ERROR:

ERROR: QA failed on ips_afp_drop_chk.

Pipeline 8410

@victorjulien victorjulien mentioned this pull request Aug 1, 2022
@victorjulien
Copy link
Member

Merged in #7670, thanks!

@jufajardini jufajardini deleted the alert-expand-fail-5319/v4 branch August 2, 2022 12:15
jasonish added a commit to jasonish/suricata that referenced this pull request Apr 10, 2025
This one is a little different as it logs to a file, and is the same
fast.lua used in the new Suricata-Verify test.

Ticket: OISF#7656
jasonish added a commit to jasonish/suricata that referenced this pull request Apr 22, 2025
This one is a little different as it logs to a file, and is the same
fast.lua used in the new Suricata-Verify test.

Ticket: OISF#7656
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Apr 22, 2025
This one is a little different as it logs to a file, and is the same
fast.lua used in the new Suricata-Verify test.

Ticket: OISF#7656
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.

3 participants