Skip to content

libfetchers: Bump tarball-cache version to v2#14799

Merged
Mic92 merged 1 commit intomasterfrom
tarball-cache-v2
Dec 15, 2025
Merged

libfetchers: Bump tarball-cache version to v2#14799
Mic92 merged 1 commit intomasterfrom
tarball-cache-v2

Conversation

@xokdvium
Copy link
Contributor

@xokdvium xokdvium commented Dec 15, 2025

Motivation

Unfortunately previous tarball caches had loose objects written to them and subsequent switch to thin packfiles. This results in possibly broken thin packfiles when the loose objects backend is disabled. Thin packfiles do not necessarily contain the whole closure of objects. When packfilesOnly is true we end up with an inconsistent state where a tree lives in a packfiles which refers to a blob in the loose objects backend.

In the future we might want to nuke old cache directories and repack the tarball cache.

Context

Fixes #14796.


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@xokdvium xokdvium requested a review from edolstra as a code owner December 15, 2025 19:10
@xokdvium xokdvium added the backport 2.33-maintenance Automatically creates a PR against the branch label Dec 15, 2025
@github-actions github-actions bot added the fetching Networking with the outside (non-Nix) world, input locking label Dec 15, 2025
Unfortunately previous tarball caches had loose objects written to
them and subsequent switch to thin packfiles. This results in possibly
broken thin packfiles when the loose objects backend is disabled. Thin
packfiles do not necessarily contain the whole closure of objects.
When packfilesOnly is true we end up with an inconsistent state where
a tree lives in a packfiles which refers to a blob in the loose objects
backend.

In the future we might want to nuke old cache directories and repack
the tarball cache.
@Mic92 Mic92 enabled auto-merge December 15, 2025 19:20
@Mic92 Mic92 added this pull request to the merge queue Dec 15, 2025
Merged via the queue into master with commit 49f666c Dec 15, 2025
21 checks passed
@Mic92 Mic92 deleted the tarball-cache-v2 branch December 15, 2025 21:59
@internal-nix-ci
Copy link

Backport failed for 2.33-maintenance, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin 2.33-maintenance
git worktree add -d .worktree/backport-14799-to-2.33-maintenance origin/2.33-maintenance
cd .worktree/backport-14799-to-2.33-maintenance
git switch --create backport-14799-to-2.33-maintenance
git cherry-pick -x 0ffe83aa14cbc6d663fb8e12ca2950581101f8ee

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 2.33-maintenance Automatically creates a PR against the branch fetching Networking with the outside (non-Nix) world, input locking

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot fetch tarball-based flake inputs as root

2 participants