Skip to content

script: Retry related fixes#41

Merged
joamaki merged 2 commits intomainfrom
pr/joamaki/retry-fixes
Feb 25, 2025
Merged

script: Retry related fixes#41
joamaki merged 2 commits intomainfrom
pr/joamaki/retry-fixes

Conversation

@joamaki
Copy link
Copy Markdown
Contributor

@joamaki joamaki commented Feb 25, 2025

  • Use the default retry intervals if they're zero
  • Require at least one retry before taking the output with -scripttest.update

Signed-off-by: Jussi Maki <jussi.maki@isovalent.com>
To make -scripttest.update slightly safer to use, require that the
section has been retried at least once before accepting the output.

Signed-off-by: Jussi Maki <jussi.maki@isovalent.com>
@joamaki joamaki requested a review from a team as a code owner February 25, 2025 11:36
@joamaki joamaki requested review from dylandreimerink and removed request for a team February 25, 2025 11:36
@joamaki joamaki merged commit 234bb76 into main Feb 25, 2025
@joamaki joamaki deleted the pr/joamaki/retry-fixes branch February 25, 2025 13:41
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.

2 participants