summary refs log tree commit diff
path: root/pkgs/os-specific/linux/zfs/git.nix
diff options
context:
space:
mode:
authorRicardo M. Correia <rcorreia@wizy.org>2014-10-22 19:17:21 +0200
committerRicardo M. Correia <rcorreia@wizy.org>2014-11-12 22:31:49 +0100
commit12e77fdc3f6f223be1a4d5c88d6c79bff63ae70c (patch)
tree532a624018d86ea3b26277d12629c430204a8cd5 /pkgs/os-specific/linux/zfs/git.nix
parent1fea5866aea0aeed2c22dcee20e3fd3f088de3a5 (diff)
downloadnixlib-12e77fdc3f6f223be1a4d5c88d6c79bff63ae70c.tar
nixlib-12e77fdc3f6f223be1a4d5c88d6c79bff63ae70c.tar.gz
nixlib-12e77fdc3f6f223be1a4d5c88d6c79bff63ae70c.tar.bz2
nixlib-12e77fdc3f6f223be1a4d5c88d6c79bff63ae70c.tar.lz
nixlib-12e77fdc3f6f223be1a4d5c88d6c79bff63ae70c.tar.xz
nixlib-12e77fdc3f6f223be1a4d5c88d6c79bff63ae70c.tar.zst
nixlib-12e77fdc3f6f223be1a4d5c88d6c79bff63ae70c.zip
nixos/zfs: Improve the ZFS boot process
It turns out that the upstream systemd services that import ZFS pools contain
serious bugs. The first major problem is that importing pools fails if there
are no pools to import. The second major problem is that if a pool ends up in
/etc/zfs/zpool.cache but it disappears from the system (e.g. if you
reboot but during the reboot you unplug your ZFS-formatted USB pen drive),
then the import service will always fail and it will be impossible to get rid
of the pool from the cache (unless you manually delete the cache).

Also, the upstream service would always import all available ZFS pools every
boot, which may not be what is desired in some cases.

This commit will solve these problems in the following ways:

1. Ignore /etc/zfs/zpool.cache. This seems to be a major source of
issues, and also does not play well with NixOS's philosophy of
reproducible configurations. Instead, on every boot NixOS will try to import
the set of pools that are specified in its configuration.  This is also the
direction that upstream is moving towards.

2. Instead of trying to import all ZFS pools, only import those that are
actually necessary. NixOS will automatically determine these from the
config.fileSystems.* option. Also, the user can import any additional
pools every boot by adding them to the config.boot.zfs.extraPools
option, but this is only necessary if their filesystems are not
specified in config.fileSystems.*.

3. Added options to configure if ZFS should force-import ZFS pools. This may
currently be necessary, especially if your pools have not been correctly
imported with a proper host id configuration (which is probably true for 99% of
current NixOS ZFS users). Once host id configuration becomes mandatory when
using ZFS in NixOS and we are sure that most users have updated their
configurations and rebooted at least once, we should disable force-import by
default. Probably, this shouldn't be done before the next stable release.

WARNING: This commit may change the order in which your non-ZFS vs ZFS
filesystems are mounted.  To avoid this problem (now or in the future)
it is recommended that you set the 'mountpoint' property of your ZFS
filesystems to 'legacy', and that you manage them using
config.fileSystems, just like any other non-ZFS filesystem is usually
managed in NixOS.
Diffstat (limited to 'pkgs/os-specific/linux/zfs/git.nix')
-rw-r--r--pkgs/os-specific/linux/zfs/git.nix11
1 files changed, 11 insertions, 0 deletions
diff --git a/pkgs/os-specific/linux/zfs/git.nix b/pkgs/os-specific/linux/zfs/git.nix
index 01e72d0af6db..e5002bbbb3f3 100644
--- a/pkgs/os-specific/linux/zfs/git.nix
+++ b/pkgs/os-specific/linux/zfs/git.nix
@@ -47,6 +47,17 @@ stdenv.mkDerivation {
 
   enableParallelBuilding = true;
 
+  # Remove provided services as they are buggy
+  postInstall = ''
+    rm $out/etc/systemd/system/zfs-import-*.service
+
+    sed -i '/zfs-import-scan.service/d' $out/etc/systemd/system/*
+
+    for i in $out/etc/systemd/system/*; do
+      substituteInPlace $i --replace "zfs-import-cache.service" "zfs-import.target"
+    done
+  '';
+
   meta = {
     description = "ZFS Filesystem Linux Kernel module";
     longDescription = ''