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

[Thomas Lamprecht]

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.