diff options
author | Eric Eastwood <erice@element.io> | 2022-09-09 11:30:06 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-09-09 11:30:06 -0500 |
commit | f694bb71b7ea7841a5b5db3d884dfda5a3f78023 (patch) | |
tree | 7dddd224184bd102ed9da0a9553cc4ed82e18e33 /synapse/logging/_remote.py | |
parent | Use an upsert for `receipts_graph`. (#13752) (diff) | |
download | synapse-f694bb71b7ea7841a5b5db3d884dfda5a3f78023.tar.xz |
Strip number suffix from instance name to consolidate services that traces are spread over (#13729)
The problem with many services is that it makes it hard to find which service has the trace you want, see https://github.com/jaegertracing/jaeger-ui/issues/985 Previously, we split traces out into services based on their instance name like `matrix.org client_reader-1`, etc but there are many worker instances of the same `client_reader` so there is a lot to click through. With this PR, all of the traces are just collected under the worker type like `client_reader`, `event_persister` 😇 Note: A Synapse worker instance name is an opaque string with the number convention only being our own thing for the `matrix.org` deployment. But seems pretty sensible to group things this way.
Diffstat (limited to 'synapse/logging/_remote.py')
0 files changed, 0 insertions, 0 deletions