summary refs log tree commit diff
path: root/synapse/logging
diff options
context:
space:
mode:
authorDavid Robertson <davidr@element.io>2023-02-09 22:33:39 +0000
committerGitHub <noreply@github.com>2023-02-09 22:33:39 +0000
commita5a799722db0c33dc61fb2c6c7282ff7e82eb2e9 (patch)
treef2da789f419ec35341cde1c305c6778b88f85429 /synapse/logging
parentAdd a class UnpersistedEventContext to allow for the batching up of storing s... (diff)
downloadsynapse-a5a799722db0c33dc61fb2c6c7282ff7e82eb2e9.tar.xz
Tag federation request spans with the worker name (#15042)
* Systematically include worker name as process info

* Changelog

* don't bother with inner setdefault
Diffstat (limited to 'synapse/logging')
-rw-r--r--synapse/logging/opentracing.py10
1 files changed, 9 insertions, 1 deletions
diff --git a/synapse/logging/opentracing.py b/synapse/logging/opentracing.py
index 8ef9a0dda8..6c7cf1b294 100644
--- a/synapse/logging/opentracing.py
+++ b/synapse/logging/opentracing.py
@@ -466,8 +466,16 @@ def init_tracer(hs: "HomeServer") -> None:
         STRIP_INSTANCE_NUMBER_SUFFIX_REGEX, "", hs.get_instance_name()
     )
 
+    jaeger_config = hs.config.tracing.jaeger_config
+    tags = jaeger_config.setdefault("tags", {})
+
+    # tag the Synapse instance name so that it's an easy jumping
+    # off point into the logs. Can also be used to filter for an
+    # instance that is under load.
+    tags[SynapseTags.INSTANCE_NAME] = hs.get_instance_name()
+
     config = JaegerConfig(
-        config=hs.config.tracing.jaeger_config,
+        config=jaeger_config,
         service_name=f"{hs.config.server.server_name} {instance_name_by_type}",
         scope_manager=LogContextScopeManager(),
         metrics_factory=PrometheusMetricsFactory(),