Skip to content

Conversation

pchaigno
Copy link
Member

@pchaigno pchaigno commented Jun 16, 2025

First two commits add support for WireGuard encryption with IPv6 underlay. Last two commits cover it in CI.

@pchaigno pchaigno added release-note/misc This PR makes changes that have no direct user impact. feature/ipv6 Relates to IPv6 protocol support feature/wireguard Relates to Cilium's Wireguard feature labels Jun 16, 2025
@pchaigno pchaigno force-pushed the pr/pchaigno/test-wireguard-ipv6-underlay branch 7 times, most recently from d10cb31 to e087a85 Compare June 17, 2025 11:00
@pchaigno pchaigno changed the title Test WireGuard Support WireGuard with IPv6 Underlay Jun 17, 2025
@pchaigno pchaigno marked this pull request as ready for review June 17, 2025 11:01
@pchaigno pchaigno requested review from a team as code owners June 17, 2025 11:01
Copy link
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.

This looks fantastic, thanks!

@pchaigno pchaigno enabled auto-merge June 17, 2025 13:23
@joestringer joestringer added the dont-merge/wait-until-release Freeze window for current release is blocking non-bugfix PRs label Jun 20, 2025
@pchaigno pchaigno removed the dont-merge/wait-until-release Freeze window for current release is blocking non-bugfix PRs label Jul 1, 2025
@pchaigno pchaigno added this pull request to the merge queue Jul 1, 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. labels Jul 1, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 1, 2025
pchaigno added 4 commits July 1, 2025 17:51
Thanks to our recent support of IPv6 underlays, IPv6 packets are not
always encapsulated in IPv4 anymore. Hence, we need to send encapsulated
traffic to the WireGuard interface regardless of its IP family.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
The previous commit fixed our datapath to allow VXLAN/GENEVE-over-IPv6
to be sent to the WireGuard interface, so WireGuard should now be
allowed with IPv6 underlay.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
When running in an IPv6-only cluster in tunneling mode, the node
encryption test fails because the tcpdump filter is incorrect. It goes
"... and host  and host  " leaving the host IP addresses empty.

This commit fixes it by supporting all cases of IPv4-only,
IPv6-only, and dual-stack.

Fixes: f5bd938 ("connectivity: Add special case to test WG w/ tunneling")
Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
@pchaigno pchaigno force-pushed the pr/pchaigno/test-wireguard-ipv6-underlay branch from e087a85 to 95a3000 Compare July 1, 2025 15:51
@pchaigno pchaigno enabled auto-merge July 1, 2025 15:51
@pchaigno
Copy link
Member Author

pchaigno commented Jul 1, 2025

/test

@pchaigno pchaigno added this pull request to the merge queue Jul 1, 2025
Merged via the queue into main with commit 7e6baca Jul 1, 2025
380 of 383 checks passed
@pchaigno pchaigno deleted the pr/pchaigno/test-wireguard-ipv6-underlay branch July 1, 2025 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/ipv6 Relates to IPv6 protocol support 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
None yet
Development

Successfully merging this pull request may close these issues.

6 participants