Andrew Sauber

icon indicating a website link https://www.wordonthestack.com/

icon location Cambridge, MA

Results 33 comments of Andrew Sauber

Proposal / RFE: Change to use the value of `external-dns.alpha.kubernetes.io/hostname` instead of `${ClusterName}.mesh.cilium.io` when `external-dns.alpha.kubernetes.io/hostname` is set on the `clustermesh-apiserver` LoadBalancer Service

I see your point. If we can avoid DNS lookups entirely, then that would be ideal. Providing `.address` for each cluster should be sufficient, given [the clustermesh config implementation](https://github.com/cilium/cilium/blob/944dddf59b1a25b4d6a1d71d468afd8c60abf731/install/kubernetes/cilium/templates/clustermesh-config/_helpers.tpl#L15) in...

Updated deprecated service annotation

Looking good. Could you please squash your commits into a single commit, and include a more descriptive summary and description in the commit message? A good example of our commit...

Updated deprecated service annotation

Thanks for the update. Not quite there with the commit message. Your commit message is: ``` Updated deprecated annotation which is deprecated https://kubernetes-sigs.github.io/aws-load-balancer-controller/v2.6/guide/service/annotations/#:~:text=string-,service.beta.kubernetes.io/aws%2Dload%2Dbalancer%2Dinternal,-boolean Signed-off-by: [email protected] ``` One issue is that...