Your message dated Sat, 24 Sep 2022 11:24:00 +0000 with message-id <[email protected]> and subject line Bug#999739: fixed in fai 6.0~alpha11 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~alpha11 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: Tue, 20 Sep 2022 12:02:28 +0200 Source: fai Architecture: source Version: 6.0~alpha11 Distribution: experimental Urgency: low Maintainer: Thomas Lange <[email protected]> Changed-By: Thomas Lange <[email protected]> Closes: 844377 951659 977832 985303 987779 989547 990633 999739 1000572 1005745 1006847 1010593 1010906 Changes: fai (6.0~alpha11) experimental; urgency=low . * add autopkgtest * install_packages: add support for alpnie Linux apk, add release=<name> feature, Closes: #844377, add pacman support * fetch-basefile: cuse curl instead of lftp, so https is now working * Commands.pm: special handling of nvme devices, Closes: #999739, quote variables, Closes: #977832 * fai-mk-network: usie iproute commands instead of brctl * fai-make-nfsroot: add dracut modul network-legacy, Closes: #1010906 * 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-kvm: add -M for creating NVMe devices * fai-mirror: use only first line of bpo in sources.list, Closes: #1006847 * fai-diskimage: call losetup with -P, Closes: #1010593, do not remove VGs, deactivate them, Closes: #990633, add option -c * fai-cd: set TMPDIR for chroot, Closes: #1000572 * 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 * enhance disk_config format for partition label support * ftar: add support for zstd * fai: check if unshare can be used, Closes: #989547 * subroutines: start dhclient, Closes: #987779 * watch: update to version 4 Checksums-Sha1: 6fb37bb4d655cb77c0c2cd3bc0c89e05eb43d5a4 2094 fai_6.0~alpha11.dsc bc09b228c7edeaca2416150be9f9f5ce898ff5ee 291044 fai_6.0~alpha11.tar.xz 3d24abd05c52558a250c286987af0214fc643660 16303 fai_6.0~alpha11_amd64.buildinfo Checksums-Sha256: ff09d7e50f343a284d93a43c5fe0d60965192d1215602342a4155190d03f9723 2094 fai_6.0~alpha11.dsc fd7891c2860564e90629f4cab285f03b00ac04555b85b91f157baab892c61cf9 291044 fai_6.0~alpha11.tar.xz 6ee3c13cbf3c8aa880f31730990673ba6ead48da53d5648ca530ca4f93b8f6c8 16303 fai_6.0~alpha11_amd64.buildinfo Files: 377b6182d96e75f59ae5ed4a52213dc4 2094 admin optional fai_6.0~alpha11.dsc 264a6b970868a496bbf22ff6452f0c36 291044 admin optional fai_6.0~alpha11.tar.xz a11e0435c2acd6dd8d70830a7d08477f 16303 admin optional fai_6.0~alpha11_amd64.buildinfo -----BEGIN PGP SIGNATURE----- iQJFBAEBCAAvFiEEsR7jJz9rLetSjJPaK/jZ/gdLzeQFAmMu4CIRHGxhbmdlQGRl Ymlhbi5vcmcACgkQK/jZ/gdLzeQmmBAAlLyespLo0Q4CiOgUYywU8j1iSaOouQmX QPHQR67AAfHcHOd9p/1NeR/Xklni/YQ2eoqQqJEiFyiaKgD/anfq5lt7DopsZghl vQntPaBZ3LT4EKJT7AyrqKBfE2wu2fWfEy07KNeQUekmEHNoD7+SrgFWVy6uTUIZ +QxhSniyN1OeT8es+usEhObNSegM128ao1DYFLoP9AhVJmAEY5aphdee33B7RPUO VVU8HOn5zhSH3LxqsWJRfHAMUfJG8A+RbGGxLAL+kO36Ubrk0x17l7Qri1vpobp/ q5GsBoKXwnXavPjdsB9E5s4I9vUlIOx7LFRepA0ztgTkODiBlhZKKrC5qfEyKvtD 58TEiGceS1NalbuLD3O0Lc8cIrprBwzeoRlLfkkK7pjtV+nOAeowMKqAOwmgm5Cb 4ZCy/BT5UQKyjf/BJlteWiy30E0QUwiM4tN3XEXT4Ox6OyFZ1FPrO3J5JSgJrX3j OpXigCYFO1vIPK9nknUb/xEcmLduBtf6hFTqCaOwWJlziyAQDAE7eO6demjOEu7u APdRZDf5WqKe1mfQ1SaYYfGCmq8FmVYKF1zDfqjk1JoAm6HZo+ZP7QBuZ2IZXWWk 1K629o/8uZf4/Pj8Hl25w4j92OyWfSEs8ZLBrMzfeZ1ES2Agd2ZECexLQ2l1Yopu IfctnZ5nywU= =12ua -----END PGP SIGNATURE-----
--- End Message ---
