[libvirt] [PATCH 4/6] port allocator: drop skip flag
Michal Privoznik
mprivozn at redhat.com
Mon Jan 29 06:09:42 UTC 2018
On 12/20/2017 07:35 AM, Nikolay Shirokovskiy wrote:
> This flag is only used for tests and tests overload socket and bind
> functions using virportallocatormock.c already in a suitable fashion
> so we don't need this flag at all.
> ---
> src/bhyve/bhyve_driver.c | 2 +-
> src/libxl/libxl_driver.c | 5 ++---
> src/qemu/qemu_driver.c | 9 +++------
> src/util/virportallocator.c | 14 ++++----------
> src/util/virportallocator.h | 7 +------
> tests/bhyvexml2argvtest.c | 3 +--
> tests/libxlxml2domconfigtest.c | 3 +--
> tests/virportallocatortest.c | 4 ++--
> 8 files changed, 15 insertions(+), 32 deletions(-)
It's not that easy. libxlxml2domconfigtest doesn't link with the mock
therefore if say port 5900 is opened at the time the test runs I get
different results.
Michal
More information about the libvir-list
mailing list