[pve-devel] applied-series: [PATCH installer 0/6] assistant: properly validate answer file settings

Thomas Lamprecht t.lamprecht at proxmox.com
Tue Dec 10 18:37:18 CET 2024


Am 09.12.24 um 13:45 schrieb Christoph Heiss:
> Currently, the answer file is only syntax-checked, i.e. if it can be
> parsed as TOML and whether it is of valid structure, but the actual
> answer settings are not looked at.
> 
> Patch #2 is a refactoring of the whole locale info generation, as to
> generate the final information structure at build-time - thus being able
> to include it in the auto-install-assistant for verify answer file
> settings.
> 
> Christoph Heiss (6):
>   tui: options: simplify unit-test setup
>   country.pl: generate final structure as json at build time directly
>   common: setup: read locale info as shipped by the installer directly
>   common: setup: include path in error message if file could not be read
>   fix #5889: assistant: validate answer email & root password settings
>   assistant: validate answer first-boot hook and locale settings
> 
>  .gitignore                                    |   2 +-
>  Makefile                                      |  16 +--
>  Proxmox/Install/ISOEnv.pm                     |  56 +--------
>  country.pl                                    | 116 ++++++++++++++----
>  proxmox-auto-install-assistant/src/main.rs    |  18 ++-
>  proxmox-auto-installer/src/utils.rs           |   6 +-
>  .../tests/resources/iso-info.json             |   2 +-
>  proxmox-installer-common/src/setup.rs         |  34 +++--
>  proxmox-low-level-installer                   |   3 -
>  proxmox-tui-installer/src/options.rs          |  26 +---
>  10 files changed, 152 insertions(+), 127 deletions(-)
> 


applied series, thanks!




More information about the pve-devel mailing list