Skip to content

fix: prevent research image CI timeout - #797

Merged
proerror77 merged 2 commits into
mainfrom
codex/research-image-timeout
Aug 10, 2026
Merged

proerror77 merged 2 commits into
mainfrom
codex/research-image-timeout

Conversation

@proerror77

@proerror77 proerror77 commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Summary

Prevent the research-image production chain from self-cancelling during cache persistence.

  • Increase the Research image binaries timeout from 30 to 45 minutes. The mainline cold-cache build completes at roughly 27 minutes, then needs additional time for the post-job target-cache archive.
  • Keep sccache reads but make its GitHub Actions backend read-only. The measured cold build issued about 2,000 object writes, hit GitHub cache throttling (873 write errors), and achieved only 0.73% hit rate. The existing rust-cache target archive is the durable warm-start cache; it saves as one artifact after the job completes.
  • Lock both safeguards into the existing CI contract test.

Issue relationship

None

Focused validation

  • .github/scripts/test-select-rust-ci-scope.sh passed locally.
  • actionlint .github/workflows/ploy-ci.yml passed locally when available.
  • git diff --check passed.

Rollout and rollback

Merge adds a 15-minute CI timeout reserve and stops sccache write attempts only for this heavy job. A successful main run must show a completed target-cache save; the following main run must show a target-cache restore. Revert this PR to restore prior behavior.

Out of scope

No collector, uploader, data, credential, ACR, or live-trading runtime change.

@proerror77
proerror77 enabled auto-merge (squash) August 10, 2026 03:21
@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The research-image-binaries job timeout increased from 30 to 45 minutes. A CI contract check now verifies the 45-minute setting.

Changes

Research image timeout

Layer / File(s) Summary
Update workflow timeout and contract
.github/workflows/ploy-ci.yml, .github/scripts/test-select-rust-ci-scope.sh
The job timeout is set to 45 minutes. The CI test requires timeout-minutes: 45.

Estimated code review effort: 2 (Simple) | ~5 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the primary change: preventing research image CI timeouts.
Description check ✅ Passed The description covers the change, issue relationship, validation, rollout, and out-of-scope work, but omits explicit dependency and scope-exception sections.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/research-image-timeout

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@proerror77
proerror77 merged commit e60c44a into main Aug 10, 2026
46 of 47 checks passed
@proerror77
proerror77 deleted the codex/research-image-timeout branch August 10, 2026 03:34
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