ci: schedule long-running OTel conformance tests - #665
Conversation
Claude AI reviewNo actionable findings. This PR adds a daily
Residual test risk: The PR's stated intent — that scheduled events "drive the reusable orchestrator's automatic long-running check/launch cycle" — depends on the external Reviewed commit |
| - "packages/aws-durable-execution-sdk-python-conformance-tests-otel/**" | ||
| - ".github/workflows/opentelemetry-conformance-tests.yml" | ||
| schedule: | ||
| - cron: "0 7 * * *" |
There was a problem hiding this comment.
Codex AI review
[P2] GitHub warns that workflows scheduled at the start of an hour may be delayed or dropped under load; this repository already avoids :00 for that reason in stale-issue-closer.yml. A dropped run skips the long-running check/launch cycle for the day. Use a nonzero minute, such as 7 7 * * *, and update the corresponding test assertion.
Codex AI reviewOne reliability issue: the daily schedule uses GitHub's peak-load minute. The contract test only validates workflow text, not scheduled execution. Reviewed commit |
Summary
Related issue: aws/aws-durable-execution-conformance-tests#90
Related PRs:
Testing
test_opentelemetry_conformance_caller_uses_current_workflow_contractdirectly.github/workflows/opentelemetry-conformance-tests.ymlsuccessfully with Ruby YAMLgit diff --check