Repository navigation
fix(ci): only apt-installed live provers refresh the apt index (stacked on #455) - #459
Conversation
The T1 provision step ran `apt-get update` before every backend, including vampire, cvc5, alt-ergo and chuffed, which install from GitHub release assets and never touch apt. On #446 the image's packages.microsoft.com source answered 403 twice, so `T1 / vampire` failed before its test ran. The update now runs inside an apt_install helper that only the apt-installed backends (z3, eprover, spass, glpk, minizinc) call. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015jzCHZx4Rt65NDaGLDrjLp
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the
|
|
@coderabbitai review This PR is stacked on #455, so its base isn't the default branch and auto-review skipped it. Requesting a review now so findings land before it retargets to main. Generated by Claude Code |
Action not completed
|
|
Static Analysis Gate / panic-attack assail is red here, but this PR didn't cause it.
Generated by Claude Code |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|



Summary
Stacked on #455, which rewrites the same lines of
live-provers.ymland carries the repair for main's current startup failures. Merge #455 first; GitHub then retargets this PR tomain.The T1 provision step ran
apt-get updatebefore every backend, including vampire, cvc5, alt-ergo and (with #455) chuffed, which install from GitHub release assets and never touch apt. On #446 the runner image'spackages.microsoft.comsource answered 403 twice, soT1 / vampirefailed before any test ran (job 118637174820). The re-run passed, so it was the mirror and not the prover.Changes
live-provers.yml, T1 provision step:apt-get updatemoves into anapt_installhelper. Only the apt-installed backends call it (z3, eprover, spass, glpk, minizinc). The release-asset backends no longer depend on the image's apt sources being reachable.An earlier version of this branch also pinned the T1 Guix job to
ubuntu-24.04. I dropped that, because #455 makes the same fix with a pinned guix version and a running daemon.RSR Quality Checklist
Required
actionlintwith shellcheck, exit 0unsafeblocks without// SAFETY:comments: n/a.envfiles includedAs Applicable
Testing
Run locally on this tree:
actionlint(with shellcheck) onlive-provers.yml: clean.scripts/ci/test-issue-regressions.sh(with mikefarah yq v4): passed.scripts/ci/check-allow-fail-expiry.sh: 10 units, passed.sudo apt-getnow appears only inside the helper.uses:changed, soactions.lockis unaffected.This PR's own CI is the live test of the T1 matrix on the new step.
Generated by Claude Code