Skip to content

datapath: switch l2-announcement to runtime configs#42587

Merged
dylandreimerink merged 2 commits intomainfrom
pr/jwi/main/l2-announcement
Nov 4, 2025
Merged

datapath: switch l2-announcement to runtime configs#42587
dylandreimerink merged 2 commits intomainfrom
pr/jwi/main/l2-announcement

Conversation

@julianwiedmann
Copy link
Copy Markdown
Member

@julianwiedmann julianwiedmann commented Nov 4, 2025

As the title says, convert ENABLE_L2_ANNOUNCEMENTS and L2_ANNOUNCEMENTS_MAX_LIVENESS to runtime config options.

Co-locate the code with its maps.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann julianwiedmann 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. feature/l2-announcement labels Nov 4, 2025
@julianwiedmann julianwiedmann changed the title Pr/jwi/main/l2 announcement datapath: switch l2-announcement to runtime configs Nov 4, 2025
As the title says, convert ENABLE_L2_ANNOUNCEMENTS and
L2_ANNOUNCEMENTS_MAX_LIVENESS to runtime config options.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann julianwiedmann force-pushed the pr/jwi/main/l2-announcement branch from fec9152 to 40f1248 Compare November 4, 2025 11:41
@julianwiedmann
Copy link
Copy Markdown
Member Author

/test

@julianwiedmann
Copy link
Copy Markdown
Member Author

This would also resolve #42562.

@julianwiedmann julianwiedmann marked this pull request as ready for review November 4, 2025 12:35
@julianwiedmann julianwiedmann requested review from a team as code owners November 4, 2025 12:35
Copy link
Copy Markdown
Member

@dylandreimerink dylandreimerink left a comment

Choose a reason for hiding this comment

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

Changes look good to me, thank you for taking this on!

@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Nov 4, 2025
@dylandreimerink dylandreimerink added this pull request to the merge queue Nov 4, 2025
Merged via the queue into main with commit 9b7be88 Nov 4, 2025
376 of 378 checks passed
@dylandreimerink dylandreimerink deleted the pr/jwi/main/l2-announcement branch November 4, 2025 13:33
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
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. feature/l2-announcement 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

No open projects
Status: Released

Development

Successfully merging this pull request may close these issues.

3 participants