[DO NOT MERGE] ci(swtbench): revert to Blacksmith runner (fallback for #495)#673
Draft
simonrosenberg wants to merge 1 commit into
Draft
[DO NOT MERGE] ci(swtbench): revert to Blacksmith runner (fallback for #495)#673simonrosenberg wants to merge 1 commit into
simonrosenberg wants to merge 1 commit into
Conversation
Surgical revert of the swtbench portion of #651 — puts the SWT-bench image-build workflow back on blacksmith-32vcpu-ubuntu-2204 with the useblacksmith/setup-docker-builder builder. Blacksmith's larger persistent disk masked the unbounded local-image growth that now fails on ubuntu-latest-8core (evaluation#495). Kept intact from main: - workflow_call trigger and cross-repo checkout (#666) - all input/env plumbing and downstream steps Intended as a parallel fallback branch while the real fix in #672 (docker rmi after push + free-disk-space + preflight) is validated.
Collaborator
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Parallel to #672 (the real fix). Keeps a known-good configuration on hand to dispatch against if the
docker rmi+free-disk-space+ preflight approach in #672 doesn't hold up under real load.What this does
Surgical revert of the swtbench portion of #651: puts
build-swtbench-images.ymlback onblacksmith-32vcpu-ubuntu-2204+useblacksmith/setup-docker-builder@v1. Everything added since — notably theworkflow_calltrigger + cross-repo checkout from #666 — is preserved.Why this "works"
Blacksmith runners have hundreds of GB of persistent disk, which masks the underlying bug in
assemble_agent_image(local images retained after push). The bug is still there; the runner just has enough headroom that 433 images don't fill it.So this is explicitly not the fix — it's the "revert the migration" escape hatch. The real fix is #672.
Relationship to other work
AI disclosure
This PR was prepared by Claude (Claude Code) on behalf of @simonrosenberg.