Skip to content

fix(otel): keep continuation spans within invocation - #601

Merged
zhongkechen merged 1 commit into
mainfrom
codex/fix-invocation-continuation-timestamp
Aug 7, 2026
Merged

fix(otel): keep continuation spans within invocation#601
zhongkechen merged 1 commit into
mainfrom
codex/fix-invocation-continuation-timestamp

Conversation

@zhongkechen

Copy link
Copy Markdown
Contributor

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Issue Link, if available

Observed in the OpenTelemetry invocation-view conformance run: https://github.com/aws/aws-durable-execution-conformance-tests/actions/runs/31059801062/job/92519884437?pr=69

Description

Invocation-view continuation spans were backdated to the logical operation start timestamp from a prior Lambda invocation. Because the continuation is parented to the current invocation span, this made it start before its parent and overlap the original STARTED operation span.

Allow continuation spans to use their creation time in the current invocation. The execution-view plugin continues to use the original operation timestamp for its single logical cross-invocation span. The unit test now verifies that a continuation is parented to and temporally contained by the current invocation span.

Demo/Screenshots

Not applicable.

Checklist

  • I have filled out every section of the PR template
  • I have thoroughly tested this change

Testing

Unit Tests

Updated InvocationOtelPluginTest with a regression test covering continuation span parentage and temporal containment.

Integration Tests

Ran mvn -pl otel-plugin -am test: reactor build succeeded, including all 148 OpenTelemetry plugin tests and 17 invocation plugin integration tests. No new integration test was needed for this isolated span timestamp change.

Examples

Not applicable.

@zhongkechen
zhongkechen requested a review from a team August 6, 2026 21:04
@zhongkechen
zhongkechen requested a deployment to ai-pr-review-runtime August 6, 2026 21:34 — with GitHub Actions Waiting
@zhongkechen
zhongkechen requested a deployment to ai-pr-review-runtime August 6, 2026 21:34 — with GitHub Actions Waiting
@zhongkechen zhongkechen self-assigned this Aug 6, 2026
@zhongkechen
zhongkechen merged commit 18b1f90 into main Aug 7, 2026
11 of 13 checks passed
@zhongkechen
zhongkechen deleted the codex/fix-invocation-continuation-timestamp branch August 7, 2026 00:15
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