Skip to content

fix(ci): Round-21 convention timeout + coverageBaseRef contract - #25

Closed
beyondwin wants to merge 5 commits into
mainfrom
fix/ci-main-timeouts
Closed

beyondwin wants to merge 5 commits into
mainfrom
fix/ci-main-timeouts

Conversation

@beyondwin

Copy link
Copy Markdown
Owner

Summary

  • Raise repository/CI outer :build-logic:convention:test timeout from 15m to 27m to match sealed Round-21 boundSeconds=1620, and bump static-analysis job timeout to 60m so lint + suite fit.
  • Update sealed Task-9 contracts/docs (repositoryAndNestedTimeoutMinutes 15→27, failure summary seconds 900→1620) and regenerate build-inputs.json.
  • Fix verify_test.sh so -Pgasstation.coverageBaseRef= is only required when merge-base ≠ HEAD (main tip correctly omits it).

Test plan

  • ./scripts/agent/tests/verify_test.sh
  • python3 scripts/quality/verify_build_inputs.py verify
  • focused quality unit tests (test_testkit_failure_evidence, test_local_colima_evidence, test_build_inputs, test_test_class_decomposition)
  • GitHub Android CI green on this PR (agent-contracts, build-inputs, static-analysis)

Made with Cursor

beyondwin and others added 5 commits August 28, 2026 21:10
Main CI killed :build-logic:convention:test at 15m while sealed Linux LPT already budgets 1620s. Raise the repository outer timeout to 27m, give static-analysis job headroom, and make coverageBaseRef assertions match merge-base==HEAD omission on main tip.

Co-authored-by: Cursor <cursoragent@cursor.com>
check_contracts still required the old 30m static-analysis cap, which rejected the Round-21 suite headroom bump.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sealed evidence entrypoints pin scripts/agent/check_contracts.py by SHA-256.

Co-authored-by: Cursor <cursoragent@cursor.com>
27m matched Round-21's 14-CPU LPT bound, but GitHub ubuntu-24.04 still aborted mid-suite. Give repository/CI outer TestKit 40m headroom while keeping nested module Test at 15m and sealed-host outer at 35m.

Co-authored-by: Cursor <cursoragent@cursor.com>
Five parallel TestKit Gradle daemons oversubscribe hosted 4-core runners, so the suite aborted at 15m, 27m, and 40m. Keep the sealed 5-fork contract for Colima and run only two forks on GITHUB_ACTIONS.

Co-authored-by: Cursor <cursoragent@cursor.com>
@beyondwin beyondwin closed this Aug 29, 2026
@beyondwin
beyondwin deleted the fix/ci-main-timeouts branch August 29, 2026 02:42
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