kumahq / kuma

🐻 The multi-zone service mesh for containers, Kubernetes and VMs. Built with Envoy. CNCF Sandbox Project.
https://kuma.io/install
Apache License 2.0
3.67k stars 333 forks source link

Inspect API `toResourceRules` to include envoy cluster name #11293

Open lobkovilya opened 2 months ago

lobkovilya commented 2 months ago

Description

Kuma GUI needs to match envoy clusters with MS, MES and MZMS. The easiest way to do that is to return clusterName in the Inspect API. This way don't have to deal with cluster name parsing on the GUI.

Depends on the https://github.com/kumahq/kuma/issues/11292, because at this moment DestinationName() returns different results on Zone and on Global.

lobkovilya commented 2 months ago

I pushed the expected change in OpenAPI spec to my branch https://github.com/kumahq/kuma/compare/master...lobkovilya:kuma:stat-prefix-spec-change?expand=1

lahabana commented 2 months ago

One the UI we've made the choice of splitting this to use in matching of policies. We may need to rework this in the future but we shouldn't need this in 2.9.x for now