Skip to content

Bump stash/restore to retry-capable revision with fail-on-download#65268

Merged
potiuk merged 2 commits intoapache:mainfrom
potiuk:bump-stash-restore-fail-on-download
Apr 18, 2026
Merged

Bump stash/restore to retry-capable revision with fail-on-download#65268
potiuk merged 2 commits intoapache:mainfrom
potiuk:bump-stash-restore-fail-on-download

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Apr 14, 2026

Bumps apache/infrastructure-actions/stash/restore and stash/save to
the revision from infrastructure-actions#716,
which adds retry-count (default 3, retries transient gh run download
failures) and fail-on-download inputs to the restore action.

Sets fail-on-download: 'true' on the two restore sites where the
following step (breeze <image>-image load) cannot continue without the
downloaded artifact:

  • .github/actions/prepare_breeze_and_image/action.yml
  • .github/actions/prepare_single_ci_image/action.yml

All other call sites are true caches (cache-mount, docs inventory,
node_modules, prek cache) where a failed download only means a slower,
cold run — they keep the default (fail-on-download: false) so a
transient artifact fetch issue won't take the whole job down.

Note: PR apache/infrastructure-actions#716 is not merged yet, so the SHA pin (d79f0a5b...) points
at the current head of its branch and will need to be rebumped once the
PR lands on apache/infrastructure-actions main.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Opus 4.6

Generated-by: Claude Opus 4.6 following the guidelines

@boring-cyborg boring-cyborg Bot added area:dev-tools backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch labels Apr 14, 2026
@potiuk potiuk marked this pull request as ready for review April 14, 2026 23:05
Comment thread .github/actions/install-prek/action.yml Outdated
potiuk added 2 commits April 18, 2026 10:15
Bumps stash/restore and stash/save to the PR apache#716 revision which adds
retry-count (default 3) and fail-on-download inputs to stash/restore.

Sets fail-on-download: true on the two restore sites where the next
step (breeze ci-image load) cannot continue without the artifact:
prepare_breeze_and_image and prepare_single_ci_image. Other call sites
are true caches (cache-mount, docs inventory, node_modules, prek cache)
where a missed download only means a slower/cold run, so they keep the
default behaviour.
PR apache/infrastructure-actions#716 (retry-count and fail-on-download
inputs on stash/restore) is now merged. Bump the pinned SHA from the
old PR branch head to the current main tip. No behaviour change from
the previous commit — just keeps the pinned SHA in sync with upstream.
@potiuk potiuk force-pushed the bump-stash-restore-fail-on-download branch from b50ca51 to d8394be Compare April 18, 2026 08:21
@potiuk potiuk merged commit e9ccf07 into apache:main Apr 18, 2026
141 checks passed
@potiuk potiuk deleted the bump-stash-restore-fail-on-download branch April 18, 2026 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants