stacklok / toolhive / 23951758196

65%

Build Type

push

github

Committed by web-flow

Commit Message

Add OTEL distributed tracing and optional Sentry integration to the API server (#4365)

* Add sentry-go dependency

* Add Sentry configuration and lifecycle package

* Wire Sentry into API server and add CLI flags

* Report panics to Sentry in recovery middleware

* Capture 5xx errors to Sentry in error handler

* Document Sentry integration for API server

* Regenerate CLI docs to include Sentry flags

* Add sentry-go/otel bridge and update OTEL SDK to v1.42.0

* Replace sentryhttp middleware with OTEL span processor
Drop NewMiddleware() backed by sentryhttp in favour of SpanProcessor()
which returns a sentryotel.NewSentrySpanProcessor. This lets the OTEL
SDK drive tracing while Sentry receives spans as a processor, aligning
the API server with W3C traceparent-based distributed tracing.
Also tags every Sentry event with custom.user_id from the anonymous
instance ID so API server events correlate with toolhive-studio.

* Support extra span processors in OTEL provider chain
Add ExtraSpanProcessors to the provider config and thread them through
NewProvider → StrategySelector → OTLPTracerStrategy → tracing.go.
When extra processors are present but no OTLP endpoint is configured, a
real SDK tracer provider is created (without an exporter) so processors
still receive spans. This enables Sentry-only tracing without requiring
an external collector.

* Wire OTEL distributed tracing into API server
Add initServeOTEL() to cmd/thv/app/server.go that reads the global OTEL
config and registers the Sentry span processor when available. In
Sentry-only mode (no OTLP endpoint) tracing is force-enabled at 100%
OTEL sampling so every span reaches the Sentry processor.
Replace sentryhttp middleware in pkg/api/server.go with otelhttp, using
WithSpanNameFormatter to produce route-aware span names like
'GET /api/v1beta/workloads/{name}' instead of a fixed 'thv-api'.

* Update docs for OTEL-first distributed tracing approach

* Record panics and 5xx errors on OTEL spans

Rep... (continued)

Coverage Stats

164 of 300 new or added lines in 11 files covered. (54.67%)

17 existing lines in 5 files now uncovered.

54537 of 83541 relevant lines covered (65.28%)

63.34 hits per line