Skip to content

test: e2e ipv6 ingress - #239

Merged
rubasov merged 2 commits into
masterfrom
e2e-ipv6-ingress
Sep 4, 2026
Merged

rubasov merged 2 commits into
masterfrom
e2e-ipv6-ingress

Conversation

@rubasov

@rubasov rubasov commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator
  • test: Rename separate-appnetwork to separate-appnetwork-v4
    In preparation of introducing separate-appnetwork-v6.

    Assisted-by: claude
    Signed-off-by: Bence Romsics bence.romsics@gmail.com

  • test: Add separate-appnetwork-v6 e2e suite
    This change adds an IPv6 variant of the separate-appnetwork end to end
    test suite, so we have some test coverage for IPv6-only usage.

    Assisted-by: claude
    Signed-off-by: Bence Romsics bence.romsics@gmail.com

@rubasov

rubasov commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator Author

We have considered the following options:

  1. Accept the IPv6 coverage of the dualstack test suite and call it a day.
  2. Modify and run the dualstack suite 3 times with different parameters: ipv4-only, ipv6-only and actual dual stack config.
  3. Create an ipv6 variant of an existing test suite like separate-appnetwork.

Option 1 did not have coverage for ipv6-only usage, so we threw that out. Options 2 and 3 would be a similarly sized change, but option 3 has better coverage of functionality so we chose that.

Questions for reviewers:

  • Do we want to rename all ipv4 test suites so they end in "-v4"?
  • Do we need a v6 variant of shared-appnetwork suite too?
  • Do we want to add the new tests to CI?

@rubasov rubasov changed the title e2e ipv6 ingress test: e2e ipv6 ingress Aug 27, 2026
@rubasov
rubasov requested a review from szvincze September 2, 2026 12:12
Comment thread test/e2e/suites/separate-appnetwork-v6/kustomization.yaml Outdated
Comment thread test/e2e/suites/separate-appnetwork-v6/kustomization.yaml Outdated
Comment thread test/e2e/suites/separate-appnetwork-v6/kustomization.yaml Outdated
In preparation of introducing separate-appnetwork-v6.

Assisted-by: claude (model auto-selected by kiro)
Signed-off-by: Bence Romsics <bence.romsics@gmail.com>
This change adds an IPv6 variant of the separate-appnetwork end to end
test suite, so we have some test coverage for IPv6-only usage.

Assisted-by: claude (model auto-selected by kiro)
Signed-off-by: Bence Romsics <bence.romsics@gmail.com>
@rubasov

rubasov commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

Thanks for the review.

@rubasov
rubasov requested a review from szvincze September 3, 2026 13:39
@rubasov
rubasov merged commit 3f08ef0 into master Sep 4, 2026
12 checks passed
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