aattuluri

@alexdotsh Thanks for the PR. This looks good, would you mind auditing other resources under the admiral/install directory for other resources that would have been deprecated in k8s 1.22. FYI,...

https://github.com/istio/istio/issues/25898

We found that using headless services (clusterIp: None) will solve the problem of load balancing across pods. We will be deferring the support for workload entry until it graduates with...

@msvechla Thanks for the bumping this up, interested in contributions upon guidance?

@maieve what version of admiral are you using, we use the service name `http-service-mesh` and it seems to be working for us. What admiral version are you using?

@wenxian Is there a spec that states deployment name ends with `env` if it has a `-`?. I will change this to feature as it's not a bug.

@jwebb49 Nice proposal! Currently istio's [merge semantics](https://istio.io/latest/docs/ops/best-practices/traffic-management/#split-virtual-services) do not allow virtual services to be merged for sidecars (only allowed at Gateway). We need this before we can implement the proposed...

@Ultrafenrir This is where its hardcoded https://github.com/istio-ecosystem/admiral/blob/70af3fb2459102bc31e47ecb1307b2e342435dab/admiral/pkg/controller/common/common.go#L22, it would be great if you can send a PR, should be a simple change to https://github.com/istio-ecosystem/admiral/blob/master/admiral/cmd/admiral/cmd/root.go if its static for all clusters...

@thinkin Locality load balancing in istio is currently supported at a host level (its configurable on DestinationRule). Supporting the behavior at a route level would need some sophisticated configuration to...

@wenxian This could be an istio issue, but I remember this was tested at least in istio version `1.5.x` i) What istio version are you using? ii) Can you paste...