[pve-devel] applied-series: [PATCH installer/{pve, pmg}-docs/proxmox-backup v3 0/8] raise minimum root password length to 8 characters

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Feb 24 12:16:22 CET 2025


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.




More information about the pve-devel mailing list