Re: [pve-devel] [PATCH manager v6 2/9] ui: guest import: add ova-needs-extracting warning text

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

Would squash that into the patch using it [0], but no hard feelings.

[0] "ui: guest import: add storage selector for ova extraction storage"

Am 15.11.24 um 16:17 schrieb Dominik Csapak:
> Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
> ---
>  www/manager6/window/GuestImport.js | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/www/manager6/window/GuestImport.js b/www/manager6/window/GuestImport.js
> index 2577ece2..1483d97f 100644
> --- a/www/manager6/window/GuestImport.js
> +++ b/www/manager6/window/GuestImport.js
> @@ -937,6 +937,7 @@ Ext.define('PVE.window.GuestImport', {
>  		    gettext('EFI state cannot be imported, you may need to reconfigure the boot order (see {0})'),
>  		    '<a href="https://pve.proxmox.com/wiki/OVMF/UEFI_Boot_Entries">OVMF/UEFI Boot Entries</a>',
>  		),
> +		'ova-needs-extracting': gettext('Importing from an OVA requires extra space while extracting the contained disks into the import or selected storage.'),

maybe one of the following two variants is a bit easier to follow:

'Importing an OVA temporarily requires additional space on the working storage while extracting the contained disks for further processing.'

or

'Importing an OVA temporarily requires additional space on the working storage while the disks are being extracted for further processing.'

I left out mention which storage, as there's a dedicated field for overriding
with a fitting emptyText for the default that should be clear already.



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