Your message dated Mon, 07 Nov 2022 00:20:14 +0000
with message-id <e1orps6-003noq...@fasolo.debian.org>
and subject line Bug#951659: fixed in fai 6.0~alpha24
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 ow...@bugs.debian.org
immediately.)
--
951659: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=951659
Debian Bug Tracking System
Contact ow...@bugs.debian.org 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~alpha24
Done: Thomas Lange <la...@debian.org>
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 951...@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.
Debian distribution maintenance software
pp.
Thomas Lange <la...@debian.org> (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 ftpmas...@ftp-master.debian.org)
-----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~alpha24
Distribution: experimental
Urgency: low
Maintainer: Thomas Lange <la...@debian.org>
Changed-By: Thomas Lange <la...@debian.org>
Closes: 844377 951659 977832 985303 987779 989547 990211 990633 999739 1000572
1005745 1006847 1010593 1010906 1011642 1020976
Changes:
fai (6.0~alpha24) 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:
b0d8ec2f61af530898db9825f3ff73a454efc80d 2094 fai_6.0~alpha24.dsc
07583430bf6aebbb2acd34f1a2d808a4c9a181eb 292820 fai_6.0~alpha24.tar.xz
6b65be2e4475a1950046afc99b1deae73fafff37 16335 fai_6.0~alpha24_amd64.buildinfo
Checksums-Sha256:
d0f2b0364318d1862917dd77dc5ee568d25251f138aeceaeef12be82f3a07a6a 2094
fai_6.0~alpha24.dsc
112b5cd06baedb7e0616183cc2bc53e6590e5fe6f5cd2909ad89cf9574d586e1 292820
fai_6.0~alpha24.tar.xz
7bf83ef98d57b4a298a54ad0316a2caaabc85a457db2d297974003a03e34f49c 16335
fai_6.0~alpha24_amd64.buildinfo
Files:
477efbde462111eda05c0bf87a4b69dd 2094 admin optional fai_6.0~alpha24.dsc
43e9469ea26df715fa4aec3548df1811 292820 admin optional fai_6.0~alpha24.tar.xz
b357b1a22fffc8eabc481d37f237a318 16335 admin optional
fai_6.0~alpha24_amd64.buildinfo
-----BEGIN PGP SIGNATURE-----
iQJFBAEBCAAvFiEEsR7jJz9rLetSjJPaK/jZ/gdLzeQFAmNoOQwRHGxhbmdlQGRl
Ymlhbi5vcmcACgkQK/jZ/gdLzeQO3RAAmVUbcFL+kh8F8gNGQyVISx2CUvhrcdZ0
x1lSdUcOZfaQzl2NFgV2oer4W/P+J9bpTG4dr0RUeR9YpsxDz15QOpJPZsw1OSCi
xgca5gGGSi/aZ3Wf+y42n6vHUOGZyxQvw8pcYeJHyXdejVf5UpPcNZoDjoZQHhKc
T0lleH5Dyg0+MoTZl8lWbC+SbXpNGDXkCNztNfhK1Oz/hyb8KeBmTKXH1LR4i5rx
hkGQfXwCWRtl+YgZtP5HJmP4L7aXrF+8O8neA5WaMe78jNQaV/TQ/jo6E1p2jG6j
bzSZtJjZYUL/fCnlFziJPwDAxf6G5AbOOwqg/hLIv+8Y+PkYWle6l3DyAP9YV41K
/GM7FEkkS/lSlJCUTlD7/0B80HB1W3cprpQGkIMwwjHyI4u5P35GlsVq8eKSqm6v
l+VJLqvH0kRnA63qLPxhtpB4RaxeXDVLyowYrc/+mByqIy9m5iVENNesRqr8lGmM
3u3R8rDDm5k4FONfdGt1vLOB3P3M/nsQlBUGFbTfZoYeR34adpoe/8y3WydaveDt
BGWlBhIWoYOxRb023B5nJ06V3fLfpNOAdJ7SQQa3tlBQM3EdPYNL1H5eI4D1eUaF
GxQh/oCB+4wmCCTWvL+LsStlzCLes8tqIdNjemmql1eU2ZyqTXx41ZKVRSdq4RjC
ZJrb3Ohhs6w=
=wUoe
-----END PGP SIGNATURE-----
--- End Message ---
