Skip to content

bpf, datapath: move CIDR identity range to runtime config#44223

Merged
julianwiedmann merged 1 commit intomainfrom
pr/vk/bpf/identity/range
Feb 16, 2026
Merged

bpf, datapath: move CIDR identity range to runtime config#44223
julianwiedmann merged 1 commit intomainfrom
pr/vk/bpf/identity/range

Conversation

@viktor-kurchenko
Copy link
Copy Markdown
Contributor

This commit replaces CIDR identity range macros with CONFIG values so BPF programs and tests consume cidr_identity_range_start/end at runtime.
The Go datapath now populates these fields across host, endpoint, overlay, WireGuard, XDP, and socket LB configs.

@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 Feb 6, 2026
@viktor-kurchenko viktor-kurchenko force-pushed the pr/vk/bpf/identity/range branch 2 times, most recently from 2582f92 to b682092 Compare February 6, 2026 09:10
@viktor-kurchenko
Copy link
Copy Markdown
Contributor Author

/test

@viktor-kurchenko viktor-kurchenko marked this pull request as ready for review February 6, 2026 10:06
@viktor-kurchenko viktor-kurchenko requested review from a team as code owners February 6, 2026 10:06
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.

@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 Feb 6, 2026
@viktor-kurchenko viktor-kurchenko added the dont-merge/discussion A discussion is ongoing and should be resolved before merging, regardless of reviews & tests status. label Feb 6, 2026
@joestringer joestringer removed the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Feb 6, 2026
@viktor-kurchenko viktor-kurchenko requested a review from a team as a code owner February 13, 2026 10:11
@viktor-kurchenko viktor-kurchenko force-pushed the pr/vk/bpf/identity/range branch 2 times, most recently from bb8548a to 4861858 Compare February 13, 2026 10:24
@viktor-kurchenko
Copy link
Copy Markdown
Contributor Author

/test

@viktor-kurchenko viktor-kurchenko removed the dont-merge/discussion A discussion is ongoing and should be resolved before merging, regardless of reviews & tests status. label Feb 13, 2026
@bimmlerd bimmlerd removed request for a team and bimmlerd February 13, 2026 12:13
@bimmlerd
Copy link
Copy Markdown
Member

(removed request for review; there's no contributing-owned files)

This change moves the CIDR identity range macros in bpf/lib/identity.h
and stops emitting CIDR_IDENTITY_RANGE_* defines from the datapath
header writer.

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

/test

@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 Feb 16, 2026
@julianwiedmann julianwiedmann added this pull request to the merge queue Feb 16, 2026
Merged via the queue into main with commit b3a7d19 Feb 16, 2026
477 of 483 checks passed
@julianwiedmann julianwiedmann deleted the pr/vk/bpf/identity/range branch February 16, 2026 18:40
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.

7 participants