Skip to content

cilium-cli: IPv6 connectivity tests for PodToHostPort #39666

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
May 29, 2025

Conversation

gentoo-root
Copy link
Contributor

@gentoo-root gentoo-root commented May 21, 2025

Test both IPv4 and IPv6 explicitly. ForEachIPFamily ensures testing those address families that are supported in the cluster.

Fixes: #37134

@gentoo-root gentoo-root added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels May 21, 2025
@github-actions github-actions bot added cilium-cli This PR contains changes related with cilium-cli cilium-cli-exclusive This PR only impacts cilium-cli binary labels May 21, 2025
@gentoo-root gentoo-root force-pushed the pr/max/ipv6-tests-2 branch from d972eb8 to 327045e Compare May 21, 2025 17:47
@gentoo-root
Copy link
Contributor Author

/test

@gentoo-root gentoo-root marked this pull request as ready for review May 21, 2025 21:07
@gentoo-root gentoo-root requested review from a team as code owners May 21, 2025 21:07
@gentoo-root gentoo-root requested review from thorn3r and Artyop May 21, 2025 21:07
@julianwiedmann julianwiedmann added the feature/ipv6 Relates to IPv6 protocol support label May 22, 2025
@julianwiedmann
Copy link
Member

@cilium/sig-agent seems like Tim is busy, could maybe someone else have a look at this PR? :)

Copy link
Member

@giorio94 giorio94 left a comment

Choose a reason for hiding this comment

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

Thanks! One comment inline.

@giorio94 giorio94 removed the request for review from thorn3r May 29, 2025 06:56
@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 May 29, 2025
Test both IPv4 and IPv6 explicitly. ForEachIPFamily ensures testing
those address families that are supported in the cluster.

Fixes: #37134
Signed-off-by: Maxim Mikityanskiy <maxim@isovalent.com>
@gentoo-root gentoo-root force-pushed the pr/max/ipv6-tests-2 branch from 327045e to 6fa079a Compare May 29, 2025 12:32
@gentoo-root
Copy link
Contributor Author

/test

@aditighag aditighag added this pull request to the merge queue May 29, 2025
Merged via the queue into main with commit 0bb5894 May 29, 2025
217 of 219 checks passed
@aditighag aditighag deleted the pr/max/ipv6-tests-2 branch May 29, 2025 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake cilium-cli This PR contains changes related with cilium-cli cilium-cli-exclusive This PR only impacts cilium-cli binary feature/ipv6 Relates to IPv6 protocol support ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add explicit IPv6 testing to PodToHostPort cilium-cli connectivity test scenario
5 participants