Skip to content

feature(ci): adding in retries - #6006

Draft
cmgoffena13 wants to merge 1 commit into
SQLMesh:mainfrom
cmgoffena13:feature/ci-retries
Draft

feature(ci): adding in retries#6006
cmgoffena13 wants to merge 1 commit into
SQLMesh:mainfrom
cmgoffena13:feature/ci-retries

Conversation

@cmgoffena13

Copy link
Copy Markdown
Collaborator

Description

Adding in loop retries on all install commands to handle transient issues.

Test Plan

Testing CI Stages

Checklist

  • I have run make style and fixed any issues
  • I have added tests for my changes (if applicable)
  • All existing tests pass (make fast-test)
  • My commits are signed off (git commit -s) per the DCO

Signed-off-by: Cortland Goffena <30168413+cmgoffena13@users.noreply.github.com>
@cmgoffena13 cmgoffena13 self-assigned this Aug 31, 2026
@mday-io
mday-io self-requested a review September 1, 2026 14:35
@mday-io

mday-io commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

I feel that this should be configurable

@cmgoffena13
cmgoffena13 marked this pull request as draft September 1, 2026 15:37
@cmgoffena13

Copy link
Copy Markdown
Collaborator Author

@mday-io dude that's what I thought, apparently retries on actions are not builtin on Github -- I'm slow to merge this one, but its better than using Custom actions that could become security vulnerabilities imo

@mday-io

mday-io commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

why don't you create something like .github/actions/retry/action.yml: which can be referenced by other steps and configured in the repo?

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