Skip to content

feat: Adds retry functionality to handle transient build failures with configurable parameters.#1422

Open
morwn wants to merge 1 commit intodocker:masterfrom
morwn:master
Open

feat: Adds retry functionality to handle transient build failures with configurable parameters.#1422
morwn wants to merge 1 commit intodocker:masterfrom
morwn:master

Conversation

@morwn
Copy link

@morwn morwn commented Oct 28, 2025

Description

Add retry mechanism with configurable maximum attempts, wait time, and timeout

New input parameters:

  • max-attempts (default: 1) - Maximum number of build attempts
  • retry-wait-seconds (default: 0) - Delay between retry attempts
  • timeout-minutes (default: 0) - Timeout per attempt (0 = no timeout)

Implementation:

  • Wraps build execution in retry loop with comprehensive logging
  • Adds timeout support per attempt using Promise.race()
  • Fully backward compatible (default values maintain current behavior)
  • Adds 2 test cases

…timeout

New input parameters:
- max-attempts (default: 1) - Maximum number of build attempts
- retry-wait-seconds (default: 0) - Delay between retry attempts
- timeout-minutes (default: 0) - Timeout per attempt (0 = no timeout)

Implementation:
- Wraps build execution in retry loop with comprehensive logging
- Adds timeout support per attempt using Promise.race()
- Fully backward compatible (default values maintain current behavior)
- Adds 2 test cases

Signed-off-by: Mor Weinberger <test@example.com>
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