Nick Fox

Results 123 comments of Nick Fox

IMO before we close out this epic we should look at consolidating what's in the `mesh` package with what's in the `graph` package. There seems to be a lot of...

Would this just be supporting mounting the secret as a file into the kiali pod and to have the kiali server read that file? @jmazzitelli I think this is just...

@josunect I made this both more complicated and more simple. I added an expiring cache which is generally useful and we can reuse it elsewhere. It made the `IsAmbientEnabled` implementation...

> I guess that should apply this changes before merge? #7162 Yeah let's get that one in first and I'll rebase.

@Aur0nd you can already use Kiali with multi-cluster istio deployments. See https://kiali.io/docs/configuration/multi-cluster/ for more info. What's left in this issue as of today is mostly cleanup or enabling specific deployments...

Closing this as completed since support for all deployment models has been added which was the goal of this epic. Future multi-cluster related work and enhancements can be tracked separately.

You can see that number of namespaces is a big issue when it comes to perf simply due to the amount of requests the frontend makes to the backend. This...

FWIW this is still an issue. You can still see these data race logs in CI: https://github.com/kiali/kiali/actions/runs/6973247695/job/18976996749?pr=6854#step:9:71. You'd be able to see these in production as well but the prod...

I took a cursory look at this and here's where I think the problem is: https://github.com/kiali/kiali/blob/2ed984f94c97c929b9f302b8454b24b0535eafb1/kubernetes/cache/kube_cache.go#L981-L993 Inside the kube cache methods, the `checkIstioAPIsExist(c.client)` is not thread safe and we need...

After some investigation, this is blocked by #6360 and #6037. Moving this back to backlog and out of beta.