-
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
Add commandline option to simulate alert queue expand failure - v3 #7622
Conversation
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
Information: QA ran without warnings. Pipeline 8208 |
Forgot to mention that the SV tests are supposed to fail, as we're still improving Suri behavior when the alert queue reallocation fails. |
So, should this be a draft PR ? |
As this is just an option to simulate a failure, I don't think this needs to be a draft. 🤔 |
Further discussion with @catenacyber indicates that the SV tests should not be linked in the PR. |
Could you edit the first message to remove the S-V PR reference and get CI running again ? |
It won't let go of the mentioned SV tests, now. Checks on new PR: #7649 |
Previous PR: #7527
Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/5319
Describe changes:
util-exception-policy
headers withinifdef DEBUG
This shows how to use it: OISF/suricata-verify#850