curtin-dev team mailing list archive

Thread
Date

[Merge] curtin:integration-memory-error into curtin:master

To: mp+442650@xxxxxxxxxxxxxxxxxx
From: Olivier Gayot <mp+442650@xxxxxxxxxxxxxxxxxx>
Date: Thu, 11 May 2023 14:02:22 -0000
In-reply-to: <168380785197.28859.11046281906228035820.launchpad@juju-4112d9-prod-launchpad-manual-servers-36.openstack.prodstack5.lan>
Reply-to: mp+442650@xxxxxxxxxxxxxxxxxx
Sender: noreply@xxxxxxxxxxxxx

The proposal to merge curtin:integration-memory-error into curtin:master has been updated.

Commit message changed to:

tests: fix memory error in integration test when we allocate a 2GiB string

When create=True, the function add_image does something similar to what
'dd if=/dev/zero bs=1 count={size}' would do.

That said, our implementation allocates a string of the requested size
in memory. This fails with MemoryError on systems with limited RAM
available.

Instead, we now call 'dd', which is more memory efficient.

For more details, see:
https://code.launchpad.net/~curtin-dev/curtin/+git/curtin/+merge/442650
-- 
Your team curtin developers is requested to review the proposed merge of curtin:integration-memory-error into curtin:master.