From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Thomas Huth <thuth@redhat.com>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
qemu-block@nongnu.org, "Michael Tokarev" <mjt@tls.msk.ru>,
"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
qemu-devel@nongnu.org, "Yonggang Luo" <luoyonggang@gmail.com>,
"Alex Bennée" <alex.bennee@linaro.org>,
"Warner Losh" <imp@bsdimp.com>
Subject: Re: [PATCH v4 4/8] tests: Refresh lcitool submodule
Date: Fri, 21 Jan 2022 11:47:16 +0000 [thread overview]
Message-ID: <YeqdROki4KJkT8RR@redhat.com> (raw)
In-Reply-To: <9d09394f-20b3-eb07-d962-9d723b0d3c3d@redhat.com>
On Fri, Jan 21, 2022 at 12:40:48PM +0100, Thomas Huth wrote:
> On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
> > Refresh lcitool submodule and the generated files by running:
> >
> > $ make lcitool-refresh
> >
> > Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
> > Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> > ---
> > .gitlab-ci.d/cirrus/freebsd-12.vars | 2 +-
> > .gitlab-ci.d/cirrus/freebsd-13.vars | 2 +-
> > tests/docker/dockerfiles/alpine.docker | 3 ++-
> > tests/docker/dockerfiles/centos8.docker | 3 +--
> > tests/docker/dockerfiles/fedora.docker | 3 +--
> > tests/docker/dockerfiles/opensuse-leap.docker | 2 +-
> > tests/docker/dockerfiles/ubuntu1804.docker | 2 +-
> > tests/docker/dockerfiles/ubuntu2004.docker | 2 +-
> > tests/lcitool/libvirt-ci | 2 +-
> > 9 files changed, 10 insertions(+), 11 deletions(-)
> >
> > diff --git a/.gitlab-ci.d/cirrus/freebsd-12.vars b/.gitlab-ci.d/cirrus/freebsd-12.vars
> > index 9c52266811f..bdcce578edf 100644
> > --- a/.gitlab-ci.d/cirrus/freebsd-12.vars
> > +++ b/.gitlab-ci.d/cirrus/freebsd-12.vars
> > @@ -11,6 +11,6 @@ MAKE='/usr/local/bin/gmake'
> > NINJA='/usr/local/bin/ninja'
> > PACKAGING_COMMAND='pkg'
> > PIP3='/usr/local/bin/pip-3.8'
> > -PKGS='alsa-lib bash bzip2 ca_root_nss capstone4 ccache cdrkit-genisoimage ctags curl cyrus-sasl dbus diffutils dtc gettext git glib gmake gnutls gsed gtk3 libepoxy libffi libgcrypt libjpeg-turbo libnfs libspice-server libssh libtasn1 libxml2 llvm lttng-ust lzo2 meson ncurses nettle ninja opencv p5-Test-Harness perl5 pixman pkgconf png py38-numpy py38-pillow py38-pip py38-sphinx py38-sphinx_rtd_theme py38-virtualenv py38-yaml python3 rpm2cpio sdl2 sdl2_image snappy spice-protocol tesseract texinfo usbredir virglrenderer vte3 zstd'
> > +PKGS='alsa-lib bash bzip2 ca_root_nss capstone4 ccache cdrkit-genisoimage ctags curl cyrus-sasl dbus diffutils dtc fusefs-libs3 gettext git glib gmake gnutls gsed gtk3 libepoxy libffi libgcrypt libjpeg-turbo libnfs libspice-server libssh libtasn1 libxml2 llvm lttng-ust lzo2 meson ncurses nettle ninja opencv p5-Test-Harness perl5 pixman pkgconf png py38-numpy py38-pillow py38-pip py38-sphinx py38-sphinx_rtd_theme py38-virtualenv py38-yaml python3 rpm2cpio sdl2 sdl2_image snappy spice-protocol tesseract texinfo usbredir virglrenderer vte3 zstd'
>
> Aren't the FreeBSD jobs currently failing due to lttng-ust not being
> available anymore? ... I'd somehow expected that this update might fix this,
> too, but I still see lttng-ust in the list here?
I had prepped an update to drop it, but @bsdimp indicated that it ought
to be a transient problem so I never applied it, hoping it would fix
itself.
https://gitlab.com/libvirt/libvirt-ci/-/merge_requests/211
It has been broken for ~2 weeks now though, and lttng-ust is not an
especially critical feature so we should probably just go ahead and
disable it regardless.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2022-01-21 11:52 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-21 10:36 [PATCH v4 0/8] tests: Refresh lcitool submodule & remove libxml2 Philippe Mathieu-Daudé via
2022-01-21 10:36 ` [PATCH v4 1/8] MAINTAINERS: Cover lcitool submodule with build test / automation Philippe Mathieu-Daudé via
2022-01-21 11:35 ` Thomas Huth
2022-01-21 10:36 ` [PATCH v4 2/8] gitmodules: Correct libvirt-ci URL Philippe Mathieu-Daudé via
2022-01-21 10:46 ` Philippe Mathieu-Daudé via
2022-01-21 10:47 ` Daniel P. Berrangé
2022-01-21 11:36 ` Thomas Huth
2022-01-21 10:36 ` [PATCH v4 3/8] tests/lcitool: Include local qemu.yml when refreshing cirrus-ci files Philippe Mathieu-Daudé via
2022-01-21 10:36 ` [PATCH v4 4/8] tests: Refresh lcitool submodule Philippe Mathieu-Daudé via
2022-01-21 11:40 ` Thomas Huth
2022-01-21 11:47 ` Daniel P. Berrangé [this message]
2022-01-21 14:30 ` Warner Losh
2022-01-21 14:34 ` Daniel P. Berrangé
2022-01-21 15:37 ` Philippe Mathieu-Daudé via
2022-01-21 10:36 ` [PATCH v4 5/8] tests/lcitool: Install libibumad to cover RDMA on Debian based distros Philippe Mathieu-Daudé via
2022-01-21 10:36 ` [PATCH v4 6/8] drop libxml2 checks since libxml is not actually used (for parallels) Philippe Mathieu-Daudé via
2022-01-21 11:45 ` Thomas Huth
2022-01-21 10:36 ` [PATCH v4 7/8] tests/lcitool: Remove libxml2 Philippe Mathieu-Daudé via
2022-01-21 11:47 ` Thomas Huth
2022-01-21 12:50 ` Philippe Mathieu-Daudé via
2022-01-21 10:36 ` [PATCH v4 8/8] tests: Manually remove libxml2 on MSYS2 runners Philippe Mathieu-Daudé via
2022-01-21 11:49 ` Thomas Huth
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YeqdROki4KJkT8RR@redhat.com \
--to=berrange@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=f4bug@amsat.org \
--cc=imp@bsdimp.com \
--cc=luoyonggang@gmail.com \
--cc=mjt@tls.msk.ru \
--cc=peter.maydell@linaro.org \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).