Skip to content

Update max retries in all build images workflows to 5#683

Merged
juanmichelini merged 1 commit intomainfrom
openhands/update-max-retries-to-5
Apr 21, 2026
Merged

Update max retries in all build images workflows to 5#683
juanmichelini merged 1 commit intomainfrom
openhands/update-max-retries-to-5

Conversation

@juanmichelini
Copy link
Copy Markdown
Collaborator

Summary

Increases the default max-retries from 2 to 5 in all build image workflows to reduce unnecessary reruns (which often succeed on retry due to cache).

Affected workflows:

  • build-swebench-images.yml
  • build-swebenchmultimodal-images.yml
  • build-swtbench-images.yml
  • build-swesmith-images.yml
  • build-swegym-images.yml
  • build-multiswebench-images.yml

fixes #682

@juanmichelini can click here to continue refining the PR

Increases the default max-retries from 2 to 5 to reduce unnecessary
reruns when building images (which often work due to cache).

Co-authored-by: openhands <openhands@all-hands.dev>
@juanmichelini juanmichelini marked this pull request as ready for review April 21, 2026 00:48
Copy link
Copy Markdown
Collaborator

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Good taste - Clean, focused configuration change.

[RISK ASSESSMENT]

  • [Overall PR] ⚠️ Risk Assessment: 🟢 LOW

Simple workflow configuration update that addresses a real operational problem (transient build failures due to caching). The change is applied consistently across all 6 affected workflows. Users can still override the default value when triggering workflows manually. No code logic or user-facing APIs affected.

VERDICT:
Worth merging: Pragmatic solution to reduce unnecessary workflow reruns.

KEY INSIGHT:
Increasing retry tolerance from 2 to 5 is a proportional response to transient infrastructure failures - it trades slightly longer worst-case runtime for significantly fewer manual reruns.

@juanmichelini juanmichelini merged commit ffa5498 into main Apr 21, 2026
4 checks passed
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.

Update max retries in all build images workflows to 5

3 participants