Skip to content

feat(otel): parent durable spans to shared trace - #693

Merged
ayushiahjolia merged 1 commit into
mainfrom
otel-shared-trace-pr3-plugins
Sep 1, 2026
Merged

feat(otel): parent durable spans to shared trace#693
ayushiahjolia merged 1 commit into
mainfrom
otel-shared-trace-pr3-plugins

Conversation

@ayushiahjolia

Copy link
Copy Markdown
Contributor

Issue #, if available: #674

Description of changes:

  • Third PR in the stack for reparenting durable OTel spans onto one shared execution trace.
  • Wires both plugins (InvocationOtelPlugin and ExecutionOtelPlugin) onto the shared execution trace and sampling. Workflow and Invocation spans now share one trace, anchored to a propagated backend parent when present or a deterministic synthetic execution root otherwise; an ambient Lambda span is used as parent only when it already belongs to the execution trace.
  • Sampling is resolved once per invocation and applied to every durable span. Also completes the context_extractors rewrite deferred from the first PR: extractors now return a structured ExtractedContext instead of an opaque OTel Context.
  • Next in stack: docs (README) update for the shared-trace model.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ayushiahjolia

Copy link
Copy Markdown
Contributor Author

/ai review

@ayushiahjolia
ayushiahjolia marked this pull request as ready for review August 31, 2026 18:26
@ayushiahjolia
ayushiahjolia deployed to ai-pr-review-runtime August 31, 2026 18:26 — with GitHub Actions Active
zhongkechen
zhongkechen previously approved these changes Sep 1, 2026
Base automatically changed from otel-shared-trace-pr2-sampling to main September 1, 2026 18:37
@ayushiahjolia
ayushiahjolia merged commit ee22f2b into main Sep 1, 2026
12 of 42 checks passed
@ayushiahjolia
ayushiahjolia deleted the otel-shared-trace-pr3-plugins branch September 1, 2026 19:42
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