Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Commit d8be7d4

Browse files
authored
Enable Prometheus metrics for the jaeger client library (#10112)
1 parent fd9856e commit d8be7d4

File tree

3 files changed

+5
-1
lines changed

3 files changed

+5
-1
lines changed

changelog.d/10112.misc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Enable Prometheus metrics for the jaeger client library.

mypy.ini

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ ignore_missing_imports = True
130130
[mypy-canonicaljson]
131131
ignore_missing_imports = True
132132

133-
[mypy-jaeger_client]
133+
[mypy-jaeger_client.*]
134134
ignore_missing_imports = True
135135

136136
[mypy-jsonschema]

synapse/logging/opentracing.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -362,10 +362,13 @@ def init_tracer(hs: "HomeServer"):
362362

363363
set_homeserver_whitelist(hs.config.opentracer_whitelist)
364364

365+
from jaeger_client.metrics.prometheus import PrometheusMetricsFactory
366+
365367
config = JaegerConfig(
366368
config=hs.config.jaeger_config,
367369
service_name="{} {}".format(hs.config.server_name, hs.get_instance_name()),
368370
scope_manager=LogContextScopeManager(hs.config),
371+
metrics_factory=PrometheusMetricsFactory(),
369372
)
370373

371374
# If we have the rust jaeger reporter available let's use that.

0 commit comments

Comments
 (0)