The goal of this series is to add additional sanity checks to the auto-installer and the TUI and GUI installers. The following checks were added: * Btrfs / ZFS RAID: check if the required number of disks is available * LVM: check if swapsize < hdsize (RFC) * LVM: check if maxroot < hdsize/4 (RFC) * Networking: check if IPv4 address is valid within subnet (RFC)
The disk-related checks aim to close [0], whereas the IPv4 check would partially close [1]. [0] https://bugzilla.proxmox.com/show_bug.cgi?id=5887 [1] https://bugzilla.proxmox.com/show_bug.cgi?id=5757 The patches marked as RFC work, but I think some discussion regarding what we even want to check is warranted here. I added some more context for a discussion to the individual patches. pve-installer: Michael Köppl (6): auto: add early answer file sanity check for RAID configurations common: use get_min_disks as single source of truth for RAID config checks close #5887: add sanity check for LVM swapsize and maxroot run rustfmt common: add more descriptive errors for invalid network configs common: add checks for valid IPv4 address within subnet Proxmox/Install.pm | 14 +++++ proxinstall | 5 +- proxmox-auto-install-assistant/src/main.rs | 7 ++- proxmox-auto-installer/src/utils.rs | 37 +++++++++-- proxmox-auto-installer/tests/parse-answer.rs | 8 ++- .../btrfs_raid_single_disk.json | 3 + .../btrfs_raid_single_disk.toml | 15 +++++ .../lvm_maxroot_greater_than_maximum.json | 3 + .../lvm_maxroot_greater_than_maximum.toml | 16 +++++ .../lvm_swapsize_greater_than_hdsize.json | 3 + .../lvm_swapsize_greater_than_hdsize.toml | 16 +++++ .../zfs_raid_single_disk.json | 3 + .../zfs_raid_single_disk.toml | 15 +++++ proxmox-installer-common/src/disk_checks.rs | 38 ++++++----- proxmox-installer-common/src/options.rs | 36 ++++++++++- proxmox-installer-common/src/utils.rs | 63 ++++++++++++++++--- proxmox-tui-installer/src/views/bootdisk.rs | 14 +++-- 17 files changed, 256 insertions(+), 40 deletions(-) create mode 100644 proxmox-auto-installer/tests/resources/parse_answer_fail/btrfs_raid_single_disk.json create mode 100644 proxmox-auto-installer/tests/resources/parse_answer_fail/btrfs_raid_single_disk.toml create mode 100644 proxmox-auto-installer/tests/resources/parse_answer_fail/lvm_maxroot_greater_than_maximum.json create mode 100644 proxmox-auto-installer/tests/resources/parse_answer_fail/lvm_maxroot_greater_than_maximum.toml create mode 100644 proxmox-auto-installer/tests/resources/parse_answer_fail/lvm_swapsize_greater_than_hdsize.json create mode 100644 proxmox-auto-installer/tests/resources/parse_answer_fail/lvm_swapsize_greater_than_hdsize.toml create mode 100644 proxmox-auto-installer/tests/resources/parse_answer_fail/zfs_raid_single_disk.json create mode 100644 proxmox-auto-installer/tests/resources/parse_answer_fail/zfs_raid_single_disk.toml Summary over all repositories: 17 files changed, 256 insertions(+), 40 deletions(-) -- Generated by git-murpp 0.8.0 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel