linkerd2 icon indicating copy to clipboard operation
linkerd2 copied to clipboard

Add support for loadBalancerCluster to multi-cluster gateway service

Open atoy3731 opened this issue 1 year ago • 3 comments

The multicluster helm chart currently does not support the loadBalancerClass field. This is required by some loadbalance services like Tailscale.

Add a conditional loadBalancerClass key to the helm chart values and gateway svc.

Once merged, deploy the multicluster helm chart and configure .Values.gateway.loadBalancerClass to a value, ensure that is part of the resulting resource. Unset the value, ensure it is not part of the resulting resource.

Fixes #12100

atoy3731 avatar Feb 21 '24 16:02 atoy3731

I agree to the DCO for all the commits in this PR.

atoy3731 avatar Feb 21 '24 21:02 atoy3731

@adleong added the comment per those docs, but let me know if there's anything else I need to do on my side. Thank you!

atoy3731 avatar Feb 22 '24 15:02 atoy3731

LGTM, thanks @atoy3731

mateiidavid avatar Feb 26 '24 14:02 mateiidavid