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

Christoph Heiss c.heiss at proxmox.com
Mon Dec 9 13:45:53 CET 2024


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(-)

-- 
2.47.0





More information about the pve-devel mailing list