On 9/8/23 13:49, Daniel P. Berrangé wrote: > On Fri, Sep 08, 2023 at 01:34:54PM +0200, Ilya Maximets wrote: >> On 9/8/23 13:19, Stefan Hajnoczi wrote: >>> Hi Ilya and Jason, >>> There is a CI failure related to a missing Debian libxdp-dev package: >>> https://gitlab.com/qemu-project/qemu/-/jobs/5046139967 >>> >>> I think the issue is that the debian-amd64 container image that QEMU >>> uses for testing is based on Debian 11 ("bullseye" aka "oldstable") >>> and libxdp is not available on that release: >>> https://packages.debian.org/search?keywords=libxdp&searchon=names&suite=oldstable§ion=all >> >> Hmm. Sorry about that. >> >>> >>> If we need to support Debian 11 CI then either XDP could be disabled >>> for that distro or libxdp could be compiled from source. >> >> I'd suggest we just remove the attempt to install the package for now, >> building libxdp from sources may be a little painful to maintain. >> >> Can be re-added later once distributions with libxdp 1.4+ will be more >> widely available, i.e. when fedora dockerfile will be updated to 39, >> for example. That should be soon-ish, right? > > If you follow the process in docs/devel/testing.rst for adding > libxdp in libvirt-ci, then lcitool will "do the right thing" > when we move the auto-generated dockerfiles to new distro versions.
Thanks! I'll prepare changes for libvirt-ci. In the meantime, none of the currently tested images will have a required version of libxdp anyway, so I'm suggesting to just drop this one dockerfile modification from the patch. What do you think? Best regards, Ilya Maximets.