[Libguestfs] [PATCH] v2v: Further increase memory allocated to the appliance (RHBZ#1418283).

[Pino Toscano]

On Wednesday, 1 February 2017 13:40:17 CET Richard W.M. Jones wrote:
> In commit 08f82f2e3d6975b72340dd59f438862e152a15ef we increased the
> memory size to 800MB (on x86) so that the semodule command would work.
> 
> However it has been discovered that another SELinux command takes
> large amounts of RAM (setfiles during the SELinux relabel step).
> Therefore increase the memory size again, this time to 2000MB.
> ---

LGTM.

Should v2v get a -m parameter like the other tools?

> diff --git a/v2v/virt-v2v.pod b/v2v/virt-v2v.pod
> index 4614888..88bfd7f 100644
> --- a/v2v/virt-v2v.pod
> +++ b/v2v/virt-v2v.pod
> @@ -1634,7 +1634,7 @@ alleviate this.
>  
>  Virt-v2v is not especially compute or RAM intensive.  If you are
>  running many parallel conversions, then you may consider allocating
> -one CPU core and between 512 MB and 1 GB of RAM per running instance.
> +one CPU core and 2 GB of RAM per running instance.

I guess pointing to LIBGUESTFS_MEMSIZE here could make it more
actionable on issues.

Thanks,
-- 
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20170201/240729eb/attachment.sig>