fix(telemetry): drop insight keys the OTel collector discards

prefix and service_instance_id are read and thrown away on this path, so
the node's identity label comes from [telemetry] instead. The old comment
claimed the insight copy was required or panels would be empty.
This commit is contained in:
Pratik Mankawde
2026-08-27 12:13:46 +01:00
parent f799678df3
commit fa23fb51ea

View File

@@ -135,15 +135,13 @@ data/logs/mainnet/debug.log
# --- Insight (native OTel metrics via beast::insight) -----------------------
# server is the only key that changes behaviour here. No prefix: formatName()
# ignores it, so names stay bare (jobq_job_count). service_instance_id is read
# and discarded (OTelCollector.cpp `(void)instanceId`); the label Prometheus
# shows comes from [telemetry] service_instance_id below.
[insight]
server=otel
endpoint=http://localhost:4318/v1/metrics
prefix=xrpld
# Sets the OTel service.instance.id resource attribute, which Prometheus
# exposes as the `service_instance_id` label. Dashboards filter on it via the
# $node template variable, so without this every insight-backed panel is
# empty. Matches [telemetry] service_instance_id for a single node identity.
service_instance_id=xrpld-mainnet
# --- OpenTelemetry tracing --------------------------------------------------