Skip to content

util: retry GNU tar setup downloads#287

Open
kevinburke wants to merge 1 commit into
uutils:mainfrom
kevinburke:gnu-clone-retry
Open

util: retry GNU tar setup downloads#287
kevinburke wants to merge 1 commit into
uutils:mainfrom
kevinburke:gnu-clone-retry

Conversation

@kevinburke
Copy link
Copy Markdown

The GNU tar CI job depends on git.savannah.gnu.org, which can fail transiently during clone or submodule setup. Retry those network-dependent setup steps with exponential backoff before failing the job.

Keep cleanup scoped to failed clone attempts, fail early if path_GNU already points at a non-git directory, and tighten the helper script with pipefail and shellcheck-clean quoting.

The GNU tar CI job depends on git.savannah.gnu.org, which can fail
transiently during clone or submodule setup. Retry those network-dependent
setup steps with exponential backoff before failing the job.

Keep cleanup scoped to failed clone attempts, fail early if path_GNU already
points at a non-git directory, and tighten the helper script with pipefail and
shellcheck-clean quoting.
@kevinburke kevinburke requested a review from kaladron as a code owner May 18, 2026 15:50
@codecov
Copy link
Copy Markdown

codecov Bot commented May 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.59%. Comparing base (fc74817) to head (b4865da).
⚠️ Report is 39 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #287   +/-   ##
=======================================
  Coverage   96.59%   96.59%           
=======================================
  Files           9        9           
  Lines         941      941           
  Branches       24       24           
=======================================
  Hits          909      909           
  Misses         31       31           
  Partials        1        1           
Flag Coverage Δ
macos_latest 96.59% <ø> (ø)
ubuntu_latest 96.59% <ø> (ø)
windows_latest 0.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kaladron
Copy link
Copy Markdown
Collaborator

I'm looking at this, and just trying to figure out if there's a way to just cache the tarball or something.

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