Skip to content

bpf: wire events map rate limits through node config#44795

Merged
ti-mo merged 1 commit intomainfrom
pr/vk/bpf/rate/limits
Mar 16, 2026
Merged

bpf: wire events map rate limits through node config#44795
ti-mo merged 1 commit intomainfrom
pr/vk/bpf/rate/limits

Conversation

@viktor-kurchenko
Copy link
Copy Markdown
Contributor

@viktor-kurchenko viktor-kurchenko commented Mar 15, 2026

Move events map rate/burst limits into node config and read them via CONFIG(events_map_{rate,burst}_limit) in BPF helpers. This drops the compile-time EVENTS_MAP* defines from the header writer and cleans up the legacy defaults in bpf/node_config.h.

Note for reviewers:
The PR was reverted previously due to the issue, but after Julian's fix, everything looks good. I've run the Cluster Mesh workflow 5 times in a row without issues.

@viktor-kurchenko viktor-kurchenko added area/loader Impacts the loading of BPF programs into the kernel. area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/misc This PR makes changes that have no direct user impact. labels Mar 15, 2026
@viktor-kurchenko
Copy link
Copy Markdown
Contributor Author

/ci-clustermesh

Move events map rate/burst limits into node config and read them via
CONFIG(events_map_{rate,burst}_limit) in BPF helpers. This drops the
compile-time EVENTS_MAP* defines from the header writer and cleans up
the legacy defaults in bpf/node_config.h.

Signed-off-by: viktor-kurchenko <viktor.kurchenko@isovalent.com>
@viktor-kurchenko
Copy link
Copy Markdown
Contributor Author

/test

@viktor-kurchenko viktor-kurchenko marked this pull request as ready for review March 15, 2026 14:13
@viktor-kurchenko viktor-kurchenko requested review from a team as code owners March 15, 2026 14:13
@ti-mo ti-mo added this pull request to the merge queue Mar 16, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 16, 2026
Merged via the queue into main with commit 9bf0885 Mar 16, 2026
597 of 603 checks passed
@ti-mo ti-mo deleted the pr/vk/bpf/rate/limits branch March 16, 2026 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. area/loader Impacts the loading of BPF programs into the kernel. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants