GC roots cannot have a physical target #1036
Labels
No labels
Affects/CppNix
Affects/Nightly
Affects/Only nightly
Affects/Stable
Area/build-packaging
Area/cli
Area/evaluator
Area/fetching
Area/flakes
Area/language
Area/lix ci
Area/nix-eval-jobs
Area/profiles
Area/protocol
Area/releng
Area/remote-builds
Area/repl
Area/repl/debugger
Area/store
awaiting
author
awaiting
contributors
bug
Context
contributors
Context
drive-by
Context
maintainers
Context
RFD
crash 💥
Cross Compilation
devx
docs
Downstream Dependents
E/easy
E/hard
E/help wanted
E/reproducible
E/requires rearchitecture
Feature/S3
imported
Language/Bash
Language/C++
Language/NixLang
Language/Python
Language/Rust
Needs Langver
OS/Linux
OS/macOS
performance
regression
release-blocker
stability
Status
blocked
Status
invalid
Status
postponed
Status
wontfix
testing
testing/flakey
Topic/Large Scale Installations
ux
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
lix-project/lix#1036
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Describe the bug
If you are using chroot stores for various separation objectives, you will run into the problem of managing the GC units for it.
Part of the job is also managing the direct roots for it.
A natural way to express GC roots is to simply symlink in the physical world and let Lix rebase the targets appropriately.
In practice, this doesn't work because GC root evaluation doesn't take place with physical vs. logical awareness, but always logical awareness.
As a consequence, it is necessary to have the symlink targets ALWAYS be logical ones, creating a skewed view outside the namespaces.
Steps To Reproduce
/mnt(i.e./mnt/nix/{store,var})/mnt/nix/var/nix/gcroots/some-path→/mnt/nix/store/....)nix-collect-garbage --store /mntsome-pathbeing GCed.Expected behavior
some-pathshould not be GCed.nix --versionoutputLix 2.93.3.
Additional context
It's pretty obvious by reading the code in
gc.cthatisInStoreusesconfig().storeDirwhich is logical awareness.Question is what should be the fix? Should the GC code perform physical awareness or should
isInStoreperform physical awareness? Not obvious.