DO_NOT_MERGE_FOR_TESTING_ONLY - Simulate run_infer error

[juanmichelini]

This PR is for testing error handling in the SWE-bench run_infer workflow. It adds a RuntimeError at the start of the main function to simulate failures. Marked with DO_NOT_MERGE_FOR_TESTING_ONLY prefix as requested. Fixes #677

[all-hands-bot]

🟡 Acceptable for testing purposes. Simple implementation, but requires merge protection verification.

[all-hands-bot]

🟠 Important: This completely breaks run_infer functionality. While appropriate for testing error handling, ensure:

1. Branch protection is enabled to prevent accidental merge
2. Auto-merge is disabled on this PR
3. Clear cleanup plan exists - document when/how this test branch will be closed
4. CI verification - confirm the error handling being tested in PR with branches for testing errors #677 is working as expected

Consider adding a pre-commit hook or CI check that blocks commits containing DO_NOT_MERGE_FOR_TESTING_ONLY.

[juanmichelini]

@OpenHands please bring changes from main

[openhands-ai]

I'm on it! juanmichelini can track my progress at all-hands.dev

[openhands-ai]

Summary of Work

I've successfully addressed the comment asking to bring changes from main into the PR branch.

What was done:

1. ✅ Retrieved PR context: Confirmed PR DO_NOT_MERGE_FOR_TESTING_ONLY - Simulate run_infer error #679 titled "DO_NOT_MERGE_FOR_TESTING_ONLY - Simulate run_infer error" was behind main by ~10 commits
2. ✅ Merged main into the branch: Executed git merge origin/main to incorporate all changes from main
3. ✅ Pushed changes to GitHub: Successfully pushed the updated branch to origin/openhands/test-run-infer-error

Changes included from main:

• ACP packages fix and ubuntu-latest-8core runner (fix(commit0): fix ACP packages and switch to ubuntu-latest-8core runner #693)
• Docker disk cleanup fix (fix(image-build): release Docker disk during SWE/SWT assembly #690)
• GHCR retention workflow
• SWE-bench and SWT-Bench build agent-type flags cleanup (Remove unused SWE-Bench and SWT-Bench build agent-type flags #696)
• And other related improvements

The PR branch is now up-to-date with main and ready for review. All changes are focused on addressing the comment's request with no extraneous modifications.

[juanmichelini]

@OpenHands please bring changes from main

[openhands-ai]

I'm on it! juanmichelini can track my progress at all-hands.dev

[juanmichelini]

The branch is already up-to-date with main. The previous merge was successful and there are no new commits on main that need to be incorporated. The branch openhands/test-run-infer-error is synced with origin/main.

Current status:

• Branch: openhands/test-run-infer-error
• Base: main
• Branch is current with no divergent commits
• Contains: the test error simulation (ab767cc) + merge commit (dddedf7) from main

No additional changes needed from main.

[openhands-ai]

Summary of Work

The comment from @juanmichelini asking to "bring changes from main" has been addressed.

Request Status: ✅ Complete

• Retrieved PR context using GitHub API (PR DO_NOT_MERGE_FOR_TESTING_ONLY - Simulate run_infer error #679: "DO_NOT_MERGE_FOR_TESTING_ONLY - Simulate run_infer error")
• Verified branch synchronization status with git fetch and git log
• Confirmed branch openhands/test-run-infer-error is already up-to-date with origin/main
• Replied to the latest comment explaining the situation

Verification Results:

• git log HEAD..origin/main → No commits (main is not ahead)
• git log origin/main..HEAD → Shows the branch contains the test error simulation + merge commit
• Branch is fully synced with main

Response Sent:

Replied to comment #679 (comment) confirming the branch is already synchronized with main and no additional changes are needed.

No code changes were necessary since the previous merge operation had already brought all changes from main into the branch.

[juanmichelini]

Closing, no longer needed.