ci: schedule long-running OTel conformance tests - #665
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
fc0bd21 to
a48092e
Compare
| - "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] Move the schedule off minute zero
GitHub may delay or drop scheduled runs during the high-load start-of-hour window. A dropped run here skips the automatic check/launch cycle for a full day. Use a non-zero minute, as the repository already does for other schedules, and update the test assertion accordingly.
Codex AI reviewFound one reliability issue in the new scheduled trigger. The external reusable workflow implementation is outside the supplied diff. Reviewed commit |
Claude AI reviewCI-only change to No actionable findings in the changed lines.
Residual risk (unverifiable here, not a defect in this diff): the PR aims for scheduled events to drive the orchestrator's long-running check/launch cycle, but the caller still passes 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