[Libguestfs] [libnbd PATCH 0/5] Introduce automated testing using GitLab CI

[Martin Kletzander]

On Tue, May 11, 2021 at 12:16:19PM +0100, Richard W.M. Jones wrote:
>On Thu, May 06, 2021 at 01:30:05PM +0200, Martin Kletzander wrote:
>> This is reworked version of
>>
>>   https://listman.redhat.com/archives/libguestfs/2021-March/msg00134.html
>>
>> with more packages installed so that more code paths are tested, also some fixes
>> in the code (mostly requirements).
>>
>> There are still some things that fail.
>>
>> FreeBSD
>> =======
>>
>> The build fails because there is no fallocate() there.
>
>I'm actually not seeing a failure (FreeBSD 12.2).  Do you happen to
>know what the build error was?
>

The error is visible here:

   https://gitlab.com/nertpinx/libnbd/-/jobs/1237433032#L2185

is it possible that I did not include some dependency in the system?  Or
maybe it is compiling a file that is not compiled on your setup?  I
could not find anything fallocate() related for FreeBSD, only
posix_fallocate().

>I'm seeing test failures though which I'll fix if they are easy.  I
>think for non-Linux right now I'm most concerned about build failures.
>
>> Many platforms (those that have nbd-server in repositories)
>> ===========================================================
>>
>> interop/interop-nbd-server fails with:
>>
>>   Error: inetd mode requires syslog
>>   Exiting.
>>
>> which makes sense as it is run with port 0, but how to check for
>> whether it is compiled with or without syslog (so that we know to
>> skip the test) or whether it is something else that is missing
>> (capabilities in a container) is beyond me.
>
>I wasn't able to reproduce this (on FreeBSD).  Do you know which
>operating systems have nbd-server compiled without syslog?
>

This was some Linux distro and I do not know if it actually is "build
without syslog", it only looked like it based on the nbd-server code
that I went through.  But it might be some missing capability or
anything else that is actually affected by the fact that the builds and
tests are running in containers.

I might have mixed this one and the below.  You can try it on Ubuntu
18.04, but you would have to try it locally, you cannot see it in the CI
because it times out and it requires manual intervention.  Basically see
below.

>> interop/list-exports-nbd-server gets stuck and in the CI times out the whole
>> test suite.  Locally I need to manually exec into the container:
>>
>>   podman exec -itu bash
>>
>> and I see the nbd-server is just not doing anything.  After killing
>> it the test ends and fails.  I did not debug nbd-server for this,
>> maybe it is again some requisite that is missing and we are not
>> checking for it (capability, nbd module access, etc.), I do not
>> know.
>
>FreeBSD seems OK here.  Do you know which OSes this affected?
>

I think Ubuntu 18.04, see above.

>> There are also some skips that I did not go into any more after
>> trying to figure out every "... no" in config.log:
>
>I wouldn't worry too much about these in terms of getting things
>working.  We can always extend the CI later.
>
>> Both interop-qemu-nbd-tls-certs and interop-qemu-nbd-tls-psk get
>> skipped because they are missing the keys, but I presume those are
>> supposed to be generated.
>
>These require "certtool" (part of gnutls).
>

I made sure I installed gnutls-bin (or whatever gnutls package provides
certtool), but I might've missed that certool is in another package on
some particular distro.

>> I think that all these can be figured out after the final patch is
>> merged so that the CI runs truly automatically.  I am posting it for
>> review though as I am not sure the solutions in first four patches
>> are the preferred ones.
>
>I agree.
>
>Thanks!
>
>Rich.
>
>-- 
>Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
>Read my programming and virtualization blog: http://rwmj.wordpress.com
>libguestfs lets you edit virtual machines.  Supports shell scripting,
>bindings from many languages.  http://libguestfs.org
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20210512/b7be980b/attachment.sig>