Your message dated Fri, 13 Jan 2023 22:22:50 +0000 with message-id <[email protected]> and subject line Bug#999739: fixed in fai 6.0 has caused the Debian Bug report #999739, regarding fai-setup-storage: BTRFS-RAID on NVMe fails to install to be marked as done.
This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the Bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what this message is talking about, this may indicate a serious mail system misconfiguration somewhere. Please contact [email protected] immediately.) -- 999739: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=999739 Debian Bug Tracking System Contact [email protected] with problems
--- Begin Message ---Package: fai-setup-storage Version: 5.8.4 Severity: normal Tags: patch Dear Maintainer, I observed an issue trying to setup an BTRFS RAID on NVMe disks. I have used FAI version 5.8.4, however, I think that the bug is still present in version 5.10.3. My disk configuration (I deliberately have the 2nd partition unconfigured): disk_config disk1 disklabel:gpt-bios fstabkey:uuid primary - 32G - - primary - -100% - - disk_config disk2 sameas:disk1 disk_config btrfs fstabkey:uuid btrfs raid1 / disk1.1,disk2.1 rw,degraded,noatime,nodiratime,compress=lzo,subvol=@ createopts="-L root" Installing this on a machine with two disks /dev/nvme0n1 and /dev/nvme1n1 terminated with the error: Cannot satisfy pre-depends for mkfs.btrfs -d raid1 -L root -f \ /dev/nvme1n1p1 /dev/nvme0n1p1: pt_complete_/dev/nvmen1p1, -- \ system left untouched. Looking into /usr/share/fai/setup-storage/Commands.pm, sub "build_btrfs_commands", I noticed that line 365 (`$tmp =~ s/\d//;`) apparently tries to strip the partition number off a partition path. This is, unfortunately, too simple for these NVMe disks. I changed the regular expression to only strip the partition no at the end, including any preceding "p": $tmp =~ s/p?\d+$//; At least my system could install successfully with this patch. It would, however, now fail for "/dev/sdpX", so maybe the regex should explicitly handle NVMe disks. Best, Maximilian
--- End Message ---
--- Begin Message ---Source: fai Source-Version: 6.0 Done: Thomas Lange <[email protected]> We believe that the bug you reported is fixed in the latest version of fai, which is due to be installed in the Debian FTP archive. A summary of the changes between this version and the previous one is attached. Thank you for reporting the bug, which will now be closed. If you have further comments please address them to [email protected], and the maintainer will reopen the bug report if appropriate. Debian distribution maintenance software pp. Thomas Lange <[email protected]> (supplier of updated fai package) (This message was generated automatically at their request; if you believe that there is a problem with it please contact the archive administrators by mailing [email protected]) -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 Format: 1.8 Date: Fri, 13 Jan 2023 22:45:03 +0100 Source: fai Architecture: source Version: 6.0 Distribution: unstable Urgency: low Maintainer: Thomas Lange <[email protected]> Changed-By: Thomas Lange <[email protected]> Closes: 844377 951659 964761 977832 985303 987779 989547 990211 990633 999739 1000572 1005745 1006847 1010593 1010906 1011642 1020976 Changes: fai (6.0) unstable; urgency=low . [ Thomas Lange ] * fai-do-scripts: write output of all scripts to scripts.log, remove code for old cfengine versions (1 and 2) * fai-make-nfsroot: drop support for .asc files, use only .gpg * add autopkgtest * install_packages: add release=<name> feature, Closes: #844377, add pacman support add --allow-change-held-packages parse apt-cache output, Closes: #990211 * fetch-basefile: use curl instead of lftp for downloading * Commands.pm: special handling of nvme devices, Closes: #999739, quote variables, Closes: #977832, remove old code need by old version of mkfs.btrfs and lvcreate, * fai-mk-network: use iproute commands instead of brctl * fai-make-nfsroot: add dracut modul network-legacy, Closes: #1010906 do not kill dhclient, Closes: #987779, drop support for dracut < 049, patch udev rule of lvm for bullseye, include empty machine-id into base.tar * Exec.pm: better error message, Closes: #1005745 * fcopy: handle broken symlinks, Closes: #985303 * fai-chboot: add variable substitution when coping a template, add option -a * fai-class: use of .source scripts now emits an error * fai-kvm: add -M for creating NVMe devices * fai-mirror: use only first line of bpo in sources.list, Closes: #1006847, print number of packages inside the mirror * fai-diskimage: call losetup with -P, Closes: #1010593, do not remove VGs, deactivate them, Closes: #990633, add option -c, print error if umount fails, increase number of tries * fai-cd: set TMPDIR for chroot, Closes: #1000572, add grub module regexp * fai-setup: create ECDSA keys * Fstab.pm: support tmpfs size -, this uses the system default size for tmpfs, Closes: #951659 * setup-storage: add support for fstabkey:partlabel and fstabkey:partuuid, remove crypto keyfile after use * enhance disk_config format for partition label support * ftar: add support for zstd * fai: check if unshare can be used, Closes: #989547 use only IPv4 address as hostname remove code for deprecated taskname prepareapt * watch: update to version 4, use new url * subroutines: umount tmpfs if config space was downloaded via http, Closes: #1011642, check if systemd-resolved is used * task_sysinfo: print machine readable partition information, do not call commands that modify the system if no run in FAI environment, print battery information * subroutines: add help functions: ctam (Chroot TArget Mount pseudo filesystems) ctau (Chroot TArget Umount) ctai (Chroot TArget Into) rm_target (remount /target read-write) * NFSROOT: cleanup package list, add cryptsetup, zstd * update my email address * mk-basefile: remove /var/lib/dbus/machine-id, use zstd instead of gzip add Rocky 9, Debian Forky * ROCKY/60-network-scripts: add support for system-connections/${NIC1}.nmconnection * simple/examples: use fai-link and fai-sed instead of ln and sed, remove cryptsetup passphrase from format.log, use class ROCKY instead of CENTOS, update Ubuntu support to 22.04, add CLOUD_EFI, use ext4 for /boot in LVM examples, install kernel packages without recommends, use $kernelname in package_config set by class/DEBIAN.var * 20-hwdetect.sh: do not load kernel modules by default * get-boot-info: set NIC1 even if no network is used during installation * grub.cfg: new code booting from local disk, support more than two disks, add menu for UEFI settings * 15-root-ssh-key: new script that adds ssh keys class based * disk_config/*_EFI: use new format for gpt, p=<partlabel> instead of primary * GRUB_PC/10-setup: add test * remove --pid from unshare call . [ Mike Gabriel ] * fai-abort:give job control to bash, Closes: #1020976 . [ Juri Grabowski ] * install_packages: add support for alpine Linux apk and Archlinux pacman . [ Guido Günther ] * subroutines: allow one to pass a debootstrap script in FAI_DEBOOTSTRAP . [ Christian Meyer ] * fai-setup: add ECDSA-key to known_hosts . [ Aki Tuomi ] * check-cross-arch: fail on errors and unbound variables . [ Hannes Eberhardt ] * package_config/ROCKY: update EFI packages . [ Christoph Pleger ] * Commands.pm: sort btrfs subvolumes, Closes: #964761 do not call mkfs mutliple times Checksums-Sha1: 3a0b76cded94dd8cfe3733a491efb24437f65c8e 2062 fai_6.0.dsc ede9f2ba29f165363c7a97c6e064f3528c544069 294104 fai_6.0.tar.xz 6f2623d54908a5efe3baaa2f488ca47252771083 16177 fai_6.0_amd64.buildinfo Checksums-Sha256: 822719ef8ff85843be942d09d962c2d56930ac1fc157c14c5fec8330e5ebabef 2062 fai_6.0.dsc 5a9a6b7cb6a60fae5ed3d5446c9b759035365340790f96fcb43e9f27391ae0a7 294104 fai_6.0.tar.xz c2015e3619183d99ea715f3f2a6ada24dd943c0173205af09c9f99a607578f58 16177 fai_6.0_amd64.buildinfo Files: fb3ca16e9afa6c89ebe669ae79d729b1 2062 admin optional fai_6.0.dsc 6620846501a0e8218fb57f49814d2c27 294104 admin optional fai_6.0.tar.xz 3f1eb8bd1408a927acfc305a0ba0b22a 16177 admin optional fai_6.0_amd64.buildinfo -----BEGIN PGP SIGNATURE----- iQJFBAEBCAAvFiEEsR7jJz9rLetSjJPaK/jZ/gdLzeQFAmPB0SERHGxhbmdlQGRl Ymlhbi5vcmcACgkQK/jZ/gdLzeTcow/9EmA/eCOTtnoQIQyNI81VtcybeYHQIdyf lPX9gLXmuC9DBtbmo1DG91ruLX88YMe8D3RGdOaTI200/jzetRx33yqkuQ3dRiYL u2c2/Hu5Sqw9W1080Ym62rxDK4WGKV2BPArS27/FmMgx2JohV5ioy3iveVB7L9Nr uvxy2aTDatT9lvwnCcooljlnW6HPXVyW9UHAbCIbQ2tYzT7jtNsEG/QKRdle+P37 PDpZkpQq9labB3GFBOYW3ZB44TLDIonX7mNOXv/mjmKAcIviOC8VQsydAJl4CySG oXXxU5S3Dwtd2j3pja2sl4AVTSE6t15nslEQho13U2Sz9TzDvpLpW57q7epdgHHK FSLgXfjkjmhuTxOFghQSUqgterFGjtCIKNHgt37n6yD0D1o7guj0epL02oLWYpby M0B2zTzGb4Bht8mknJfvNS7MwOHtHQHa1KPuT+gW+pyo5J0jHyQyG4RHvLa+lmXk bs4eU3q34QuVskmwOzsma5+AwVJiUOXYOHaGxlW12828oDR6QaYkAdHVPMiCEu0k UAuqbtHCeT2fnzSaMw/VwZrXYGiGOKc2uHsU2o9jdWcfblbA/HFj3EpOrtfAKxYN RBLTTY6qPyhFVMcWV8YnXSiYq+GGsF5M1TNGeUx3+USs80sRLfnplTgpEnwplWzv Q65Sqzf8Ijo= =1ynP -----END PGP SIGNATURE-----
--- End Message ---
