-
Notifications
You must be signed in to change notification settings - Fork 1.9k
cmd/k8s-operator: explicitly set tcp on VIPService port configuration for Ingress with ProxyGroup #16199
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
cmd/k8s-operator: explicitly set tcp on VIPService port configuration for Ingress with ProxyGroup #16199
Conversation
Pull Request Revisions
☑️ AI review skipped for r2 HelpReact with emojis to give feedback on AI-generated reviews:
We'd love to hear from you—reach out anytime at team@review.ai. |
… for Ingress with ProxyGroup Updates tailscale/corp#24795 Signed-off-by: chaosinthecrd <tom@tmlabs.co.uk>
2370214
to
3fc6bcb
Compare
Still needs manual testing |
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!
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.
The only minor nit is that we should probably start calling these 'Tailscale Services' especially in PR titles etc, to make this less confusing for folks who might be reading this in future/external contributors etc
This seems to be working well for both Ingress and Services HA 😄 |
… for Ingress with ProxyGroup (#16199) Updates tailscale/corp#24795 Signed-off-by: chaosinthecrd <tom@tmlabs.co.uk>
… for Ingress with ProxyGroup (#16199) Updates tailscale/corp#24795 Signed-off-by: chaosinthecrd <tom@tmlabs.co.uk> (cherry picked from commit 4456f77)
… for Ingress with ProxyGroup (#16199) (#16226) Updates tailscale/corp#24795 (cherry picked from commit 4456f77) Signed-off-by: chaosinthecrd <tom@tmlabs.co.uk>
… for Ingress with ProxyGroup (tailscale#16199) (tailscale#16226) Updates tailscale/corp#24795 (cherry picked from commit 4456f77) Signed-off-by: chaosinthecrd <tom@tmlabs.co.uk>
Updates https://github.com/tailscale/corp/issues/24795