The source package is zfs-linux, not zfsutils-linux. I had this wrong
in the fix for #314.
Closes#318
Reported-by: Piscium
Signed-off-by: Richard Laager <rlaager@wiktel.com>
We stick to two rules here:
* -o options come before -O
* pool-specific options are wedged in between -o and
-O (e.g. compatibility=grub2)
I don't know if that was a deliberate policy, I'm just trying to guess
what the pattern was
Signed-off-by: Antoine Beaupré <anarcat@debian.org>
Signed-off-by: Richard Laager <rlaager@wiktel.com>
This regroups the options passed to zpool create in a more logical
way.
1. the ACL/xattr/dnodesize settings are all on one line, since they
are all related (latter being optimizations because we enable the
former)
2. ashift, autotrim, compression, normalization, relatime are all
independent options that merit their own line for better
visibility, they are also optional and therefore it makes sense to
have them separate
3. canmount, mountpoint and -R are all related as well
This is only a "whitespace" fix: there should be no functional change
to this patch.
I also noted that dnodesize=auto is *not* passed to the bpool cache
creation, even though it's passed to the rpool creation. I suspect
that might be an omission. This also goes to show that ordering those
options consistently makes it easier to spot those errors.
This is a followup to #307.
Signed-off-by: Antoine Beaupré <anarcat@debian.org>
Signed-off-by: Richard Laager <rlaager@wiktel.com>
According to the apt_preferences man page, we should be able to use
"Package: src:zfsutils-linux". Then we don't have to worry about keeping
the list of packages up-to-date, especially as library soversions change.
Reported-by: udf2457
Closes#314
Signed-off-by: Richard Laager <rlaager@wiktel.com>
This doesn't work on Debian Bullseye. While I tested the actual
create command on unstable, I know I looked at a file list on
packages.debian.org. I must have looked at bullseye-backports by
mistake. That's what I get for not testing the whole thing!
This partially reverts commit 7343c2253f.
Reported-by: anarcat
Closes#309
Signed-off-by: Richard Laager <rlaager@wiktel.com>
The major change for Ubuntu is to mark many of these as optional, like
with Debian and the old guides. As I move away from zsys, this will end
up being more like the old way.
This was also discussed in #306 and #307, but this change has
trade-offs. It can reduce the number of datasets created on the
system, but it does so by increasing the complexity to read and follow
the guide.
Then I just harmonized Debian with Ubuntu. Aside from whitespace,
reordering, rewording, the substantive changes were to drop /opt and
add /var/lib/NetworkManager.
Signed-off-by: Richard Laager <rlaager@wiktel.com>
Running "zfs set canmount=off rpool/ROOT/debian" causes the root
filesystem to be remounted read-only. I have no idea why, but that's
mostly beside the point.
I have no idea why I added these commands in the first place. The
other guides were not this way. These steps showed up in the initial
add of the Bullseye instructions.
Reported-by: Cameron Skappel <cskappel@wiktel.com>
Signed-off-by: Richard Laager <rlaager@wiktel.com>
This switch makes tasksel include the "standard system utilities" task.
Unfortunately, it also defaults to installing a desktop environment, so
we need to warn users about that.
Co-authored-by: Marcello Mamino <m2@ciovil.li>
Signed-off-by: Richard Laager <rlaager@wiktel.com>
Credit: @mdiepart from #282Closes#282#262
Fix variable export before chroot
Credit: @mdiepart from #282
remind user to enable multi-disk support
NixOS: handle installation failure wrt encrypted boot pool
Closes#279
Signed-off-by: Maurice Zhou <jasper@apvc.uk>
The patch has been brought into Ubuntu 20.04.3 and is no longer needed.
I (rlaager) missed making this change to the 22.04 instructions too.
Signed-off-by: Richard Laager <rlaager@wiktel.com>
