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

Coledishington/flow force reuse 6293 v7 #2021

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

catenacyber
Copy link
Collaborator

Add test for verifying that if the memcap is reached, alive flows will be reused with flow.force-reuse on and not be re-used with flow.force-reuse off.

Redmine ticket:
https://redmine.openinfosecfoundation.org/issues/6293

#1699 with additional commit to run only on arch x86_64

pcap: ../test.pcap

args:
- --set flow.memcap-policy=drop-flow
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this matter in this case?

Copy link
Member

@inashivb inashivb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good. Minor comments inline

coledishington and others added 3 commits September 18, 2024 14:58
Add test for verifying that if the memcap is reached, alive flows will
be reused with flow.force-reuse on and not be re-used with
flow.force-reuse off.

Ticket: #6293
@catenacyber catenacyber force-pushed the coledishington/flow-force-reuse-6293-v7 branch from 19fdf2b to ef05bb8 Compare September 18, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires suricata pr Depends on a PR in Suricata
Development

Successfully merging this pull request may close these issues.

3 participants