[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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



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

Attachment: signature.asc
Description: PGP signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]