mirror of
https://github.com/NixOS/nix.git
synced 2025-11-16 23:42:43 +01:00
f68cdf1801
First the motivation: I recently faced a bug that I assume is coming
from the topoSortPaths function where the GC was trying to delete a
path having some alive referrers. I resolved this by manually deleting
the faulty path referrers using nix-store --query --referrers. I sadly
did not manage to reproduce this bug.
This bug alone is not a big deal. However, this bug is
triggering a cascading failure: invalidatePathChecked is throwing a
PathInUse exception. This exception is not catched and fails the whole GC
run. From there, the machine (a builder machine) was unable to GC its
Nix store, which led to an almost full disk with no way to
automatically delete the dead Nix paths.
Instead, I think we should log the error for the specific store path
we're trying to delete, specifying we can't delete this path because
it still has referrers. Once we're done with logging that, the GC run
should continue to delete the dead store paths it can delete.
(cherry picked from commit
|
||
|---|---|---|
| .. | ||
| build-remote | ||
| external-api-docs | ||
| internal-api-docs | ||
| libcmd | ||
| libexpr | ||
| libexpr-c | ||
| libfetchers | ||
| libflake | ||
| libmain | ||
| libmain-c | ||
| libstore | ||
| libstore-c | ||
| libutil | ||
| libutil-c | ||
| libutil-tests | ||
| nix | ||
| nix-build | ||
| nix-channel | ||
| nix-collect-garbage | ||
| nix-copy-closure | ||
| nix-env | ||
| nix-instantiate | ||
| nix-store | ||
| perl | ||
| nix-expr-test-support | ||
| nix-expr-tests | ||
| nix-fetchers-tests | ||
| nix-flake-tests | ||
| nix-store-test-support | ||
| nix-store-tests | ||
| nix-util-test-support | ||
| nix-util-tests | ||