Skip to content

bpf: Migrate WG_IFINDEX and WG_PORT to runtime config#41137

Merged
julianwiedmann merged 1 commit intocilium:mainfrom
jrife:jrife/clang-free-config
Sep 24, 2025
Merged

bpf: Migrate WG_IFINDEX and WG_PORT to runtime config#41137
julianwiedmann merged 1 commit intocilium:mainfrom
jrife:jrife/clang-free-config

Conversation

@jrife
Copy link
Copy Markdown
Contributor

@jrife jrife commented Aug 13, 2025 •
edited
Loading

Migrate WG_IFINDEX and WG_PORT to runtime config.

Related: #38370

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 13, 2025
@jrife jrife force-pushed the jrife/clang-free-config branch 3 times, most recently from 158fb32 to 7650157 Compare August 13, 2025 21:15
@jrife
Copy link
Copy Markdown
Contributor Author

jrife commented Aug 13, 2025

/test

@jrife jrife force-pushed the jrife/clang-free-config branch from 7650157 to 984d0bd Compare August 14, 2025 14:29
@jrife
Copy link
Copy Markdown
Contributor Author

jrife commented Aug 14, 2025

/test

@jrife
Copy link
Copy Markdown
Contributor Author

jrife commented Aug 14, 2025

/ci-integration

1 similar comment
@jrife
Copy link
Copy Markdown
Contributor Author

jrife commented Aug 14, 2025

/ci-integration

@jrife
Copy link
Copy Markdown
Contributor Author

jrife commented Aug 14, 2025

/ci-clustermesh

@jrife jrife force-pushed the jrife/clang-free-config branch from 984d0bd to 2199c6b Compare September 10, 2025 18:23
@jrife
Copy link
Copy Markdown
Contributor Author

jrife commented Sep 10, 2025

/test

@jrife jrife force-pushed the jrife/clang-free-config branch from 2199c6b to 903d02e Compare September 10, 2025 18:45
@jrife
Copy link
Copy Markdown
Contributor Author

jrife commented Sep 10, 2025

/test

@jrife jrife marked this pull request as ready for review September 10, 2025 20:47
@jrife jrife requested review from a team as code owners September 10, 2025 20:47
smagnani96
smagnani96 approved these changes Sep 11, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@smagnani96 smagnani96 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Jordan!

dylandreimerink
dylandreimerink approved these changes Sep 16, 2025
View reviewed changes
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.

Looks good overall, I would just suggest to use the DECLARE_CONFIG over NODE_CONFIG in this case.

@jrife jrife force-pushed the jrife/clang-free-config branch 2 times, most recently from 01d3343 to 5b6ba2b Compare September 17, 2025 17:16
@jrife
bpf: Migrate WG_IFINDEX and WG_PORT to runtime config
59cc375 
Related: cilium#38370

Signed-off-by: Jordan Rife <jrife@google.com>
@jrife jrife force-pushed the jrife/clang-free-config branch from 5b6ba2b to 59cc375 Compare September 17, 2025 17:47
@jrife
Copy link
Copy Markdown
Contributor Author

jrife commented Sep 17, 2025

/test

@julianwiedmann julianwiedmann added area/loader Impacts the loading of BPF programs into the kernel. release-note/misc This PR makes changes that have no direct user impact. feature/wireguard Relates to Cilium's Wireguard feature labels Sep 24, 2025
@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. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Sep 24, 2025
@julianwiedmann julianwiedmann added this pull request to the merge queue Sep 24, 2025
Merged via the queue into cilium:main with commit fd2d7b6 Sep 24, 2025
69 checks passed
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
@julianwiedmann julianwiedmann mentioned this pull request Apr 1, 2026
Open
33 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dylandreimerink dylandreimerink dylandreimerink approved these changes

@smagnani96 smagnani96 smagnani96 approved these changes

Assignees

No one assigned

Labels

area/loader Impacts the loading of BPF programs into the kernel. feature/wireguard Relates to Cilium's Wireguard feature 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
cilium v1.19.0
Status: Released

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jrife @dylandreimerink @smagnani96 @julianwiedmann