-
Notifications
You must be signed in to change notification settings - Fork 473
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
Changes for deployment of VPN2 after rewrite to Go. #9774
base: master
Are you sure you want to change the base?
Conversation
Skipping CI for Draft Pull Request. |
5eb0c21
to
12d3aad
Compare
The Gardener project currently lacks enough active contributors to adequately respond to all PRs.
You can:
/lifecycle stale |
1eaebb9
to
ded8fe1
Compare
e10eb8a
to
0dc3d0c
Compare
0dc3d0c
to
ef9059f
Compare
/remove-lifecycle stale |
ad8ce28
to
05f5df0
Compare
chore(apiserver): adjust vpn-path-controller in HA apiserver Signed-off-by: Lukas Hoehl <lukas.hoehl@stackit.de> Set higher limits for vpn init container in shoot use subcommand in initCommand chore(vpn): remove unused exit env var Signed-off-by: Lukas Hoehl <lukas.hoehl@stackit.de> clean up env variables; fix tests add IS_HA env run openvpn for shoot-client from shell annotation to support disable VPN Go rewrite fixes after rebase refactor entrypoints Signed-off-by: Lukas Hoehl <lukas.hoehl@stackit.de> fix kube-apiserver vpn init container Signed-off-by: Lukas Hoehl <lukas.hoehl@stackit.de> fix tests rename new vpn go images add SEED_POD_NETWORK add tunnel-controller fix HA IPv6 adjust to renamed vpn images and binaries fixes after rebase fix tunnel-controller binary name
update VPN2 images; fix inconsistencies
d738c65
to
5930e45
Compare
/invite @ScheererJ |
/retest |
/assign |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for this big improvement and allowing it to run in parallel to the old vpn so that we can easily switch between both variants.
I have a few questions/comments, though.
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test pull-gardener-integration |
/test pull-gardener-e2e-kind-migration-ha-single-zone |
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
How to categorize this PR?
/area control-plane
/kind enhancement
What this PR does / why we need it:
The rewrite of the bash scripts to Go in vpn2 brings some changes in the deployment of the vpn-seed-server and vpn-shoot-client deployments.
These adjustments must be applied together with the new release of VPN2 containing
The feature gate
NewVPN
is introduced for thegardenlet
component to enable the new implementation.Co-authored-by: @dergeberl
Co-authored-by: @hown3d
Co-authored-by: @mreiger
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note: