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

Make size of event queue configurable #620

Merged

Conversation

kevsecurity
Copy link
Contributor

The internal event queue had a fixed size of 100 events. If the event queue became full, additional events would be dropped. This commit raises the default size to 10000 events and makes the size configurable through a --event-queue-size command line flag.

Signed-off-by: Kevin Sheldrake kevin.sheldrake@isovalent.com

@kevsecurity kevsecurity requested a review from a team as a code owner January 11, 2023 13:13
@kevsecurity kevsecurity requested review from tpapagian and removed request for a team January 11, 2023 13:13
willfindlay and others added 2 commits January 11, 2023 13:36
Move away from the regular GitHub runner to a large one, which supports KVM acceleration.
This should significantly speed up test performance, so let's drop down to one group per
test and add a 5.15 test while we're at it.

Signed-off-by: William Findlay <will@isovalent.com>
The internal event queue had a fixed size of 100 events. If the event
queue became full, additional events would be dropped. This commit
raises the default size to 10000 events and makes the size configurable
through a --event-queue-size command line flag.

Signed-off-by: Kevin Sheldrake <kevin.sheldrake@isovalent.com>
@kevsecurity kevsecurity force-pushed the backport/0.8/kevsecurity/increase-event-channel-size branch from 7df2539 to 2b73ff2 Compare January 11, 2023 14:07
Copy link
Member

@tpapagian tpapagian left a comment

Choose a reason for hiding this comment

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

LGTM!

@kevsecurity kevsecurity merged commit 9afb3d6 into v0.8 Jan 11, 2023
@kevsecurity kevsecurity deleted the backport/0.8/kevsecurity/increase-event-channel-size branch January 11, 2023 14:36
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.

3 participants