feat: add OpenTelemetry distributed tracing#92
Merged
Conversation
…agent spans - Inject/extract W3C TraceContext headers on NATS publish/consume in natsclient - Add natsHeaderCarrier TextMapCarrier implementation for NATS headers - Add spans to keycloak ProcessDeploy and provisionConfidentialClient - Initialize TracerProvider in keycloak launcher - Wire tracer from registry into pmanager and tmanager provision handlers - Export TelemetryServiceAssembly type, switch to WithBatcher, use semconv - Remove common/runtime/telemetry.go (superseded by common/telemetry/assembly.go) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…setup Remove TelemetryServiceAssembly and the TracerProviderKey service registry dependency in favor of a SetupTelemetry() helper in common/runtime that configures the global OpenTelemetry provider. Components now obtain tracers via otel.GetTracerProvider() instead of resolving from the service registry, eliminating explicit tracer injection across handlers, NATS clients, and agents. Tracer names standardized to cfm.<component> convention. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…tion - Add tracer to EDCVActivityProcessor with spans for deploy, IdentityHub, and control plane operations - Add spans to onboarding activity for credential request and check flows - Add span to registration activity for holder creation - Wrap HTTP client transport with otelhttp for automatic outbound trace propagation - Restore trace context from orchestration processing data before starting new spans to maintain trace continuity across rescheduled activities Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
instrumentation
paullatzelsperger
requested review from
jimmarino and
wolf4ood
and removed request for
wolf4ood
wolf4ood
approved these changes
Apr 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds end-to-end OpenTelemetry (OTel) distributed tracing across the CFM service mesh.
SetupTelemetry()helper incommon/runtimethat configures the global OTel provider; components obtain tracers viaotel.GetTracerProvider()with standardizedcfm.<component>namesnatsHeaderCarrierTextMapCarrier, maintaining trace continuity across async message boundariesotelhttpfor automatic trace propagation on all external calls; all HTTP requests are bound to acontext.Contextto enable thisadding metrics and logs will come in a subsequent PR
🤖 Generated with Claude Code