Re: [PATCH v3 0/4] tests/vm/freebsd: Get up-to-date package list from lcitool
On Mon, Jul 17, 2023 at 02:30:03PM +0200, Erik Skultety wrote: > On Mon, Jul 17, 2023 at 02:18:34PM +0200, Thomas Huth wrote: > > On 11/07/2023 16.49, Philippe Mathieu-Daudé wrote: > > > Inspired by this patch from Thomas: > > > https://lore.kernel.org/qemu-devel/20230531090415.40421-1-th...@redhat.com/ > > > > > > Instead of updating the package list manually, use lcitool vars file. > > > > > > Since v2: > > > - Commit generated json (Daniel) > > > > > > Since v1: > > > - Addressed Erik & Daniel comments (generate in JSON) > > > > > > Philippe Mathieu-Daudé (4): > > >tests/lcitool: Generate distribution packages list in JSON format > > >tests/lcitool: Refresh generated files > > >tests/vm: Introduce get_qemu_packages_from_lcitool_json() helper > > >tests/vm/freebsd: Get up-to-date package list from lcitool vars file > > > > > > tests/docker/dockerfiles/debian-amd64.docker | 2 - > > > tests/docker/dockerfiles/ubuntu2004.docker | 2 - > > > tests/docker/dockerfiles/ubuntu2204.docker | 2 - > > > > Thanks, I'm queuing your patches for my next pull request - but I'm dropping > > the hunks that change libpmem-dev and libxen-dev in the docker files - > > otherwise this would just get reverted once somebody using a x86 host runs > > the refresh step again. I hope this issue can be tackled by a future lcitool > > update instead. > > > > Thomas > > > > Can you guys please create an issue at > https://gitlab.com/libvirt/libvirt-ci.git to have this officially tracked? Done: https://gitlab.com/libvirt/libvirt-ci/-/issues/26 With 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 :|
Re: [PATCH v3 0/4] tests/vm/freebsd: Get up-to-date package list from lcitool
On Mon, Jul 17, 2023 at 02:18:34PM +0200, Thomas Huth wrote: > On 11/07/2023 16.49, Philippe Mathieu-Daudé wrote: > > Inspired by this patch from Thomas: > > https://lore.kernel.org/qemu-devel/20230531090415.40421-1-th...@redhat.com/ > > > > Instead of updating the package list manually, use lcitool vars file. > > > > Since v2: > > - Commit generated json (Daniel) > > > > Since v1: > > - Addressed Erik & Daniel comments (generate in JSON) > > > > Philippe Mathieu-Daudé (4): > >tests/lcitool: Generate distribution packages list in JSON format > >tests/lcitool: Refresh generated files > >tests/vm: Introduce get_qemu_packages_from_lcitool_json() helper > >tests/vm/freebsd: Get up-to-date package list from lcitool vars file > > > > tests/docker/dockerfiles/debian-amd64.docker | 2 - > > tests/docker/dockerfiles/ubuntu2004.docker | 2 - > > tests/docker/dockerfiles/ubuntu2204.docker | 2 - > > Thanks, I'm queuing your patches for my next pull request - but I'm dropping > the hunks that change libpmem-dev and libxen-dev in the docker files - > otherwise this would just get reverted once somebody using a x86 host runs > the refresh step again. I hope this issue can be tackled by a future lcitool > update instead. > > Thomas > Can you guys please create an issue at https://gitlab.com/libvirt/libvirt-ci.git to have this officially tracked? Thanks, Erik
Re: [PATCH v3 0/4] tests/vm/freebsd: Get up-to-date package list from lcitool
On 11/07/2023 16.49, Philippe Mathieu-Daudé wrote: Inspired by this patch from Thomas: https://lore.kernel.org/qemu-devel/20230531090415.40421-1-th...@redhat.com/ Instead of updating the package list manually, use lcitool vars file. Since v2: - Commit generated json (Daniel) Since v1: - Addressed Erik & Daniel comments (generate in JSON) Philippe Mathieu-Daudé (4): tests/lcitool: Generate distribution packages list in JSON format tests/lcitool: Refresh generated files tests/vm: Introduce get_qemu_packages_from_lcitool_json() helper tests/vm/freebsd: Get up-to-date package list from lcitool vars file tests/docker/dockerfiles/debian-amd64.docker | 2 - tests/docker/dockerfiles/ubuntu2004.docker | 2 - tests/docker/dockerfiles/ubuntu2204.docker | 2 - Thanks, I'm queuing your patches for my next pull request - but I'm dropping the hunks that change libpmem-dev and libxen-dev in the docker files - otherwise this would just get reverted once somebody using a x86 host runs the refresh step again. I hope this issue can be tackled by a future lcitool update instead. Thomas
[PATCH v3 0/4] tests/vm/freebsd: Get up-to-date package list from lcitool
Inspired by this patch from Thomas: https://lore.kernel.org/qemu-devel/20230531090415.40421-1-th...@redhat.com/ Instead of updating the package list manually, use lcitool vars file. Since v2: - Commit generated json (Daniel) Since v1: - Addressed Erik & Daniel comments (generate in JSON) Philippe Mathieu-Daudé (4): tests/lcitool: Generate distribution packages list in JSON format tests/lcitool: Refresh generated files tests/vm: Introduce get_qemu_packages_from_lcitool_json() helper tests/vm/freebsd: Get up-to-date package list from lcitool vars file tests/docker/dockerfiles/debian-amd64.docker | 2 - tests/docker/dockerfiles/ubuntu2004.docker | 2 - tests/docker/dockerfiles/ubuntu2204.docker | 2 - tests/lcitool/refresh| 11 +++ tests/vm/basevm.py | 11 +++ tests/vm/freebsd | 42 +-- tests/vm/generated/README| 5 ++ tests/vm/generated/freebsd.json | 77 8 files changed, 106 insertions(+), 46 deletions(-) create mode 100644 tests/vm/generated/README create mode 100644 tests/vm/generated/freebsd.json -- 2.38.1