Skip to content

feat(flow.entry): make the aggressive reap threshold configurable#1320

Merged
qmonnet merged 1 commit intomainfrom
pr/fredi/fix_test_flow_agressive_reap
Mar 9, 2026
Merged

feat(flow.entry): make the aggressive reap threshold configurable#1320
qmonnet merged 1 commit intomainfrom
pr/fredi/fix_test_flow_agressive_reap

Conversation

@Fredi-raspall
Copy link
Contributor

Some tests have been failing because they used the 1M default threshold and flows' lifetime was 10 minutes only, which was probably exceeded by the number of flows if the CI infra was loaded. Fix this by making the threshold configurable and setting a smaller value in the tests.

@Fredi-raspall Fredi-raspall requested a review from qmonnet March 7, 2026 11:16
@Fredi-raspall Fredi-raspall requested a review from a team as a code owner March 7, 2026 11:16
Some tests have been failing because they used the 1M default
threshold and flows' lifetime was 10 minutes only, which was
probably exceeded by the number of flows if the CI infra was
loaded. Fix this by making the threshold configurable and
setting a smaller value in the tests.

Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
@Fredi-raspall Fredi-raspall force-pushed the pr/fredi/fix_test_flow_agressive_reap branch from 4bd4c50 to b20284e Compare March 7, 2026 11:42
Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

Thank you!

@qmonnet qmonnet added this pull request to the merge queue Mar 9, 2026
Merged via the queue into main with commit d87724c Mar 9, 2026
21 checks passed
@qmonnet qmonnet deleted the pr/fredi/fix_test_flow_agressive_reap branch March 9, 2026 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants