Skip to content

ci: extend behavior test timeout - #43

Open
ruby-dlee wants to merge 4 commits into
mainfrom
fm/fm-auto-reap-ci-timeout
Open

ci: extend behavior test timeout#43
ruby-dlee wants to merge 4 commits into
mainfrom
fm/fm-auto-reap-ci-timeout

Conversation

@ruby-dlee

@ruby-dlee ruby-dlee commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Intent

Raise only the Firstmate Behavior GitHub Actions job timeout from 45 minutes to 90 minutes so the unchanged full behavior suite can finish in isolated CI. Do not weaken, skip, delete, shard, or serialize away tests or assertions, and make no other repository change. This is a separate narrow dependency PR for the automatic reaping work. The documented environmental local aggregate-test gate is accepted because concurrent fleet load and shared-machine contention make that replay non-authoritative; isolated GitHub CI must pass. The exact final PR head must receive review. Push/update PR 43, but do not merge or deploy. The deterministic local aggregate reproduction of the pre-existing account-routing assertion is therefore not an authorized repository fix; preserve the timeout-only diff.

What Changed

  • Increase the GitHub Actions Behavior tests job timeout from 45 to 90 minutes while leaving the full test suite unchanged.

Risk Assessment

✅ Low: The final diff is strictly limited to increasing only the Behavior tests job timeout from 45 to 90 minutes, with the full suite and all other repository content unchanged.

Testing

  • ⏭️ Test - skipped

Pipeline

Updates from git push no-mistakes

✅ **intent** - passed

✅ No issues found.

✅ **Rebase** - passed

✅ No issues found.

✅ **Review** - passed

✅ No issues found.

⏭️ **Test** - skipped

Step was skipped.

✅ **Document** - passed

✅ No issues found.

✅ **Lint** - passed

✅ No issues found.

✅ **Push** - passed

✅ No issues found.

@ruby-dlee ruby-dlee changed the title ci: extend behavior test timeout ci: extend behavior test timeout to 90 minutes Jul 28, 2026

@ruby-dlee ruby-dlee left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed exact head 9955918 against the stated narrow intent. The PR changes only the Behavior job timeout from 45 to 90 minutes; it does not alter the behavior suite, tests, assertions, sharding, or serialization. The workflow YAML parses and the diff is whitespace-clean.

@ruby-dlee ruby-dlee changed the title ci: extend behavior test timeout to 90 minutes ci: extend behavior test timeout Jul 28, 2026

@ruby-dlee ruby-dlee left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed exact head 032a74e against the authoritative timeout-only intent. The PR diff changes only the Behavior job timeout from 45 to 90 minutes; it does not alter, skip, weaken, shard, delete, or serialize tests or assertions. git diff --check and the targeted workflow timeout assertion pass. The reported account-routing assertion is reproduced on base/main commit 18f29ab and is therefore not attributable to this PR.

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.

1 participant