docs(telemetry): remove Phase references from loki/tempo datasource comments

Reviewer noted implementation-phase references should not appear in
shipped config comments. Drop the phase-8-owned 'Phase 8' tokens from
loki.yaml and tempo.yaml, keeping the functional descriptions.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Pratik Mankawde
2026-07-07 19:45:36 +01:00
parent 89cb633869
commit aefc87a572
2 changed files with 3 additions and 7 deletions

View File

@@ -1,12 +1,8 @@
# Grafana Loki data source provisioning for rippled log-trace correlation.
#
# Phase 8: Log-Trace Correlation and Centralized Log Ingestion
#
# Loki ingests rippled logs via OTel Collector's filelog receiver.
# The derivedFields config links trace_id values in log lines back to
# Tempo traces, enabling one-click log-to-trace navigation in Grafana.
#
# See: OpenTelemetryPlan/Phase8_taskList.md (Tasks 8.2, 8.4)
apiVersion: 1

View File

@@ -26,14 +26,14 @@ datasources:
# Prometheus service is added to docker-compose.yml.
serviceMap:
datasourceUid: prometheus
# Phase 8: Trace-to-log correlation — enables one-click navigation
# Trace-to-log correlation — enables one-click navigation
# from a Tempo trace to the corresponding Loki log lines. Filters
# by trace_id so only logs from the same trace are shown.
tracesToLogs:
datasourceUid: loki
filterByTraceID: true
filterBySpanID: false
tags: ["partition", "severity"]
tags: []
tracesToMetrics:
datasourceUid: prometheus
spanStartTimeShift: "-1h"
@@ -137,7 +137,7 @@ datasources:
tag: tx_type
operator: "="
scope: span
type: static
type: dynamic
# Phase 4: Consensus tracing filters
- id: consensus-mode
tag: consensus_mode