Keith Mattix II
Keith Mattix II
+1 on server->app - I've worked with a customer really looking for that functionality
I believe this is already supported https://istio.io/latest/news/releases/1.22.x/announcing-1.22/#path-templating-in-authorization-policy
No I didn't catch this in my initial PR; I can make a follow-up with the same approach. I'd prefer to get #52031 merged first though
> @keithmattix, we just rolled out the update. We are seeing a double `_` in the metrics: > > `envoy_http__downstream_rq_total{http_conn_manager_prefix="outbound_172.20.75.34_8081"} 0` > > > > Would this be an easy...
> @keithmattix is this the simple fix you were alluding to? #52234 Nope it was #52829 and the accompanying cherry-pick
During the next 1.23 release; should be ~1 month?
+1 - I would like to see something like this across Istio tbh
At one point we had coverage on testgrid; can we just do that again as a first step?
> Do you want to backport to v1.30? If so, it is probably possible, and will require a subsequent PR to that release branch. Sure, I'm happy to submit the...
> Thank you for this contribution! The change in source/common/formatter/stream_info_formatter.cc seems pretty large compared to what this PR should achieve. It seems to be mainly due to a format related...