Skip to content

datapath: Synchronize backend with the Linux neighbor table for new LB control-plane #39001

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 24, 2025

Conversation

joamaki
Copy link
Contributor

@joamaki joamaki commented Apr 17, 2025

This implements the missing "InsertMiscNeighbor" call for when backends are added or removed to keep
the Linux neighbor table up-to-date for e.g. XDP programs.

@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 Apr 17, 2025
@joamaki joamaki force-pushed the pr/joamaki/lb-backend-neighbor branch from dee47f7 to 2f69583 Compare April 17, 2025 10:33
Add support for syncing the load-balancing backend addresses to the
Linux neighbor table for the new load-balancing control-plane.

Implemented as a background job that watches upserts and deletes
happening to the Table[*Backend] and calling InsertMiscNeighbor/DeleteMiscNeighbor
accordingly.

Signed-off-by: Jussi Maki <jussi@isovalent.com>
@joamaki joamaki force-pushed the pr/joamaki/lb-backend-neighbor branch from 2f69583 to 8f00d11 Compare April 23, 2025 08:05
@joamaki joamaki added the release-note/misc This PR makes changes that have no direct user impact. label Apr 23, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 23, 2025
@joamaki
Copy link
Contributor Author

joamaki commented Apr 23, 2025

/test

@joamaki joamaki requested a review from borkmann April 23, 2025 08:13
@joamaki joamaki marked this pull request as ready for review April 23, 2025 08:13
@joamaki joamaki requested review from a team as code owners April 23, 2025 08:13
@joamaki joamaki enabled auto-merge April 23, 2025 13:20
@joamaki joamaki added this pull request to the merge queue Apr 24, 2025
@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 Apr 24, 2025
Merged via the queue into cilium:main with commit 49a3684 Apr 24, 2025
72 checks passed
@joamaki joamaki deleted the pr/joamaki/lb-backend-neighbor branch April 24, 2025 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants