Skip to content

Conversation

MartinWeindel
Copy link
Member

How to categorize this PR?

/area control-plane
/kind enhancement

What this PR does / why we need it:
If the NewVPN feature gate is enabled, an IPv6 transport network is used by the VPN tunnel. An IPv6 address needs to be set in the vpn-seed-server for both HA and non-HA cases. If IPv6 is disabled in the pod (e.g. seen for GKE clusters with IPv4 stack only), the init containers must enable IPv6 by changing a kernel setting.
This PR covers the non-HA case.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
Follow-up of #10641

Release note:

[NewVPN] Enable IPv6 for non-HA if needed.

Copy link
Contributor

gardener-prow bot commented Dec 6, 2024

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@gardener-prow gardener-prow bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. area/control-plane Control plane related kind/enhancement Enhancement, improvement, extension labels Dec 6, 2024
@gardener-prow gardener-prow bot added cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Dec 6, 2024
@MartinWeindel
Copy link
Member Author

Depends on upcoming vpn2 release 0.32.0

@marc1404
Copy link
Member

marc1404 commented Dec 10, 2024

Depends on upcoming vpn2 release 0.32.0

@MartinWeindel The update of vpn2 to version 0.33.0 has been merged 🎉
👉 #10996

@MartinWeindel MartinWeindel marked this pull request as ready for review December 10, 2024 15:53
@gardener-prow gardener-prow bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 10, 2024
@gardener-prow gardener-prow bot requested review from acumino and shafeeqes December 10, 2024 15:53
Copy link
Member

@ScheererJ ScheererJ left a comment

Choose a reason for hiding this comment

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

Thanks for helping also the non-HA clusters on IPv6-hating infrastructure ;-) .

/assign
/lgtm
/approve

@gardener-prow gardener-prow bot added the lgtm Indicates that a PR is ready to be merged. label Dec 11, 2024
Copy link
Contributor

gardener-prow bot commented Dec 11, 2024

LGTM label has been added.

Git tree hash: 62be32054085144d91c3947a841b9d61472b7c4a

Copy link
Contributor

gardener-prow bot commented Dec 11, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ScheererJ

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gardener-prow gardener-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 11, 2024
@gardener-prow gardener-prow bot merged commit 84c5986 into gardener:master Dec 11, 2024
18 checks passed
@MartinWeindel MartinWeindel deleted the fix/newvpn-enable-ipv6 branch December 11, 2024 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/control-plane Control plane related cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. kind/enhancement Enhancement, improvement, extension lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants