[PATCH instance-debootstrap 4/4] Add a test for exercising the scripts

2016-05-29 Thread Iustin Pop
From: Iustin Pop To allow _some_ testing of this code without Ganeti, let's add a simple test that exercises the create/export/import scripts. There are a number of downsides to this script: - it needs real root (due to losetup/kpartx/chroot), so the script checks for this

[PATCH instance-debootstrap 0/4] Cleanups and simple tests

2016-05-29 Thread Iustin Pop
From: Iustin Pop A few cleanups, and adding a test suite to be run without Ganeti, i.e. on standalone systems. However it has downsides, see that patch's description. Note: I haven't tested this patch series under Ganeti itself, but the only non-test modification is the suite

[PATCH instance-debootstrap 1/4] Don't mark common.sh as executable

2016-05-29 Thread Iustin Pop
From: Iustin Pop This is not needed, and is an artifact from the time when we autogenerated the `create' script (commit 6c4bbf6); the migration to common.sh (commit eb23d04) forgot to remove this. Signed-off-by: Iustin Pop --- Makefile.am | 1 - 1 file

[PATCH instance-debootstrap 3/4] Switch default suite to jessie and bump min disk size

2016-05-29 Thread Iustin Pop
From: Iustin Pop wheezy is oldstable now, so let's bump the default. Sadly, this needs a bump in the min disk size, since on a fresh debootstrap call: $ du -sh wheezy jessie 229Mwheezy 283Mjessie So we're over the current 256MB limit during debootstrap run itself,

[PATCH instance-debootstrap 2/4] Check for required programs at configure time

2016-05-29 Thread Iustin Pop
From: Iustin Pop This is not terribly helpful, especially when instance-deboostrap is installed from packages and not from source, however it helps preventing "obvious" missing dependencies for from-source installs. Only tools that come from extra packages are checked, not

Re: [PATCH stable-2.16 1/2] Tune getNodeInstances DRBD secondary computation

2016-05-29 Thread 'Iustin Pop' via ganeti-devel
On Thu, May 26, 2016 at 03:56:10PM +0100, Ganeti Development List wrote: > This is exercised by the luxi QueryInstances call when the sinst_cnt and > sint_list fields are used. This uses a lot of CPU and does a lot of > short-lived heap allocation on clusters with many instances. > > The