[OTel]: Complete replay and chained-execution conformance coverage
維護者通常 1 天內回覆
還沒有人認領這個 Issue。
評估
- 難度
- 5/5
- 預估耗時
- 一週以上
- 新手友好度
- 25/100
- Issue 類型
- 功能
- 描述清晰度
- 描述清楚
- 活躍度
- 活躍
- 技術堆疊
- aws, github-actions, python
- 領域
- backend, observability, testing
研究方向
Start with the conformance package at the referenced commit, especially its handlers, template.yaml, src/Makefile, README, local tests, and tests/test_otel_examples.py; read the linked shared PR #131 and workflow contract tests before changing caller inputs. Determine the service guarantee for case 21 event ordering from cloud history and trace evidence, then update the owning fixture or requirement if ordering is variable. Done means focused cases 21–24 and 27 are registered and documented in both views, deferred cases remain explicitly unsupported, and cloud qualification results and revisions are recorded.
由索引模型根據 Issue 內容生成。
描述
Parent: #873 — Remaining Python OpenTelemetry work.
Track the remaining conformance coverage and qualification represented by PR #796, using dc736d5 as the current reference. Implement through focused conformance PRs; keep runtime and testing-library fixes in their existing issues.
Coverage to deliver
| Case | Required behavior |
|---|---|
| 21 — completed-step replay | A completed step body runs once and returns its saved result after a durable wait/resume. Validate telemetry and replay independently of incidental event timing. |
| 22 — user-function context | Ordinary user spans inherit the SDK-owned handler, step/condition attempt, child, branch, and iteration contexts without supplying a repaired parent. |
| 23 — callback-function context | Callback submitters, wrapped retry callbacks, and virtual-child bodies retain the expected tracing parent in both views. |
| 24 — invocation retry | A retryable invocation error produces RETRYING/UNSET, followed by successful replay using the saved step result. |
| 27 — suspended chained child | A real chained child performs a one-second durable wait, resumes, and returns the original payload; check caller/child invocations, wait identity, parentage, and continuation links. |
Case 11's replay-barrier fix is already delivered by merged #864. Reuse it as a regression control rather than duplicating that implementation. PR #796 currently targets main; its description still contains historical stack text, so use the actual branch state when preparing a smaller PR.
Implementation scope
- The owning package is
packages/aws-durable-execution-sdk-python-conformance-tests-otel: handlers,template.yaml,src/Makefile, README, and local public-handler tests. - Keep requirement-to-handler mappings, deployed function names/targets, and the local catalog assertions synchronized in both Invocation and Execution views.
- Preserve the current shared workflow/catalog defaults to the conformance repository's
main, the manualconformance_test_refoverride, and SDK handlers built from the exact commit under test. Validate the shared workflow's current inputs/secrets contract when changing the caller. - The shared orchestrator runs the Invocation and Execution suites. Existing legacy long-running templates/tests are not a request to re-enable that suite.
- Coordinate language-neutral requirements and runner changes in aws/aws-durable-execution-conformance-tests, including the related work in shared PR #131. Use the public SDK API rather than compensating for a missing SDK capability in a handler.
Deferred callback qualification
Keep cases 25–26 explicitly NOT_IMPLEMENTED with issue references until their prerequisites are ready. Their deferred source and tests remain available at 6484e577.
- Case 25: requires terminal status/event semantics (#853) and empty-error history/runtime consistency (#862).
- Case 26: requires replay-completion telemetry (#854), callback updated-operation production (#860), and input acknowledgment (#861). Preserve the assertion inside the observed step that completion has already reached the exporter, plus raw export counts at both later barriers; recorded timestamps alone do not prove delivery order.
- Additional case 17 error regression: requires error snapshot isolation (#852) and the applicable callback-error fixes. The existing case 17 handler remains active.
Before restoring these tests, correct their history expectations: included diagnostic payloads are untruncated, while metadata-only history omits payloads and cannot reconstruct error details. The empty callback-failure Error.Payload service shape remains unverified against deployed AWS Lambda and needs evidence or an explicit qualification decision.
Known qualification gap
PR #796 reports that the S3 Execution-view job failed case 21 because WaitSucceeded preceded InvocationCompleted by 63 ms, contrary to a fixed history-order expectation. The execution returned before-after successfully. Determine which event order the service contract guarantees, collect the relevant trace/history evidence, and fix the owning requirement/fixture if the order is legitimately variable. Preserve the saved-step, result, and telemetry assertions; do not count execution success alone as conformance.
Acceptance criteria
- Cases 21–24 and 27 have public-API handlers, template/Makefile registration, documentation, and focused local regressions in both views.
- Case 11 remains covered by #864's implementation and tests.
- The complete catalog accounts for active and deferred requirements; unsupported cases are never reported as passing.
- Case 21's cloud history-order failure is resolved or explicitly dispositioned with service evidence and the appropriate shared-repository change.
- Restore deferred cases 25–26 and the additional case 17 regression after their implementation prerequisites land, through separate focused PRs as needed.
- Qualify the intended behavior against the deployed Invocation/Execution backend matrix, including raw export counts and parent/trace identity. Link exact SDK/catalog revisions and run results.
- Update the PR/README to reflect current coverage, dependencies, and validation; identify changed test expectations and their reasons.
Starting points and validation
Reference files at the current PR head:
Use the repository's pinned Hatch version and run the package tests and focused workflow tests, plus package lint/format and commit-message checks. The PR description reports 75 local package tests on Python 3.13.17 and an earlier 10-test workflow validation; those are historical results, not new validation performed while filing this issue. Cloud qualification remains a separate requirement.
- 主要語言
- Python
- 星號
- 58
- 分支
- 32
- 平均合併
- 1 天 6 小時
- 30 天內合併 PR
- 80
環境準備
- 沒有 Dockerfile 或 Docker Compose 檔案
- 有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue,再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫,在一個分支上完成修改。
- 送出 Pull Request,並在描述裡引用這個 Issue 編號。
aws/aws-durable-execution-sdk-python 的其他 Issue
-
enhancement needs-triage
難度 1/5 1-3 小時 新手友好度 72/100
aws/aws-durable-execution-sdk-python#927 ·
維護者通常 1 天內回覆
-
enhancement
難度 2/5 1-3 小時 新手友好度 82/100
aws/aws-durable-execution-sdk-python#875 ·
維護者通常 1 天內回覆
-
enhancement
難度 1/5 1 小時以內 新手友好度 85/100
aws/aws-durable-execution-sdk-python#794 ·
維護者通常 1 天內回覆
-
bug
難度 3/5 1-2 天 新手友好度 58/100
aws/aws-durable-execution-sdk-python#921 ·
維護者通常 1 天內回覆
-
[Bug]: JS examples CI still invokes Jest after Vitest migration可能已有人在做 關聯的 PR 仍在進行中或已合併。 未關閉bug needs-triage
難度 3/5 1-2 天 新手友好度 55/100
aws/aws-durable-execution-sdk-python#918 ·
維護者通常 1 天內回覆
查看 aws/aws-durable-execution-sdk-python 的全部 Issue
相似的 Issue
-
physical
難度 2/5 1-3 小時 新手友好度 74/100
antonyrag/ragleap-core#1339 ·
-
enhancement good first issue
難度 2/5 1-3 小時 新手友好度 83/100
AllenCell/SubCellSeg#5 ·
-
[🐛 Bug]: oidc_authverifier.py strips trailing slash causing "Invalid token: Invalid issuer" error未關閉
難度 2/5 1-3 小時 新手友好度 76/100
keephq/keep#6934 · 1 個 reaction ·
維護者通常 1 天內回覆
-
bug game: 01008BA02525A000 ios
難度 2/5 1-3 小時 新手友好度 76/100
維護者通常 1 天內回覆
-
needs-triage
難度 1/5 1 小時以內 新手友好度 87/100
gke-labs/kube-agents#5501 ·
維護者通常 1 天內回覆