Am 16.12.24 um 10:40 schrieb Christoph Heiss: > Diffstat > ======== > > pve-installer: > > Christoph Heiss (5): > common: convert `InstallRootPassword` into an enum > proxinstall: raise minimum root password length to 8 characters > tui: raise minimum root password length to 8 characters > auto: raise minimum root password length to 8 characters > auto: add negative tests for root password option > > Proxmox/Sys.pm | 5 ++ > html/pbs/passwd.htm | 2 +- > html/pmg/passwd.htm | 2 +- > html/pve/passwd.htm | 2 +- > proxinstall | 7 ++- > proxmox-auto-installer/src/utils.rs | 45 ++++++++++---- > proxmox-auto-installer/tests/parse-answer.rs | 61 +++++++++++++++---- > .../tests/resources/parse_answer/btrfs.json | 2 +- > .../tests/resources/parse_answer/btrfs.toml | 2 +- > .../btrfs_raid_level_uppercase.json | 2 +- > .../btrfs_raid_level_uppercase.toml | 2 +- > .../resources/parse_answer/disk_match.json | 2 +- > .../resources/parse_answer/disk_match.toml | 2 +- > .../parse_answer/disk_match_all.json | 2 +- > .../parse_answer/disk_match_all.toml | 2 +- > .../parse_answer/disk_match_any.json | 2 +- > .../parse_answer/disk_match_any.toml | 2 +- > .../resources/parse_answer/first_boot.json | 2 +- > .../resources/parse_answer/first_boot.toml | 2 +- > .../tests/resources/parse_answer/minimal.json | 2 +- > .../tests/resources/parse_answer/minimal.toml | 2 +- > .../resources/parse_answer/nic_matching.json | 2 +- > .../resources/parse_answer/nic_matching.toml | 2 +- > .../resources/parse_answer/specific_nic.json | 2 +- > .../resources/parse_answer/specific_nic.toml | 2 +- > .../tests/resources/parse_answer/zfs.json | 2 +- > .../tests/resources/parse_answer/zfs.toml | 2 +- > .../zfs_raid_level_uppercase.json | 2 +- > .../zfs_raid_level_uppercase.toml | 2 +- > .../both_password_and_hashed_set.json | 3 + > .../both_password_and_hashed_set.toml | 15 +++++ > .../no_root_password_set.json | 3 + > .../no_root_password_set.toml | 13 ++++ > .../parse_answer_fail/short_password.json | 3 + > .../parse_answer_fail/short_password.toml | 14 +++++ > proxmox-installer-common/src/lib.rs | 3 + > proxmox-installer-common/src/setup.rs | 25 +++++--- > proxmox-tui-installer/src/main.rs | 10 ++- > proxmox-tui-installer/src/setup.rs | 5 +- > 39 files changed, 197 insertions(+), 65 deletions(-) > create mode 100644 > proxmox-auto-installer/tests/resources/parse_answer_fail/both_password_and_hashed_set.json > create mode 100644 > proxmox-auto-installer/tests/resources/parse_answer_fail/both_password_and_hashed_set.toml > create mode 100644 > proxmox-auto-installer/tests/resources/parse_answer_fail/no_root_password_set.json > create mode 100644 > proxmox-auto-installer/tests/resources/parse_answer_fail/no_root_password_set.toml > create mode 100644 > proxmox-auto-installer/tests/resources/parse_answer_fail/short_password.json > create mode 100644 > proxmox-auto-installer/tests/resources/parse_answer_fail/short_password.toml > > pve-docs: > > Christoph Heiss (1): > installation: adapt to raised root password length requirement > > pve-installation.adoc | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > pmg-docs: > > Christoph Heiss (1): > installation: adapt to raised root password length requirement > > pmg-installation.adoc | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > proxmox-backup: > > Christoph Heiss (1): > using-the-installer: adapt to raised root password length requirement > > docs/using-the-installer.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >
applied whole series, thanks! I added a bit more context in the first installer patch's commit message, to highlight that this is no semantic change and the previously theoretical possible case of having neither password variant set was not allowed due to a dedicated check. _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
