From e1ef6ba18372230d00ccd9ce68c5814ef3621cd7 Mon Sep 17 00:00:00 2001
From: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Date: Fri, 11 Sep 2026 11:30:17 +0100
Subject: [PATCH] docs(telemetry): drop the inert insight endpoint from the
 test config template

On the OTel path only [insight] server is load-bearing. CollectorManager reads
endpoint and hands it to OTelCollector, which logs it at startup and routes
nothing with it; the real export endpoint is [telemetry] metrics_endpoint,
which the template already sets. service_instance_id and service_name in that
section are read and discarded.

Leaving the line invited an operator to reconcile a mismatch that has no
effect. integration-test.sh already emits only server=otel with the same
explanation, so the two now agree.
---
 docker/telemetry/TESTING.md | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/docker/telemetry/TESTING.md b/docker/telemetry/TESTING.md
index a18baaadc7..d18114d80c 100644
--- a/docker/telemetry/TESTING.md
+++ b/docker/telemetry/TESTING.md
@@ -266,8 +266,10 @@ trace_peer=1
 trace_ledger=1
 
 [insight]
+# server=otel is the only load-bearing key here -- it selects OTelCollector.
+# The export endpoint comes from [telemetry] metrics_endpoint, and [insight]'s
+# own service_instance_id/service_name keys are ignored.
 server=otel
-endpoint=http://localhost:4318/v1/metrics
 
 [rpc_startup]
 { "command": "log_level", "severity": "warning" }