[pve-devel] [PATCH installer 1/4] low-level: initialize UI backend for 'dump-env' subcommand too

Aaron Lauterer a.lauterer at proxmox.com
Fri Feb 9 13:36:12 CET 2024


Is this something I should consider for the new dump-udev subcommand as well?

https://lists.proxmox.com/pipermail/pve-devel/2024-January/061433.html

On 2/9/24 11:55, Christoph Heiss wrote:
> Some detection routines might try to log things and call some
> Proxmox::Ui functions all the way down, so just initialize it with the
> stdio backend to avoid errors.
> 
> Signed-off-by: Christoph Heiss <c.heiss at proxmox.com>
> ---
>   proxmox-low-level-installer | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/proxmox-low-level-installer b/proxmox-low-level-installer
> index d127a40..2848295 100755
> --- a/proxmox-low-level-installer
> +++ b/proxmox-low-level-installer
> @@ -91,6 +91,7 @@ Proxmox::Log::init("/tmp/install-low-level-${cmd}.log");
>   
>   my $env = Proxmox::Install::ISOEnv::get();
>   if ($cmd eq 'dump-env') {
> +    Proxmox::UI::init_stdio({}, $env);
>   
>       my $out_dir = $env->{locations}->{run};
>       make_path($out_dir);




More information about the pve-devel mailing list