Re: [PATCH] tests: refresh package lists with latest libvirt-ci

[Thomas Huth <thuth@redhat.com>]

On 05/11/2024 16.24, Daniel P. Berrangé wrote:
> This updates the libvirt-ci  submodule to pull in various fixes,
> the most notable being
> 
>   * Removal of the macos-15 target. Cirrus CI has changed their
>     infra to only permit testing of a single macOS version,
>     currently set to 14. Our attempt to test macOS 15 was being
>     quietly converted into a macOS 14 test, which we already
>     have.
> 
>   * Reducing native package sets in cross builds. Some packages
>     were mistakenly marked as native, rather than foreign, in
>     libvirt-ci. Fixing this causes our dockerfiles to pick up
>     the cross arch package instead of native one, thus improving
>     our test coverage in a few areas.
> 
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
...
> diff --git a/tests/docker/dockerfiles/debian-mips64el-cross.docker b/tests/docker/dockerfiles/debian-mips64el-cross.docker
> index bfa96cb507..5f298a0f50 100644
> --- a/tests/docker/dockerfiles/debian-mips64el-cross.docker
> +++ b/tests/docker/dockerfiles/debian-mips64el-cross.docker
> @@ -30,10 +30,6 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
>                         git \
>                         hostname \
>                         libglib2.0-dev \
> -                      libgtk-vnc-2.0-dev \
> -                      libpcre2-dev \
> -                      libsndio-dev \
> -                      libspice-protocol-dev \
>                         llvm \
>                         locales \
>                         make \
> @@ -101,6 +97,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
>                         libglib2.0-dev:mips64el \
>                         libglusterfs-dev:mips64el \
>                         libgnutls28-dev:mips64el \
> +                      libgtk-vnc-2.0-dev:mips64el \
>                         libibverbs-dev:mips64el \
>                         libiscsi-dev:mips64el \
>                         libjemalloc-dev:mips64el \
> @@ -112,6 +109,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
>                         libnfs-dev:mips64el \
>                         libnuma-dev:mips64el \
>                         libpam0g-dev:mips64el \
> +                      libpcre2-dev:mips64el \
>                         libpipewire-0.3-dev:mips64el \
>                         libpixman-1-dev:mips64el \
>                         libpng-dev:mips64el \
> @@ -123,6 +121,8 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
>                         libselinux1-dev:mips64el \
>                         libslirp-dev:mips64el \
>                         libsnappy-dev:mips64el \
> +                      libsndio-dev:mips64el \
> +                      libspice-protocol-dev:mips64el \
>                         libspice-server-dev:mips64el \
>                         libssh-gcrypt-dev:mips64el \
>                         libsystemd-dev:mips64el \

  Hi Daniel,

sorry, but looks like this breaks the mips64el container:

  https://gitlab.com/thuth/qemu/-/jobs/8281891574

I guess you likely have to add some of these packages to our mappings.yml 
file now like we've done it here:

  https://gitlab.com/qemu-project/qemu/-/commit/c60473d29254b79d9437e#37203e5ef4f28ec1d635ffdb22d85d7ffb8cf714

  Thomas