Skip to content

ci: retry the modflow executable download - #117

Merged
jdhughes-dev merged 1 commit into
MODFLOW-ORG:developfrom
jdhughes-dev:ci-retry-modflow-download
Sep 6, 2026
Merged

ci: retry the modflow executable download#117
jdhughes-dev merged 1 commit into
MODFLOW-ORG:developfrom
jdhughes-dev:ci-retry-modflow-download

Conversation

@jdhughes-dev

Copy link
Copy Markdown
Contributor

The executables are fetched from github at the start of every test job. One job of thirty failed on develop when that connection was reset with URLError: [Errno 104] Connection reset by peer, before any test ran. The download is attempted twice before the job fails.

Passing github_token would not have helped: the action already falls back to github.token when none is given, so the request was authenticated.

The step in the release workflow is already conditional, so its retry combines that condition with the outcome of the first attempt.

The executables are fetched from github at the start of every test job.
One job of thirty failed on develop when that connection was reset, before
any test ran. The download is attempted twice before the job fails.

The step in the release workflow is already conditional, so its retry
combines that condition with the outcome of the first attempt rather than
carrying a second if key.
@jdhughes-dev
jdhughes-dev merged commit 8aebb9b into MODFLOW-ORG:develop Sep 6, 2026
35 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.

1 participant