Your message dated Sun, 06 Nov 2022 00:06:44 +0000 with message-id <[email protected]> and subject line Bug#951659: fixed in fai 6.0~alpha21 has caused the Debian Bug report #951659, regarding setup-storage should allow default tmpfs settings 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.) -- 951659: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=951659 Debian Bug Tracking System Contact [email protected] with problems
--- Begin Message ---Package: fai-setup-storage Version: 5.8.4 Severity: wishlist File: /usr/sbin/setup-storage Tags: upstream The setup-storage(8) command has the following grammar for tmpfs blocks: volume ::= tmpfs <mountpoint> <tmpfs_size> <mount_options> tmpfs_size ::= <sizespec> sizespec ::= RAM:[[:digit:]]+%|[[:digit:]]+[kKMGTP%iB]* This forces a line that looks like the examples in the manpage: tmpfs /tmp RAM:20% defaults tmpfs /scratch 3GiB defaults ... which is great if you want to control that. But in our configuration, we're happy to delegate the decision of how big a tmpfs should be to the kernel, which has its own defaults (50% of RAM at the time of writing, I believe). It seems there's no way to do that delegation in setup-storage(8) right now: if I use `-` as a <sizespec>, I get the following error: # setup-storage -f "disk-config/gnt-fsn" -X Starting setup-storage 2.2 Using config file: disk-config/gnt-fsn ERROR (line 15): invalid tmpfs size near " - defaults disk_config nvme0n1 disklabel:gpt-bios gpt_bios_part:1 bootable:2 [...] ERROR (line 15): Invalid volume: Was expecting tmpfs size but found "- defaults" instead ERROR (line 15): Invalid line: Was expecting '\n' but found "tmpfs /tmp - defaults" instead Syntax error (I'll also note the duplicate errors, with the first strangely containing the entire configuration after the tmpfs keyword, stripped here for brievety.) It seems to me "-" should be allowed here, like this: disk_config tmpfs tmpfs /tmp - defaults It's allowed in fstab and just worked fine in our previous configuration. Thanks! -- System Information: Debian Release: 10.3 APT prefers stable-debug APT policy: (500, 'stable-debug'), (500, 'stable'), (1, 'experimental'), (1, 'unstable') Architecture: amd64 (x86_64) Kernel: Linux 4.19.0-8-amd64 (SMP w/4 CPU cores) Locale: LANG=fr_CA.UTF-8, LC_CTYPE=fr_CA.UTF-8 (charmap=UTF-8), LANGUAGE=fr_CA.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Init: systemd (via /run/systemd/system) LSM: AppArmor: enabled Versions of packages fai-setup-storage depends on: ii e2fsprogs 1.44.5-1+deb10u3 ii liblinux-lvm-perl 0.17-2 ii libparse-recdescent-perl 1.967015+dfsg-2 ii parted 3.2-25 ii perl 5.28.1-6 Versions of packages fai-setup-storage recommends: ii lvm2 2.03.02-3 ii mdadm 4.1-1 Versions of packages fai-setup-storage suggests: ii cryptsetup 2:2.1.0-5+deb10u2 ii dmsetup 2:1.02.155-3 ii dosfstools 4.1-2 pn jfsutils <none> ii ntfs-3g 1:2017.3.23AR.3-3 ii reiserfsprogs 1:3.6.27-3 ii xfsprogs 4.20.0-1 -- no debconf information
--- End Message ---
--- Begin Message ---Source: fai Source-Version: 6.0~alpha21 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, 01 Nov 2022 10:35:51 +0100 Source: fai Architecture: source Version: 6.0~alpha21 Distribution: experimental Urgency: low Maintainer: Thomas Lange <[email protected]> Changed-By: Thomas Lange <[email protected]> Closes: 844377 951659 977832 985303 987779 989547 990211 990633 999739 1000572 1005745 1006847 1010593 1010906 1011642 1020976 Changes: fai (6.0~alpha21) experimental; 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 * 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: usie 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 * 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 * install_packages: parse apt-cache output, Closes: #990211 * subroutines: umount tmpfs if config space was downloaded via http, Closes: #1011642 * 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 * update my email address * mk-basefile: remove /var/lib/dbus/machine-id, use zstd instead of gzip * simple/examples: use fai-link and fai-sed instead of ln and sed, remove cryptsetup passphrase from logs, use class ROCKY instead of CENTOS, update Ubuntu support to 22.04, add CLOUD_EFI, use ext4 for /boot in LVM examples * get-boot-info: set NIC1 even if no network is used during installation . [ 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 Checksums-Sha1: 674897c217fddb8ecb8d60988b11a6d5003291bf 2094 fai_6.0~alpha21.dsc dadbce601b35d10861555016eafd1f2dc3ad57f6 292776 fai_6.0~alpha21.tar.xz 52ce7e0ca0b47abff0eab60f3badcd3370e74525 16335 fai_6.0~alpha21_amd64.buildinfo Checksums-Sha256: 4738945c7c375c4682740044b614ccbd830e3a93ba0523390e8397c404b1493b 2094 fai_6.0~alpha21.dsc 50b7f48d475beab3f951ca69a66c5a6c9dba51b33ee0449dd522c2c75a349d0c 292776 fai_6.0~alpha21.tar.xz 47db447a69cd91715b73597518f08fbd0a058dbd10b56588755130c3ff76e7e1 16335 fai_6.0~alpha21_amd64.buildinfo Files: 5308cd37843e9d773746bec06312fd05 2094 admin optional fai_6.0~alpha21.dsc 6979444dd9ede59d41a4b4a6f49ff480 292776 admin optional fai_6.0~alpha21.tar.xz 7e616c1142ea7e1d70fbdc57d4c09db5 16335 admin optional fai_6.0~alpha21_amd64.buildinfo -----BEGIN PGP SIGNATURE----- iQJFBAEBCAAvFiEEsR7jJz9rLetSjJPaK/jZ/gdLzeQFAmNm960RHGxhbmdlQGRl Ymlhbi5vcmcACgkQK/jZ/gdLzeQX6w//bsSF9QAzQhG95CVymWCy1f9yiwekVRUz V8XvlaGYcQO4/WmebW9tKRg7O2JgZEQ5nBobbl5NQDnQR5a4Et0OAG6WRHXGZzKk T1/UXEuY+VsVUwZuLqO9vG3vPfPEGjjjpN/ocAfBKasRDpP6cqmdCUdgXSgT/Wts sSYDvuFEjZlhu11bPU2SlCNCm2OS0GYm8RzbjDCXAzSz0Nsg+dVxYZnh/4/HMW6E 74pmoAoUAqMTgveNWlDr4g3BviysIErA3fR9HAvXlBhOd1mU9Ahfjz0rCZNzsjZ1 9TP5rY/Ev605D6Ew6kxv3ENi7mNIFMxJ9Y8611ALhzBXbj6mv1c788O7sGTQ1+cj Joo81fM4of1iVhv3kviDvYRK3ALPxY+Mz/E3pNL6xX/jb24Pc4gWF2XXufgn0WSh fDgGmwtW2hkDRLO5WLobUnTbACKuUj9pHPre/MMF647rl6+fu0//0Ad1+Uw8tvZ2 BL9KTnjL7x9P9gI83v/Nc/eoz60yhYlk0e89NKeZV7A3WTf7QjwomccuW5253a0i W7YFwWoITwy1077BY6xjcqUT7YYjWArg78XaD19R1bVnxurepxxVt4kUwM91pBpu qkV5ppxDzolFQzDD1uIn4keza3OCLIspdgatT5eJaiwvicwJ3q9MzvxdZI+dNB0O 5sE36Y+SMPc= =K803 -----END PGP SIGNATURE-----
--- End Message ---
