Skip to content

fix(otel): set invocation span metadata#577

Merged
zhongkechen merged 2 commits into
mainfrom
codex/fix-otel-invocation-span-metadata
Jul 23, 2026
Merged

fix(otel): set invocation span metadata#577
zhongkechen merged 2 commits into
mainfrom
codex/fix-otel-invocation-span-metadata

Conversation

@zhongkechen

@zhongkechen zhongkechen commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • explicitly mark every span created by the OTel plugin as SpanKind.INTERNAL
  • add durable.invocation.first as a boolean when each invocation span starts
  • add durable.invocation.status as a string when each invocation span ends
  • keep invocation span status UNSET while an execution is pending or retrying
  • mark the terminal invocation span OK for successful executions and ERROR for failed executions
  • add regression coverage for span kinds, both first-invocation values, and every invocation status

issue: aws/aws-durable-execution-conformance-tests#23

Verification

  • hatch run dev-otel:test (60 passed)
  • hatch run dev-otel:typecheck
  • hatch fmt --check from the OTel package
  • hatch run python .github/scripts/lintcommit.py

@zhongkechen zhongkechen changed the title fix(otel): set invocation span metadata fix(otel): align span metadata and attempt parenting Jul 23, 2026
@zhongkechen
zhongkechen force-pushed the codex/fix-otel-invocation-span-metadata branch from c97a95f to 6098c5f Compare July 23, 2026 19:41
@zhongkechen zhongkechen changed the title fix(otel): align span metadata and attempt parenting fix(otel): set invocation span metadata Jul 23, 2026
@zhongkechen zhongkechen self-assigned this Jul 23, 2026
@zhongkechen
zhongkechen merged commit 160b9ef into main Jul 23, 2026
17 of 25 checks passed
@zhongkechen
zhongkechen deleted the codex/fix-otel-invocation-span-metadata branch July 23, 2026 20:08
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