Skip to content

Make prek cache restoration resilient to corrupted tarballs#63684

Merged
potiuk merged 1 commit intoapache:mainfrom
potiuk:fix-prek-cache-resilience
Mar 15, 2026
Merged

Make prek cache restoration resilient to corrupted tarballs#63684
potiuk merged 1 commit intoapache:mainfrom
potiuk:fix-prek-cache-resilience

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Mar 15, 2026

Previously, a missing or corrupted cache tarball would hard-fail the
entire static checks job even when stash-hit reported true. This
merges the check/extract/show steps into a single resilient step that
gracefully falls through to a clean prek install-hooks on failure.


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

Generated-by: Claude Opus 4.6 following the guidelines

Previously, a missing or corrupted cache tarball would hard-fail the
entire static checks job even when stash-hit reported true. Now the
extraction is wrapped in error handling: if the tarball is missing or
tar fails, it gracefully falls through to a clean prek install-hooks
instead of aborting.
@boring-cyborg boring-cyborg bot added area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Mar 15, 2026
@potiuk potiuk merged commit 20486bb into apache:main Mar 15, 2026
64 checks passed
@potiuk potiuk deleted the fix-prek-cache-resilience branch March 15, 2026 23:14
github-actions bot pushed a commit that referenced this pull request Mar 15, 2026
…ls (#63684)

Previously, a missing or corrupted cache tarball would hard-fail the
entire static checks job even when stash-hit reported true. Now the
extraction is wrapped in error handling: if the tarball is missing or
tar fails, it gracefully falls through to a clean prek install-hooks
instead of aborting.
(cherry picked from commit 20486bb)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
@github-actions
Copy link

Backport successfully created: v3-1-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
v3-1-test PR Link

github-actions bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Mar 15, 2026
…ls (apache#63684)

Previously, a missing or corrupted cache tarball would hard-fail the
entire static checks job even when stash-hit reported true. Now the
extraction is wrapped in error handling: if the tarball is missing or
tar fails, it gracefully falls through to a clean prek install-hooks
instead of aborting.
(cherry picked from commit 20486bb)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
potiuk added a commit that referenced this pull request Mar 16, 2026
…ls (#63684) (#63685)

Previously, a missing or corrupted cache tarball would hard-fail the
entire static checks job even when stash-hit reported true. Now the
extraction is wrapped in error handling: if the tarball is missing or
tar fails, it gracefully falls through to a clean prek install-hooks
instead of aborting.
(cherry picked from commit 20486bb)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
fat-catTW pushed a commit to fat-catTW/airflow that referenced this pull request Mar 22, 2026
…3684)

Previously, a missing or corrupted cache tarball would hard-fail the
entire static checks job even when stash-hit reported true. Now the
extraction is wrapped in error handling: if the tarball is missing or
tar fails, it gracefully falls through to a clean prek install-hooks
instead of aborting.
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-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants