Jesse Rosenberger
Jesse Rosenberger
This might be related to https://github.com/apollographql/router/pull/5235, which should land _reasonably_ soon. If you wanted to try with that PR included, that might be a worthwhile try. 😄
We can re-open this if/when needed.
👋 Mostly an FYI while I close this out, but we have `apollo.router.pipelines` (https://github.com/apollographql/router/pull/6967) and `apollo.router.open_connections` metrics (https://github.com/apollographql/router/pull/7023) available in https://github.com/apollographql/router/releases/tag/v2.1.0 (it is also available as of https://github.com/apollographql/router/releases/tag/v1.61.1 on the...
This was partially fixed by #5159, but there are still some remaining things to be looked at.
Ah, thanks for opening this. I had it on my list to chase. I'll tag it.
This seems like it would be great to land. Do we think we could continue to talk about this more? Perhaps next week?
We'll take a look!
I'll close this in lieu of https://github.com/apollographql/router/pull/5100 which took a similar approach to solve for the intent of this PR, along with taking some other considerations. Appreciate you taking the...
Do you think this is ultimately the same as #4892? If so, it should/could be fixed by https://github.com/apollographql/router/pull/5071? (It was recently opened). Can you take a look?
@BrynCooke You're pretty connected to the work here - Can you advise? (Look carefully they're both different nuances of dashes in trace IDs)