beatzaplenty/nixos
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Unmount everything under /mnt before zpool export, not just chroot dirs #43

Merged
beatzaplenty merged 1 commits from worktree-baremetal-esp-unmount into main 2026-07-22 04:30:06 +00:00
Conversation 0 Commits 1 Files Changed 1
+16 -13
1 changed files with 16 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
scripts/installer/auto-install.sh
+15 -12
View File
@@ -180,19 +180,22 @@ nixos-install \
# if the chosen host has no ZFS root, e.g. proxmox-*/linode-*) clears # if the chosen host has no ZFS root, e.g. proxmox-*/linode-*) clears
# that in-use state so the next import, from any hostid, succeeds. # that in-use state so the next import, from any hostid, succeeds.
# #
# nixos-install bind-mounts /dev, /proc, /sys (and usually /run) into # Anything still mounted under /mnt -- nixos-install's own leftover
# /mnt to run the target's activation script (switch-to-configuration # chroot bind mounts for running the target's activation script
# boot) in a chroot, and doesn't unmount them again once it's done. # (/mnt/dev, /mnt/proc, /mnt/sys, /mnt/run), and disko's own /mnt/boot
# Left in place, those nested mounts make ZFS refuse to unmount its own # ESP mount (modules/disko/baremetal.nix) -- blocks ZFS from unmounting
# root dataset at /mnt -- confirmed live: zpool export failed with # its root dataset at /mnt, the same way any nested mount blocks
# "cannot unmount '/mnt': pool or dataset busy", and because of this # unmounting its parent. Confirmed live: zpool export failed with
# script's `set -e`, that killed the script before it ever reached # "cannot unmount '/mnt': pool or dataset busy" even after handling the
# reboot, silently defeating the whole point of exporting first. # chroot mounts alone, because /mnt/boot was still mounted too. Because
for chroot_mount in dev proc sys run; do # of this script's `set -e`, that killed the script before it ever
if mountpoint -q "/mnt/${chroot_mount}"; then # reached reboot, silently defeating the whole point of exporting first.
umount -R "/mnt/${chroot_mount}" # Unmounting everything under /mnt up front (recursively, so nested
# mounts like /mnt/dev/pts come along for free) sidesteps needing to
# enumerate every mount disko/nixos-install might leave behind.
if mountpoint -q /mnt; then
umount -R /mnt
fi fi
done
if [[ -n "$(zpool list -H -o name 2>/dev/null)" ]]; then if [[ -n "$(zpool list -H -o name 2>/dev/null)" ]]; then
echo "Exporting ZFS pool(s) before reboot..." echo "Exporting ZFS pool(s) before reboot..."