Skip to content

Tracing with opentelemetry#895

Merged
Assem-Uber merged 19 commits intocadence-workflow:masterfrom
Assem-Uber:feature/12765/tracing
May 13, 2025
Merged

Tracing with opentelemetry#895
Assem-Uber merged 19 commits intocadence-workflow:masterfrom
Assem-Uber:feature/12765/tracing

Conversation

@Assem-Uber
Copy link
Contributor

@Assem-Uber Assem-Uber commented May 12, 2025

Summary

Add opentelemetry sdk traces to support tracing for common node server operations (Pages rendering and http endpoints)

Changes

  • Create register otel utility to create the node sdk with default configurations
  • Instrument gRPC/HTTP/Fetch by default
  • Propagate trace and baggage using JaegerPropagator
  • add '@grpc/grpc-js', 'require-in-the-middle', '@opentelemetry/exporter-jaeger' as external services to prevent nextjs from bundling them.
  • otel traces are exported using gRPC exporter

Testing

Tested locally by:

  • Running Jaeger container locally
  • Setting env varaiable OTEL_SDK_DISABLED=false
  • Setting OTEL_EXPORTER_OTLP_TRACES_ENDPOINT Jaeger collector http endpoint. By default it is http://localhost:4317

@Assem-Uber Assem-Uber changed the title Feature/12765/tracing Tracing with opentelemetry May 13, 2025
@Assem-Uber Assem-Uber merged commit c0a6241 into cadence-workflow:master May 13, 2025
1 check passed
@Assem-Uber Assem-Uber deleted the feature/12765/tracing branch May 13, 2025 11:14
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.

2 participants