Skip to content

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

Merged

Conversation

ChaosInTheCRD
Copy link
Contributor

@ChaosInTheCRD ChaosInTheCRD commented Jun 5, 2025

Copy link

review-ai-agent bot commented Jun 5, 2025

Pull Request Revisions

RevisionDescription
r2No changes since last revision
r1
Added tcp prefix to service portsModified Tailscale service port configuration to explicitly prefix ports with "tcp:" in multiple files and test cases

☑️ AI review skipped for r2
Help React with emojis to give feedback on AI-generated reviews:
  • 👍 means the feedback was helpful and actionable
  • 👎 means the feedback was incorrect or unhelpful
💬 Replying to feedback with a comment helps us improve the system. Your input also contributes to shaping future interactions with the AI reviewer.

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>
@ChaosInTheCRD ChaosInTheCRD force-pushed the chaosinthecrd/k8s-operator-tailscale-service-ports branch from 2370214 to 3fc6bcb Compare June 5, 2025 16:59
@ChaosInTheCRD
Copy link
Contributor Author

Still needs manual testing

Copy link
Contributor

@irbekrm irbekrm left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Contributor

@irbekrm irbekrm left a 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

@ChaosInTheCRD
Copy link
Contributor Author

This seems to be working well for both Ingress and Services HA 😄

@ChaosInTheCRD ChaosInTheCRD merged commit 4456f77 into main Jun 9, 2025
51 checks passed
@ChaosInTheCRD ChaosInTheCRD deleted the chaosinthecrd/k8s-operator-tailscale-service-ports branch June 9, 2025 10:13
ChaosInTheCRD added a commit that referenced this pull request Jun 9, 2025
… for Ingress with ProxyGroup (#16199)

Updates tailscale/corp#24795

Signed-off-by: chaosinthecrd <tom@tmlabs.co.uk>
ChaosInTheCRD added a commit that referenced this pull request Jun 9, 2025
… for Ingress with ProxyGroup (#16199)

Updates tailscale/corp#24795

Signed-off-by: chaosinthecrd <tom@tmlabs.co.uk>
(cherry picked from commit 4456f77)
ChaosInTheCRD added a commit that referenced this pull request Jun 9, 2025
… for Ingress with ProxyGroup (#16199) (#16226)

Updates tailscale/corp#24795


(cherry picked from commit 4456f77)

Signed-off-by: chaosinthecrd <tom@tmlabs.co.uk>
@ChaosInTheCRD ChaosInTheCRD restored the chaosinthecrd/k8s-operator-tailscale-service-ports branch June 18, 2025 17:25
thirdeyenick pushed a commit to ninech/tailscale that referenced this pull request Jul 2, 2025
… 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>
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.

3 participants