nixos/module/deluge: fix directory ownership. #67852
Conversation
Fixes nixos/tests/deluge.nix test. The pre-script was failing with 'Permission denied' errors. I have tracked it down to the unusual permissions on /var/lib/deluge/.config directory tree. Additionally errors of form "Detected unsafe path transition xxx → yyy during canonicalization of zzz" were printed by systemd. This makes it work.
|
@GrahamcOfBorg test deluge |
There was a problem hiding this comment.
The change to use tmpfiles.d to manage the directory layout declaratively was intentional.
The error message looks more as if there are some symlinks involved / cf https://github.com/systemd/systemd/blob/master/src/tmpfiles/tmpfiles.c#L883
Can you elaborate on how your filesystem layout looks there?
|
also cc @aanderse |
|
@gleber please paste your |
|
@flokli Yes, I suspected that :) The issue is that it does not work for me and the appropriate test in the repo fails on master. I am happy to get it fixed with using @aanderse It is a bit messy, hard to get to get it shared. But the issue is reproducible with I am happy to get hints how to fix it with |
|
Closing in favor of #67888 |
Fixes nixos/tests/deluge.nix test.
The pre-script was failing with 'Permission denied' errors. I have tracked it
down to the unusual permissions on /var/lib/deluge/.config directory tree.
Additionally errors of form "Detected unsafe path transition xxx → yyy during
canonicalization of zzz" were printed by systemd. This makes it work.
Motivation for this change
Things done
sandboxinnix.confon non-NixOS)nix-shell -p nix-review --run "nix-review wip"./result/bin/)nix path-info -Sbefore and after)Notify maintainers
cc @infinisil @domenkozar @ebzzry @davidtwco