Skip to content

Conversation

@geyslan
Copy link
Member

@geyslan geyslan commented Sep 25, 2024

1. Explain what the PR does

5ee8caf fix(ebpf): set pipeline chan size from config

Make all pipeline channels have the size set from the config option.

2. Explain how to test it

3. Other comments

@NDStrahilevitz
Copy link
Collaborator

Also i'd argue this is a fix kind commit and not a chore as it is a bug that these sizes were not set by the config.

Make all pipeline channels have the size set from the config option.
@geyslan geyslan force-pushed the pipeline-chan-default branch from 1f6b112 to 5ee8caf Compare September 25, 2024 17:21
@github-actions github-actions bot removed the area/UX label Sep 25, 2024
@geyslan geyslan changed the title chore: add DefaultPipelineChannelSize constant fix(ebpf): set pipeline chan size from config Sep 25, 2024
Copy link
Collaborator

@NDStrahilevitz NDStrahilevitz left a comment

Choose a reason for hiding this comment

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

LGTM

@geyslan geyslan merged commit 1a0065b into aquasecurity:main Sep 25, 2024
@geyslan geyslan deleted the pipeline-chan-default branch February 19, 2025 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants