Skip to content

feat: add OpenTelemetry tracing support - #419

Draft
miyunari wants to merge 1 commit into
mainfrom
tracing
Draft

feat: add OpenTelemetry tracing support#419
miyunari wants to merge 1 commit into
mainfrom
tracing

Conversation

@miyunari

Copy link
Copy Markdown

Summary

  • Add OpenTelemetry tracing with OTLP gRPC exporter
  • New CLI flags: --tracing-enabled, --tracing-endpoint, --tracing-insecure, --tracing-stdout
  • HTTP handler wrapped with otelhttp.NewHandler for automatic span creation
  • Uses shared tracing library from sigstore/model-validation-operator/pkg/tracing

Changes

  • cmd/timestamp-server/app/root.go: Add 4 tracing CLI flags
  • cmd/timestamp-server/app/serve.go: Initialize tracing provider on startup
  • pkg/generated/restapi/configure_timestamp_server.go: Wrap HTTP handler with otelhttp.NewHandler
  • go.mod/go.sum: Add tracing dependencies

Test plan

  • Verify build passes
  • Deploy with --tracing-enabled --tracing-endpoint <collector> and verify traces in Jaeger
  • Verify --tracing-stdout outputs traces to stdout
  • Verify no impact when tracing is disabled (default)

Add distributed tracing using OpenTelemetry with OTLP gRPC exporter.
HTTP handler wrapped with otelhttp for automatic span creation.
Tracing is controlled via CLI flags.

Uses the shared tracing library from sigstore/model-validation-operator.

Signed-off-by: Nina Bongartz <nolear@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant