Re: [pdm-devel] [PATCH proxmox/datacenter-manager 00/13] initial auto-installer integration

From: "Lukas Wagner" <l.wagner@proxmox.com>
To: "Proxmox Datacenter Manager development discussion"
	<pdm-devel@lists.proxmox.com>,
	"Christoph Heiss" <c.heiss@proxmox.com>
Subject: Re: [pdm-devel] [PATCH proxmox/datacenter-manager 00/13] initial auto-installer integration
Date: Thu, 04 Dec 2025 15:17:45 +0100	[thread overview]
Message-ID: <DEPHWEJKLI1W.1STZHWCK10P0H@proxmox.com> (raw)
In-Reply-To: <20251204125122.945961-1-c.heiss@proxmox.com>

On Thu Dec 4, 2025 at 1:51 PM CET, Christoph Heiss wrote:
> This series adds integration with our automated installer [0] for all our
> products. With this, Proxmox Datacenter Manager can be used for serving
> answer files via HTTP(S) in an automated fashion.
>
> It provides two new panels:
>
> - Automated Installations: List all past and currently ongoing
>   installations.
> - Prepared Answers: Enables users to do the whole CRUD cycle for
>   prepared answers, i.e. create new answers (from scratch or based on
>   existing ones), edit and delete.
>
> Permission-wise, everything is currently scoped on
> /system/auto-installation.
>
> UI
> ==
>
> Happy about feedback regarding the UI, especially the editing dialog! By
> the nature of it, the auto-installer has a lot more options than the
> standard GUI/TUI installer, and it's kind of hard to not fully cramp up
> the UI while still providing all options.
>
> TOML API
> ========
>
> The auto-installer (currently) only supports TOML as input format for
> the answer, so we need to hack around a bit to serve TOML in the API.
> This is done in patch #12, by implementing the api method
> directly. 
>
> Serving TOML from an endpoint under /api2/json/ obviously is rather
> wrong, so I'm definitely open for suggestions. 
> We probably also don't want to implement `/api2/toml/` if I'd have to
> guess (or would that be possible selectively, i.e. just for this one
> endpoint?).
>
> Adding JSON support to proxmox-fetch-answer is on my list, but this
> would still be needed if we want to keep compatibility with older ISOs.
>

Cool stuff, I didn't even know that somebody was working on this!

Couldn't really try it out, I think there is some issue in the patches
that you've posted. I'm getting a 'unresolved import
`proxmox_installer_types::answer::fetch::AnswerFetchResponse' error when
trying to compile PDM. Checking the code in the installer-types crate, I
also couldn't find a type like this - could you cross-check?

Thanks!

_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel