Skip to content

ipsec: Cover IPv6-only clusters in CI #39567

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 6 commits into from
May 19, 2025
Merged

Conversation

pchaigno
Copy link
Member

This pull request adds coverage for IPv6-only configurations in the IPsec workflows, in preparation for IPsec + IPv6 underlay coverage. See commits for details.

pchaigno added 6 commits May 15, 2025 23:40
We keep bumping the limit every time we add new test cases in the
end-to-end tests. We might as well increase a lot once and for all and
we can adjust if it becomes an issue.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
The IPsec key rotation action currently needs to know if IPv6 is enabled
to double the number of expected keys in that case. Checking only IPv6
works because IPv4 was always enabled... until now.

What we actually want to do is double the number of keys when we are
running in dual-stack mode. If we are either in IPv4-only or IPv6-only
mode, the number of keys should stay as is.

This commit moves from an IPv6 parameter to a Dual Stack parameter. The
EKS workflow is IPv4-only at the moment; the E2E IPsec workflow is
dual-stack at the moment.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
Add some missing logic to be able to run IPv6-only configurations in the
IPsec workflows.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
The bpftrace script for leak detection didn't support IPv6-only
clusters. This commit fixes it.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
Same as 67bd786 ("workflows/e2e: Skip tests to outside world in
IPv6-only clusters") but for IPsec workflows.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
When IPv4 is disabled, we skip the to-fqdn tests and therefore don't end
up with DNS proxy traffic. The bpftrace script would therefore fail to
find any such traffic.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
@pchaigno pchaigno added release-note/ci This PR makes changes to the CI. feature/ipsec Relates to Cilium's IPsec feature feature/ipv6-only Relates to single-stack IPv6 support. labels May 15, 2025
@pchaigno
Copy link
Member Author

/test

@pchaigno pchaigno marked this pull request as ready for review May 16, 2025 07:19
@pchaigno pchaigno requested review from a team as code owners May 16, 2025 07:19
@pchaigno pchaigno enabled auto-merge May 16, 2025 07:19
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.

Thanks! Patches very clear and easy to follow. LGTM.

@pchaigno pchaigno added this pull request to the merge queue May 19, 2025
Merged via the queue into main with commit 8dc7793 May 19, 2025
299 of 301 checks passed
@pchaigno pchaigno deleted the pr/pchaigno/ipsec-ipv6-only branch May 19, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/ipsec Relates to Cilium's IPsec feature feature/ipv6-only Relates to single-stack IPv6 support. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants