kumahq / kuma-website

🐻 The official website for Kuma, the control plane for modern service connectivity.
https://kuma.io
Apache License 2.0
104 stars 87 forks source link

docs(policies): added references for target ref policies #1608

Closed lukidzi closed 7 months ago

lukidzi commented 7 months ago

<Explain your change!>

Did you sign your commit? Instructions

Have you read Contributing guidelines?

netlify[bot] commented 7 months ago

Deploy Preview for kuma ready!

Name Link
Latest commit fcae9b003f7e8f4ad9066e636415c193befd1bb1
Latest deploy log https://app.netlify.com/sites/kuma/deploys/65b3ac41f2b5d80009fad9e2
Deploy Preview https://deploy-preview-1608--kuma.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

lukidzi commented 7 months ago

This PR changes links to locality-aware- load balancing (get's replaced by MeshLoadBalancingStrategy) but this link in external services is left as is:

https://github.com/kumahq/kuma-website/blob/1b41f5dd68cc5ed4f60d8c6b27ba186940ac8e9f/app/_src/policies/external-services.md?plain=1#L131

. How do external services work with MeshLoadBalancing strategy?

I think we cover most of the cases for external services. I am not sure if routing though the specific zone works with MeshLoadBalancingStrategy

Edit: checked it works, changed links