[pve-devel] applied: [PATCH installer v2 0/5] fix #5579: allow specifying optional first-boot script

[Thomas Lamprecht]

Am 18.11.24 um 13:38 schrieb Christoph Heiss:
> This implements #5579 [0] as proposed by Thomas [1].
> 
> Adds a new (optional) section to the auto-installer answer file called
> `first-boot`, which can be used to the configure a script/executable
> file to run on the first boot after the installation.
> 
> To used the baked-in (via the `proxmox-auto-install-assistant prepare-iso
> --on-first-boot`) file from the ISO:
> 
>   [first-boot]
>   source = "from-iso"
> 
> Or fetching it from a URL:
> 
>   [first-boot]
>   source = "from-url"
>   url = "http://example.com/first-boot"
>   cert_fingerprint = ".." # if needed
> 
> Optionally, a ordering can be specified, when to run the script:
> 
>   [first-boot]
>   source = "from-url"
>   ordering = "fully-up" # default value
>   # one of "before-network", "network-online" or "fully-up"
> 
> The structure the section is mostly taken from the `network` section to
> provide consistency.
> 
> [0] https://bugzilla.proxmox.com/show_bug.cgi?id=5579
> [1] https://bugzilla.proxmox.com/show_bug.cgi?id=5579#c5
> 

Looks all right to me, nice work! Would be great if you could add that new feature
to our auto-installer docs.

> Diffstat
> ========
> 
> Christoph Heiss (5):
>   fix #5579: first-boot: add initial service packaging
>   fix #5579: setup: introduce 'first_boot' low-level installer options
>   fix #5579: auto-install-assistant: enable baking in first-boot script
>   fix #5579: auto-installer: add optional first-boot hook script
>   fix #5579: install: setup proxmox-first-boot service if enabled


applied, thanks!