Skip to content

vpp-manager: disable punting NAs toward the host#940

Merged
aritrbas merged 1 commit intomasterfrom
nsk-no-punt-na
Mar 11, 2026
Merged

vpp-manager: disable punting NAs toward the host#940
aritrbas merged 1 commit intomasterfrom
nsk-no-punt-na

Conversation

@sknat
Copy link
Collaborator

@sknat sknat commented Mar 3, 2026

This patch prevents neighbor advertisements from being punted toward the host. The expectation is that linux will issue NS that will hit NDproxy, thus we should consume NAs in VPP, and learn neighbors, but not transmit the packets to linux.

This is especially useful in the case where we loose the node IP (e.g. when dhcpv6 lease renewal fails), as forwarding NAs destined to the nodeIP to linux with a default route towards VPP will result in VPP learning neighbors on the tap0 instead of the uplink.

We also include an evolution in ND proxy that removes the need
to allowlist destination IPs that VPP will reply to. That way
we can have the guarantee that all next hops in linux will
be the MAC address of the Gateway, regardless of the routing
on the node.

@sknat sknat marked this pull request as draft March 5, 2026 15:33
@sknat sknat force-pushed the nsk-no-punt-na branch 2 times, most recently from 932ffa1 to f2f6a6e Compare March 5, 2026 16:34
This patch prevents neighbor advertisements from being
punted toward the host. The expectation is that linux
will issue NS that will hit NDproxy, thus we should
consume NAs in VPP, and learn neighbors, but not transmit
the packets to linux.

This is especially useful in the case where we loose the
node IP (e.g. when dhcpv6 lease renewal fails), as forwarding
NAs destined to the nodeIP to linux with a default route
towards VPP will result in VPP learning neighbors on the tap0
instead of the uplink.

We also include an evolution in ND proxy that removes the need
to allowlist destination IPs that VPP will reply to. That way
we can have the guarantee that all next hops in linux will
be the MAC address of the Gateway, regardless of the routing
on the node.

Signed-off-by: Nathan Skrzypczak <nathan.skrzypczak@gmail.com>
@sknat sknat marked this pull request as ready for review March 5, 2026 16:38
@aritrbas aritrbas merged commit b550e69 into master Mar 11, 2026
5 checks passed
@aritrbas aritrbas deleted the nsk-no-punt-na branch March 11, 2026 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants