Re: [PATCH v2 2/3] CI: Update x86 to use Debian Trixie

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

On 11/09/2025 4:06 am, Marek Marczykowski-Górecki wrote:
> On Wed, Sep 10, 2025 at 11:23:12PM +0100, Andrew Cooper wrote:
>> diff --git a/automation/build/debian/13-x86_64.dockerfile 
>> b/automation/build/debian/13-x86_64.dockerfile
>> new file mode 100644
>> index 000000000000..20e9d2f3f52d
>> --- /dev/null
>> +++ b/automation/build/debian/13-x86_64.dockerfile
>> @@ -0,0 +1,76 @@
>> +# syntax=docker/dockerfile:1
>> +FROM --platform=linux/amd64 debian:trixie-slim
>> +LABEL maintainer.name="The Xen Project"
>> +LABEL maintainer.email="xen-devel@xxxxxxxxxxxxxxxxxxxx"
>> +
>> +ENV DEBIAN_FRONTEND=noninteractive
>> +
>> +RUN <<EOF
>> +#!/bin/bash
>> +    set -eu
>> +
>> +    useradd --create-home user
>> +
>> +    apt-get update
>> +
>> +    DEPS=(
>> +        # Xen
>> +        bison
>> +        build-essential
>> +        checkpolicy
>> +        clang
>> +        flex
>> +
>> +        # Tools (general)
>> +        ca-certificates
>> +        git-core
>> +        pkg-config
>> +        wget
>> +        # libxenguest dombuilder
>> +        libbz2-dev
>> +        liblzma-dev
>> +        liblzo2-dev
>> +        libzstd-dev
>> +        zlib1g-dev
>> +        # libacpi
>> +        acpica-tools
>> +        # libxl
>> +        uuid-dev
>> +        libnl-3-dev
>> +        libyajl-dev
>> +        # RomBIOS
>> +        bcc
>> +        bin86
>> +        # xentop
>> +        libncurses5-dev
>> +        # Python bindings
>> +        python3-dev
>> +        python3-setuptools
>> +        # Golang bindings
>> +        golang-go
>> +        # Ocaml bindings/oxenstored
>> +        ocaml-nox
>> +        ocaml-findlib
>> +
>> +        # for test phase, qemu-smoke-* jobs
>> +        expect
>> +        qemu-system-x86
> qemu-smoke-* remain at debian:12 here.

Oh, that was an oversight.  And I missed in in RISC-V.


>  And IIUC the plan is to split
> build and test containers, so those dependencies are not needed here.

I maybe didn't phrase the comment very well, but the split I intended
was the hardware-running containers vs everything else.

What the hardware containers need is almost disjoint with everything else.

>
>> +        # for build-each-commit-gcc
>> +        ccache
>> +
>> +        # for qemu-alpine-x86_64-gcc
>> +        busybox-static
> Same comment as above, but ...
>
>> +        cpio
> ... this should remain, and likely moved to "Tools (general)" section.

Hmm, probably yes, although I'll do that as a prep patch and fix all the
impacted containers.

>
>> +
>> +        # For *-efi jobs
>> +        ovmf
> And this one too.

Tools general?  I don't think so, no.

But we probably should have a combined section for qemu based testing
and call it done.  This is overly fine-grained IMO.

~Andrew