* [PATCH 1/4] ci: remove centos-steam-8 customer runner
2024-05-21 12:53 [PATCH 0/4] testing/next: purging remaining centos 8 bits Alex Bennée
@ 2024-05-21 12:53 ` Alex Bennée
2024-05-21 12:53 ` [PATCH 2/4] docs/devel: update references to centos to later version Alex Bennée
` (4 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Alex Bennée @ 2024-05-21 12:53 UTC (permalink / raw)
To: qemu-devel
Cc: Marc-André Lureau, Pavel Dovgalyuk, Beraldo Leal, qemu-s390x,
John Snow, Daniel P. Berrangé, Richard Henderson,
Paolo Bonzini, qemu-arm, Peter Maydell,
Philippe Mathieu-Daudé, Jiaxun Yang, Joel Stanley,
Wainer dos Santos Moschetta, Mark Cave-Ayland, Aurelien Jarno,
Thomas Huth, Cleber Rosa, Markus Armbruster, Laurent Vivier,
Alex Bennée
This broke since eef0bae3a7 (migration: Remove block migration) but
even after that was addressed it still fails to complete. As it will
shortly be EOL lets to remove the runner definition and the related
ansible setup bits.
We still have centos9 docker images build and test.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
docs/devel/ci-jobs.rst.inc | 7 -
.gitlab-ci.d/custom-runners.yml | 1 -
.../custom-runners/centos-stream-8-x86_64.yml | 24 ---
.../org.centos/stream/8/build-environment.yml | 82 --------
.../ci/org.centos/stream/8/x86_64/configure | 198 ------------------
.../org.centos/stream/8/x86_64/test-avocado | 65 ------
scripts/ci/org.centos/stream/README | 17 --
7 files changed, 394 deletions(-)
delete mode 100644 .gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml
delete mode 100644 scripts/ci/org.centos/stream/8/build-environment.yml
delete mode 100755 scripts/ci/org.centos/stream/8/x86_64/configure
delete mode 100755 scripts/ci/org.centos/stream/8/x86_64/test-avocado
delete mode 100644 scripts/ci/org.centos/stream/README
diff --git a/docs/devel/ci-jobs.rst.inc b/docs/devel/ci-jobs.rst.inc
index be06322279..3756bbe355 100644
--- a/docs/devel/ci-jobs.rst.inc
+++ b/docs/devel/ci-jobs.rst.inc
@@ -182,13 +182,6 @@ If you've got access to an IBM Z host that can be used as a gitlab-CI
runner, you can set this variable to enable the tests that require this
kind of host. The runner should be tagged with "s390x".
-CENTOS_STREAM_8_x86_64_RUNNER_AVAILABLE
-~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-If you've got access to a CentOS Stream 8 x86_64 host that can be
-used as a gitlab-CI runner, you can set this variable to enable the
-tests that require this kind of host. The runner should be tagged with
-both "centos_stream_8" and "x86_64".
-
CCACHE_DISABLE
~~~~~~~~~~~~~~
The jobs are configured to use "ccache" by default since this typically
diff --git a/.gitlab-ci.d/custom-runners.yml b/.gitlab-ci.d/custom-runners.yml
index 29e52df283..1aa3c60efe 100644
--- a/.gitlab-ci.d/custom-runners.yml
+++ b/.gitlab-ci.d/custom-runners.yml
@@ -32,4 +32,3 @@ include:
- local: '/.gitlab-ci.d/custom-runners/ubuntu-22.04-s390x.yml'
- local: '/.gitlab-ci.d/custom-runners/ubuntu-22.04-aarch64.yml'
- local: '/.gitlab-ci.d/custom-runners/ubuntu-22.04-aarch32.yml'
- - local: '/.gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml'
diff --git a/.gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml b/.gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml
deleted file mode 100644
index 367424db78..0000000000
--- a/.gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml
+++ /dev/null
@@ -1,24 +0,0 @@
-# All centos-stream-8 jobs should run successfully in an environment
-# setup by the scripts/ci/setup/stream/8/build-environment.yml task
-# "Installation of extra packages to build QEMU"
-
-centos-stream-8-x86_64:
- extends: .custom_runner_template
- allow_failure: true
- needs: []
- stage: build
- tags:
- - centos_stream_8
- - x86_64
- rules:
- - if: '$CI_PROJECT_NAMESPACE == "qemu-project" && $CI_COMMIT_BRANCH =~ /^staging/'
- - if: "$CENTOS_STREAM_8_x86_64_RUNNER_AVAILABLE"
- before_script:
- - JOBS=$(expr $(nproc) + 1)
- script:
- - mkdir build
- - cd build
- - ../scripts/ci/org.centos/stream/8/x86_64/configure
- || { cat config.log meson-logs/meson-log.txt; exit 1; }
- - make -j"$JOBS"
- - make NINJA=":" check check-avocado
diff --git a/scripts/ci/org.centos/stream/8/build-environment.yml b/scripts/ci/org.centos/stream/8/build-environment.yml
deleted file mode 100644
index 1ead77e2cb..0000000000
--- a/scripts/ci/org.centos/stream/8/build-environment.yml
+++ /dev/null
@@ -1,82 +0,0 @@
----
-- name: Installation of extra packages to build QEMU
- hosts: all
- tasks:
- - name: Extra check for CentOS Stream 8
- lineinfile:
- path: /etc/redhat-release
- line: CentOS Stream release 8
- state: present
- check_mode: yes
- register: centos_stream_8
-
- - name: Enable EPEL repo on CentOS Stream 8
- dnf:
- name:
- - epel-release
- state: present
- when:
- - centos_stream_8
-
- - name: Enable PowerTools repo on CentOS Stream 8
- ini_file:
- path: /etc/yum.repos.d/CentOS-Stream-PowerTools.repo
- section: powertools
- option: enabled
- value: "1"
- when:
- - centos_stream_8
-
- - name: Install basic packages to build QEMU on CentOS Stream 8
- dnf:
- name:
- - bzip2
- - bzip2-devel
- - capstone-devel
- - dbus-daemon
- - device-mapper-multipath-devel
- - diffutils
- - gcc
- - gcc-c++
- - genisoimage
- - gettext
- - git
- - glib2-devel
- - glusterfs-api-devel
- - gnutls-devel
- - libaio-devel
- - libcap-ng-devel
- - libcurl-devel
- - libepoxy-devel
- - libfdt-devel
- - libgcrypt-devel
- - libiscsi-devel
- - libpmem-devel
- - librados-devel
- - librbd-devel
- - libseccomp-devel
- - libslirp-devel
- - libssh-devel
- - libxkbcommon-devel
- - lzo-devel
- - make
- - mesa-libEGL-devel
- - nettle-devel
- - ninja-build
- - nmap-ncat
- - numactl-devel
- - pixman-devel
- - python38
- - python3-sphinx
- - rdma-core-devel
- - redhat-rpm-config
- - snappy-devel
- - spice-glib-devel
- - spice-server-devel
- - systemd-devel
- - systemtap-sdt-devel
- - tar
- - zlib-devel
- state: present
- when:
- - centos_stream_8
diff --git a/scripts/ci/org.centos/stream/8/x86_64/configure b/scripts/ci/org.centos/stream/8/x86_64/configure
deleted file mode 100755
index 868db665f6..0000000000
--- a/scripts/ci/org.centos/stream/8/x86_64/configure
+++ /dev/null
@@ -1,198 +0,0 @@
-#!/bin/sh -e
-#
-# Configuration for QEMU based on CentOS Stream 8 x86_64 builds
-#
-# The "configure" command line is based on:
-#
-# https://git.centos.org/rpms/qemu-kvm/blob/c8s-stream-rhel/f/SPECS/qemu-kvm.spec
-#
-# But, because the SPEC file contains a number of conditionals and
-# variable and expansions only available at RPM build time, this version
-# was initially generated from an actual RPM build on an x86_64 platform.
-#
-# From that initial version, options that are required or are a
-# consequence of non-upstream patches have been adapted. One example
-# is "--without-default-devices" which is *not* present here, given
-# that patches adding downstream specific devices are not available.
-#
-../configure \
---python=/usr/bin/python3.8 \
---prefix="/usr" \
---libdir="/usr/lib64" \
---datadir="/usr/share" \
---sysconfdir="/etc" \
---interp-prefix=/usr/qemu-%M \
---localstatedir="/var" \
---docdir="/usr/share/doc" \
---libexecdir="/usr/libexec" \
---extra-ldflags="-Wl,--build-id -Wl,-z,relro -Wl,-z,now" \
---extra-cflags="-O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fexceptions -fstack-protector-strong -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection" \
---with-suffix="qemu-kvm" \
---firmwarepath=/usr/share/qemu-firmware \
---target-list="x86_64-softmmu" \
---block-drv-rw-whitelist="qcow2,raw,file,host_device,nbd,iscsi,rbd,blkdebug,luks,null-co,nvme,copy-on-read,throttle,gluster" \
---audio-drv-list="" \
---block-drv-ro-whitelist="vmdk,vhdx,vpc,https,ssh" \
---with-coroutine=ucontext \
---tls-priority=@QEMU,SYSTEM \
---disable-af-xdp \
---disable-attr \
---disable-auth-pam \
---disable-avx2 \
---disable-avx512f \
---disable-bochs \
---disable-bpf \
---disable-brlapi \
---disable-bsd-user \
---disable-bzip2 \
---disable-cap-ng \
---disable-capstone \
---disable-cfi \
---disable-cfi-debug \
---disable-cloop \
---disable-cocoa \
---disable-coroutine-pool \
---disable-crypto-afalg \
---disable-curl \
---disable-curses \
---disable-debug-info \
---disable-debug-mutex \
---disable-debug-tcg \
---disable-dmg \
---disable-docs \
---disable-fuse \
---disable-fuse-lseek \
---disable-gcrypt \
---disable-gio \
---disable-glusterfs \
---disable-gnutls \
---disable-gtk \
---disable-guest-agent \
---disable-guest-agent-msi \
---disable-hvf \
---disable-iconv \
---disable-kvm \
---disable-libdaxctl \
---disable-libiscsi \
---disable-libnfs \
---disable-libpmem \
---disable-libssh \
---disable-libudev \
---disable-libusb \
---disable-linux-aio \
---disable-linux-io-uring \
---disable-linux-user \
---disable-live-block-migration \
---disable-lto \
---disable-lzfse \
---disable-lzo \
---disable-malloc-trim \
---disable-membarrier \
---disable-modules \
---disable-module-upgrades \
---disable-mpath \
---disable-multiprocess \
---disable-netmap \
---disable-nettle \
---disable-numa \
---disable-nvmm \
---disable-opengl \
---disable-parallels \
---disable-pie \
---disable-qcow1 \
---disable-qed \
---disable-qom-cast-debug \
---disable-rbd \
---disable-rdma \
---disable-replication \
---disable-rng-none \
---disable-safe-stack \
---disable-sanitizers \
---disable-sdl \
---disable-sdl-image \
---disable-seccomp \
---disable-slirp-smbd \
---disable-smartcard \
---disable-snappy \
---disable-sparse \
---disable-spice \
---disable-strip \
---disable-system \
---disable-tcg \
---disable-tools \
---disable-tpm \
---disable-u2f \
---disable-usb-redir \
---disable-user \
---disable-vde \
---disable-vdi \
---disable-vhost-crypto \
---disable-vhost-kernel \
---disable-vhost-net \
---disable-vhost-user \
---disable-vhost-user-blk-server \
---disable-vhost-vdpa \
---disable-virglrenderer \
---disable-virtfs \
---disable-vnc \
---disable-vnc-jpeg \
---disable-png \
---disable-vnc-sasl \
---disable-vte \
---disable-vvfat \
---disable-werror \
---disable-whpx \
---disable-xen \
---disable-xen-pci-passthrough \
---disable-xkbcommon \
---disable-zstd \
---enable-attr \
---enable-avx2 \
---enable-cap-ng \
---enable-capstone \
---enable-coroutine-pool \
---enable-curl \
---enable-debug-info \
---enable-docs \
---enable-fdt \
---enable-gcrypt \
---enable-glusterfs \
---enable-gnutls \
---enable-guest-agent \
---enable-iconv \
---enable-kvm \
---enable-libiscsi \
---enable-libpmem \
---enable-libssh \
---enable-libusb \
---enable-libudev \
---enable-linux-aio \
---enable-lzo \
---enable-malloc-trim \
---enable-modules \
---enable-mpath \
---enable-numa \
---enable-opengl \
---enable-pie \
---enable-rbd \
---enable-rdma \
---enable-seccomp \
---enable-snappy \
---enable-smartcard \
---enable-spice \
---enable-system \
---enable-tcg \
---enable-tools \
---enable-tpm \
---enable-trace-backends=dtrace \
---enable-usb-redir \
---enable-vhost-kernel \
---enable-vhost-net \
---enable-vhost-user \
---enable-vhost-user-blk-server \
---enable-vhost-vdpa \
---enable-vnc \
---enable-png \
---enable-vnc-sasl \
---enable-werror \
---enable-xkbcommon
diff --git a/scripts/ci/org.centos/stream/8/x86_64/test-avocado b/scripts/ci/org.centos/stream/8/x86_64/test-avocado
deleted file mode 100755
index 73e7a1a312..0000000000
--- a/scripts/ci/org.centos/stream/8/x86_64/test-avocado
+++ /dev/null
@@ -1,65 +0,0 @@
-#!/bin/sh -e
-#
-# Runs a previously vetted list of tests, either marked explicitly for
-# KVM and x86_64, or tests that are generic enough to be valid for all
-# targets. Such a test list can be generated with:
-#
-# ./pyvenv/bin/avocado list --filter-by-tags-include-empty \
-# --filter-by-tags-include-empty-key -t accel:kvm,arch:x86_64 \
-# tests/avocado/
-#
-# This is almost the complete list of avocado based tests available at
-# the time this was compile, with the following exceptions:
-#
-# * Require machine type "x-remote":
-# - tests/avocado/multiprocess.py:Multiprocess.test_multiprocess_x86_64
-#
-# * Requires display type "egl-headless":
-# - tests/avocado/virtio-gpu.py:VirtioGPUx86.test_virtio_vga_virgl
-# - tests/avocado/virtio-gpu.py:VirtioGPUx86.test_vhost_user_vga_virgl
-#
-# * Test is marked (unconditionally) to be skipped:
-# - tests/avocado/virtio_check_params.py:VirtioMaxSegSettingsCheck.test_machine_types
-#
-make get-vm-images
-./pyvenv/bin/avocado run \
- --job-results-dir=tests/results/ \
- tests/avocado/boot_linux.py:BootLinuxX8664.test_pc_i440fx_kvm \
- tests/avocado/boot_linux.py:BootLinuxX8664.test_pc_q35_kvm \
- tests/avocado/boot_linux_console.py:BootLinuxConsole.test_x86_64_pc \
- tests/avocado/cpu_queries.py:QueryCPUModelExpansion.test \
- tests/avocado/empty_cpu_model.py:EmptyCPUModel.test \
- tests/avocado/hotplug_cpu.py:HotPlugCPU.test \
- tests/avocado/netdev-ethtool.py:NetDevEthtool.test_igb \
- tests/avocado/netdev-ethtool.py:NetDevEthtool.test_igb_nomsi \
- tests/avocado/info_usernet.py:InfoUsernet.test_hostfwd \
- tests/avocado/intel_iommu.py:IntelIOMMU.test_intel_iommu \
- tests/avocado/intel_iommu.py:IntelIOMMU.test_intel_iommu_pt \
- tests/avocado/intel_iommu.py:IntelIOMMU.test_intel_iommu_strict \
- tests/avocado/intel_iommu.py:IntelIOMMU.test_intel_iommu_strict_cm \
- tests/avocado/linux_initrd.py:LinuxInitrd.test_with_2gib_file_should_exit_error_msg_with_linux_v3_6 \
- tests/avocado/linux_initrd.py:LinuxInitrd.test_with_2gib_file_should_work_with_linux_v4_16 \
- tests/avocado/migration.py:Migration.test_migration_with_exec \
- tests/avocado/migration.py:Migration.test_migration_with_tcp_localhost \
- tests/avocado/migration.py:Migration.test_migration_with_unix \
- tests/avocado/pc_cpu_hotplug_props.py:OmittedCPUProps.test_no_die_id \
- tests/avocado/replay_kernel.py:ReplayKernelNormal.test_x86_64_pc \
- tests/avocado/reverse_debugging.py:ReverseDebugging_X86_64.test_x86_64_pc \
- tests/avocado/version.py:Version.test_qmp_human_info_version \
- tests/avocado/virtio_version.py:VirtioVersionCheck.test_conventional_devs \
- tests/avocado/virtio_version.py:VirtioVersionCheck.test_modern_only_devs \
- tests/avocado/vnc.py:Vnc.test_change_password \
- tests/avocado/vnc.py:Vnc.test_change_password_requires_a_password \
- tests/avocado/vnc.py:Vnc.test_no_vnc \
- tests/avocado/vnc.py:Vnc.test_no_vnc_change_password \
- tests/avocado/x86_cpu_model_versions.py:CascadelakeArchCapabilities.test_4_0 \
- tests/avocado/x86_cpu_model_versions.py:CascadelakeArchCapabilities.test_4_1 \
- tests/avocado/x86_cpu_model_versions.py:CascadelakeArchCapabilities.test_set_4_0 \
- tests/avocado/x86_cpu_model_versions.py:CascadelakeArchCapabilities.test_unset_4_1 \
- tests/avocado/x86_cpu_model_versions.py:CascadelakeArchCapabilities.test_v1_4_0 \
- tests/avocado/x86_cpu_model_versions.py:CascadelakeArchCapabilities.test_v1_set_4_0 \
- tests/avocado/x86_cpu_model_versions.py:CascadelakeArchCapabilities.test_v2_4_0 \
- tests/avocado/x86_cpu_model_versions.py:CascadelakeArchCapabilities.test_v2_unset_4_1 \
- tests/avocado/x86_cpu_model_versions.py:X86CPUModelAliases.test_4_0_alias_compatibility \
- tests/avocado/x86_cpu_model_versions.py:X86CPUModelAliases.test_4_1_alias \
- tests/avocado/x86_cpu_model_versions.py:X86CPUModelAliases.test_none_alias
diff --git a/scripts/ci/org.centos/stream/README b/scripts/ci/org.centos/stream/README
deleted file mode 100644
index e3eadfe3ea..0000000000
--- a/scripts/ci/org.centos/stream/README
+++ /dev/null
@@ -1,17 +0,0 @@
-This directory contains scripts for generating a build of QEMU that
-closely matches the CentOS Stream[1] builds of the qemu-kvm package.
-
-To have the environment ready to configure, build QEMU and run tests,
-please start with a CentOS Stream machine and:
-
- * apply the generic "build-environment.yml" playbook located at
- scripts/ci/setup
-
- * apply the "build-environment.yml" in the directory following the
- CentOS Stream version (such as "8").
-
-This currently only covers CentOS Stream 8 environments and
-packages[2].
-
-[1] https://www.centos.org/centos-stream/
-[2] https://git.centos.org/rpms/qemu-kvm/commits/c8s-stream-rhel
--
2.39.2
^ permalink raw reply related [flat|nested] 11+ messages in thread* [PATCH 2/4] docs/devel: update references to centos to later version
2024-05-21 12:53 [PATCH 0/4] testing/next: purging remaining centos 8 bits Alex Bennée
2024-05-21 12:53 ` [PATCH 1/4] ci: remove centos-steam-8 customer runner Alex Bennée
@ 2024-05-21 12:53 ` Alex Bennée
2024-05-28 6:46 ` Thomas Huth
2024-05-21 12:53 ` [PATCH 3/4] tests/vm: update centos.aarch64 image to 9 Alex Bennée
` (3 subsequent siblings)
5 siblings, 1 reply; 11+ messages in thread
From: Alex Bennée @ 2024-05-21 12:53 UTC (permalink / raw)
To: qemu-devel
Cc: Marc-André Lureau, Pavel Dovgalyuk, Beraldo Leal, qemu-s390x,
John Snow, Daniel P. Berrangé, Richard Henderson,
Paolo Bonzini, qemu-arm, Peter Maydell,
Philippe Mathieu-Daudé, Jiaxun Yang, Joel Stanley,
Wainer dos Santos Moschetta, Mark Cave-Ayland, Aurelien Jarno,
Thomas Huth, Cleber Rosa, Markus Armbruster, Laurent Vivier,
Alex Bennée
From the website:
"After May 31, 2024, CentOS Stream 8 will be archived and no further
updates will be provided."
We have updated a few bits but there are still references that need
fixing.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
docs/devel/testing.rst | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
index fa28e3ecb2..c312465fa7 100644
--- a/docs/devel/testing.rst
+++ b/docs/devel/testing.rst
@@ -387,9 +387,9 @@ make target):
.. code::
- make docker-test-build@centos8
+ make docker-test-build@centos9
-This will create a container instance using the ``centos8`` image (the image
+This will create a container instance using the ``centos9`` image (the image
is downloaded and initialized automatically), in which the ``test-build`` job
is executed.
@@ -410,8 +410,8 @@ locally by using the ``NOCACHE`` build option:
Images
~~~~~~
-Along with many other images, the ``centos8`` image is defined in a Dockerfile
-in ``tests/docker/dockerfiles/``, called ``centos8.docker``. ``make docker-help``
+Along with many other images, the ``centos9`` image is defined in a Dockerfile
+in ``tests/docker/dockerfiles/``, called ``centos9.docker``. ``make docker-help``
command will list all the available images.
A ``.pre`` script can be added beside the ``.docker`` file, which will be
--
2.39.2
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH 2/4] docs/devel: update references to centos to later version
2024-05-21 12:53 ` [PATCH 2/4] docs/devel: update references to centos to later version Alex Bennée
@ 2024-05-28 6:46 ` Thomas Huth
0 siblings, 0 replies; 11+ messages in thread
From: Thomas Huth @ 2024-05-28 6:46 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: Marc-André Lureau, Pavel Dovgalyuk, Beraldo Leal, qemu-s390x,
John Snow, Daniel P. Berrangé, Richard Henderson,
Paolo Bonzini, qemu-arm, Peter Maydell,
Philippe Mathieu-Daudé, Jiaxun Yang, Joel Stanley,
Wainer dos Santos Moschetta, Mark Cave-Ayland, Aurelien Jarno,
Cleber Rosa, Markus Armbruster, Laurent Vivier
On 21/05/2024 14.53, Alex Bennée wrote:
> From the website:
>
> "After May 31, 2024, CentOS Stream 8 will be archived and no further
> updates will be provided."
>
> We have updated a few bits but there are still references that need
> fixing.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> docs/devel/testing.rst | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
> index fa28e3ecb2..c312465fa7 100644
> --- a/docs/devel/testing.rst
> +++ b/docs/devel/testing.rst
> @@ -387,9 +387,9 @@ make target):
>
> .. code::
>
> - make docker-test-build@centos8
> + make docker-test-build@centos9
Basically ACK, but I wonder whether we should maybe switch the documentation
to another container here that does not have a version in its name, so we
don't have to change it again and again each time a new version is released.
For example, what about using @fedora instead of @centos9 ?
Thomas
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 3/4] tests/vm: update centos.aarch64 image to 9
2024-05-21 12:53 [PATCH 0/4] testing/next: purging remaining centos 8 bits Alex Bennée
2024-05-21 12:53 ` [PATCH 1/4] ci: remove centos-steam-8 customer runner Alex Bennée
2024-05-21 12:53 ` [PATCH 2/4] docs/devel: update references to centos to later version Alex Bennée
@ 2024-05-21 12:53 ` Alex Bennée
2024-05-28 6:48 ` Thomas Huth
2024-05-21 12:53 ` [PATCH 4/4] tests/vm: remove plain centos image Alex Bennée
` (2 subsequent siblings)
5 siblings, 1 reply; 11+ messages in thread
From: Alex Bennée @ 2024-05-21 12:53 UTC (permalink / raw)
To: qemu-devel
Cc: Marc-André Lureau, Pavel Dovgalyuk, Beraldo Leal, qemu-s390x,
John Snow, Daniel P. Berrangé, Richard Henderson,
Paolo Bonzini, qemu-arm, Peter Maydell,
Philippe Mathieu-Daudé, Jiaxun Yang, Joel Stanley,
Wainer dos Santos Moschetta, Mark Cave-Ayland, Aurelien Jarno,
Thomas Huth, Cleber Rosa, Markus Armbruster, Laurent Vivier,
Alex Bennée
As Centos Stream 8 goes out of support we need to update. To do this
powertools is replaced by crb and we don't over specify the python3 we
want.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/vm/centos.aarch64 | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tests/vm/centos.aarch64 b/tests/vm/centos.aarch64
index 3f58de1e64..fcf9e08c87 100755
--- a/tests/vm/centos.aarch64
+++ b/tests/vm/centos.aarch64
@@ -25,9 +25,9 @@ DEFAULT_CONFIG = {
'cpu' : "max",
'machine' : "virt,gic-version=max",
'install_cmds' : (
- "dnf config-manager --set-enabled powertools, "
+ "dnf config-manager --enable crb, "
"dnf config-manager --add-repo=https://download.docker.com/linux/centos/docker-ce.repo, "
- "dnf install -y make ninja-build git python38 gcc gcc-c++ flex bison "\
+ "dnf install -y make ninja-build git python3 gcc gcc-c++ flex bison "\
"glib2-devel pixman-devel zlib-devel docker-ce.aarch64, "
"systemctl enable docker, "
),
@@ -38,10 +38,10 @@ DEFAULT_CONFIG = {
class CentosAarch64VM(basevm.BaseVM):
- name = "centos8.aarch64"
+ name = "centos9.aarch64"
arch = "aarch64"
- image_name = "CentOS-Stream-GenericCloud-8-20220125.1.aarch64.qcow2"
- image_link = "https://cloud.centos.org/centos/8-stream/aarch64/images/"
+ image_name = "CentOS-Stream-GenericCloud-9-20230501.0.aarch64.qcow2"
+ image_link = "https://cloud.centos.org/centos/9-stream/aarch64/images/"
image_link += image_name
BUILD_SCRIPT = """
set -e;
--
2.39.2
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH 3/4] tests/vm: update centos.aarch64 image to 9
2024-05-21 12:53 ` [PATCH 3/4] tests/vm: update centos.aarch64 image to 9 Alex Bennée
@ 2024-05-28 6:48 ` Thomas Huth
0 siblings, 0 replies; 11+ messages in thread
From: Thomas Huth @ 2024-05-28 6:48 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: Marc-André Lureau, Pavel Dovgalyuk, Beraldo Leal, qemu-s390x,
John Snow, Daniel P. Berrangé, Richard Henderson,
Paolo Bonzini, qemu-arm, Peter Maydell,
Philippe Mathieu-Daudé, Jiaxun Yang, Joel Stanley,
Wainer dos Santos Moschetta, Mark Cave-Ayland, Aurelien Jarno,
Cleber Rosa, Markus Armbruster, Laurent Vivier
On 21/05/2024 14.53, Alex Bennée wrote:
> As Centos Stream 8 goes out of support we need to update. To do this
> powertools is replaced by crb and we don't over specify the python3 we
> want.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> tests/vm/centos.aarch64 | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
Reviewed-by: Thomas Huth <thuth@redhat.com>
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 4/4] tests/vm: remove plain centos image
2024-05-21 12:53 [PATCH 0/4] testing/next: purging remaining centos 8 bits Alex Bennée
` (2 preceding siblings ...)
2024-05-21 12:53 ` [PATCH 3/4] tests/vm: update centos.aarch64 image to 9 Alex Bennée
@ 2024-05-21 12:53 ` Alex Bennée
2024-05-21 13:11 ` Thomas Huth
2024-05-21 14:57 ` [PATCH 0/4] testing/next: purging remaining centos 8 bits Richard Henderson
2024-05-27 9:32 ` Philippe Mathieu-Daudé
5 siblings, 1 reply; 11+ messages in thread
From: Alex Bennée @ 2024-05-21 12:53 UTC (permalink / raw)
To: qemu-devel
Cc: Marc-André Lureau, Pavel Dovgalyuk, Beraldo Leal, qemu-s390x,
John Snow, Daniel P. Berrangé, Richard Henderson,
Paolo Bonzini, qemu-arm, Peter Maydell,
Philippe Mathieu-Daudé, Jiaxun Yang, Joel Stanley,
Wainer dos Santos Moschetta, Mark Cave-Ayland, Aurelien Jarno,
Thomas Huth, Cleber Rosa, Markus Armbruster, Laurent Vivier,
Alex Bennée
This isn't really used and we have lighter weight docker containers
for testing this stuff directly.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/vm/Makefile.include | 1 -
tests/vm/centos | 51 ---------------------------------------
2 files changed, 52 deletions(-)
delete mode 100755 tests/vm/centos
diff --git a/tests/vm/Makefile.include b/tests/vm/Makefile.include
index ac56824a87..13ed80f72d 100644
--- a/tests/vm/Makefile.include
+++ b/tests/vm/Makefile.include
@@ -45,7 +45,6 @@ vm-help vm-test:
@echo " vm-build-netbsd - Build QEMU in NetBSD VM"
@echo " vm-build-openbsd - Build QEMU in OpenBSD VM"
ifneq ($(GENISOIMAGE),)
- @echo " vm-build-centos - Build QEMU in CentOS VM, with Docker"
ifneq ($(EFI_AARCH64),)
@echo " vm-build-ubuntu.aarch64 - Build QEMU in ubuntu aarch64 VM"
@echo " vm-build-centos.aarch64 - Build QEMU in CentOS aarch64 VM"
diff --git a/tests/vm/centos b/tests/vm/centos
deleted file mode 100755
index d25c8f8b5b..0000000000
--- a/tests/vm/centos
+++ /dev/null
@@ -1,51 +0,0 @@
-#!/usr/bin/env python3
-#
-# CentOS 8 Stream image
-#
-# Copyright 2018, 2022 Red Hat Inc.
-#
-# Authors:
-# Fam Zheng <famz@redhat.com>
-#
-# This code is licensed under the GPL version 2 or later. See
-# the COPYING file in the top-level directory.
-#
-
-import os
-import sys
-import subprocess
-import basevm
-import time
-
-class CentosVM(basevm.BaseVM):
- name = "centos"
- arch = "x86_64"
- BUILD_SCRIPT = """
- set -e;
- cd $(mktemp -d);
- export SRC_ARCHIVE=/dev/vdb;
- sudo chmod a+r $SRC_ARCHIVE;
- tar -xf $SRC_ARCHIVE;
- make docker-test-block@centos9 {verbose} J={jobs} NETWORK=1;
- make docker-test-quick@centos9 {verbose} J={jobs} NETWORK=1;
- """
-
- def build_image(self, img):
- cimg = self._download_with_cache("https://cloud.centos.org/centos/8-stream/x86_64/images/CentOS-Stream-GenericCloud-8-20220125.1.x86_64.qcow2")
- img_tmp = img + ".tmp"
- subprocess.check_call(['cp', '-f', cimg, img_tmp])
- self.exec_qemu_img("resize", img_tmp, "50G")
- self.boot(img_tmp, extra_args = ["-cdrom", self.gen_cloud_init_iso()])
- self.wait_ssh()
- self.ssh_root_check("touch /etc/cloud/cloud-init.disabled")
- self.ssh_root_check("dnf update -y")
- self.ssh_root_check("dnf install -y dnf-plugins-core")
- self.ssh_root_check("dnf config-manager --set-enabled powertools")
- self.ssh_root_check("dnf install -y podman make ninja-build git python3")
- self.ssh_root("poweroff")
- self.wait()
- os.rename(img_tmp, img)
- return 0
-
-if __name__ == "__main__":
- sys.exit(basevm.main(CentosVM))
--
2.39.2
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH 4/4] tests/vm: remove plain centos image
2024-05-21 12:53 ` [PATCH 4/4] tests/vm: remove plain centos image Alex Bennée
@ 2024-05-21 13:11 ` Thomas Huth
0 siblings, 0 replies; 11+ messages in thread
From: Thomas Huth @ 2024-05-21 13:11 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: Marc-André Lureau, Pavel Dovgalyuk, Beraldo Leal, qemu-s390x,
John Snow, Daniel P. Berrangé, Richard Henderson,
Paolo Bonzini, qemu-arm, Peter Maydell,
Philippe Mathieu-Daudé, Jiaxun Yang, Joel Stanley,
Wainer dos Santos Moschetta, Mark Cave-Ayland, Aurelien Jarno,
Cleber Rosa, Markus Armbruster, Laurent Vivier
On 21/05/2024 14.53, Alex Bennée wrote:
> This isn't really used and we have lighter weight docker containers
> for testing this stuff directly.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> tests/vm/Makefile.include | 1 -
> tests/vm/centos | 51 ---------------------------------------
> 2 files changed, 52 deletions(-)
> delete mode 100755 tests/vm/centos
Reviewed-by: Thomas Huth <thuth@redhat.com>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 0/4] testing/next: purging remaining centos 8 bits
2024-05-21 12:53 [PATCH 0/4] testing/next: purging remaining centos 8 bits Alex Bennée
` (3 preceding siblings ...)
2024-05-21 12:53 ` [PATCH 4/4] tests/vm: remove plain centos image Alex Bennée
@ 2024-05-21 14:57 ` Richard Henderson
2024-05-27 9:32 ` Philippe Mathieu-Daudé
5 siblings, 0 replies; 11+ messages in thread
From: Richard Henderson @ 2024-05-21 14:57 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
On 5/21/24 05:53, Alex Bennée wrote:
> Alex Bennée (4):
> ci: remove centos-steam-8 customer runner
> docs/devel: update references to centos to later version
> tests/vm: update centos.aarch64 image to 9
> tests/vm: remove plain centos image
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH 0/4] testing/next: purging remaining centos 8 bits
2024-05-21 12:53 [PATCH 0/4] testing/next: purging remaining centos 8 bits Alex Bennée
` (4 preceding siblings ...)
2024-05-21 14:57 ` [PATCH 0/4] testing/next: purging remaining centos 8 bits Richard Henderson
@ 2024-05-27 9:32 ` Philippe Mathieu-Daudé
2024-05-28 12:51 ` Alex Bennée
5 siblings, 1 reply; 11+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-05-27 9:32 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: Marc-André Lureau, Pavel Dovgalyuk, Beraldo Leal, qemu-s390x,
John Snow, Daniel P. Berrangé, Richard Henderson,
Paolo Bonzini, qemu-arm, Peter Maydell, Jiaxun Yang, Joel Stanley,
Wainer dos Santos Moschetta, Mark Cave-Ayland, Aurelien Jarno,
Thomas Huth, Cleber Rosa, Markus Armbruster, Laurent Vivier
Hi Alex,
On 21/5/24 14:53, Alex Bennée wrote:
> There are a few more bits referencing centos8 in the tree which needed
> cleaning up. After this we can remove the dedicated runner from the
> gitlab registration. If we want to keep a dedicated Centos runner then
> we can add back the bits needed to set it up (although arguably we
> could have a single build-environment setup that handles all distros
> and integrates with lcitool).
Do you you mean we should generate
scripts/ci/setup/build-environment.yml with lcitool?
Otherwise should we update it?
>
> Alex.
>
> Alex Bennée (4):
> ci: remove centos-steam-8 customer runner
> docs/devel: update references to centos to later version
> tests/vm: update centos.aarch64 image to 9
> tests/vm: remove plain centos image
>
> docs/devel/ci-jobs.rst.inc | 7 -
> docs/devel/testing.rst | 8 +-
> .gitlab-ci.d/custom-runners.yml | 1 -
> .../custom-runners/centos-stream-8-x86_64.yml | 24 ---
> .../org.centos/stream/8/build-environment.yml | 82 --------
> .../ci/org.centos/stream/8/x86_64/configure | 198 ------------------
> .../org.centos/stream/8/x86_64/test-avocado | 65 ------
> scripts/ci/org.centos/stream/README | 17 --
> tests/vm/Makefile.include | 1 -
> tests/vm/centos | 51 -----
> tests/vm/centos.aarch64 | 10 +-
> 11 files changed, 9 insertions(+), 455 deletions(-)
> delete mode 100644 .gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml
> delete mode 100644 scripts/ci/org.centos/stream/8/build-environment.yml
> delete mode 100755 scripts/ci/org.centos/stream/8/x86_64/configure
> delete mode 100755 scripts/ci/org.centos/stream/8/x86_64/test-avocado
> delete mode 100644 scripts/ci/org.centos/stream/README
> delete mode 100755 tests/vm/centos
>
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH 0/4] testing/next: purging remaining centos 8 bits
2024-05-27 9:32 ` Philippe Mathieu-Daudé
@ 2024-05-28 12:51 ` Alex Bennée
0 siblings, 0 replies; 11+ messages in thread
From: Alex Bennée @ 2024-05-28 12:51 UTC (permalink / raw)
To: Philippe Mathieu-Daudé
Cc: qemu-devel, Marc-André Lureau, Pavel Dovgalyuk, Beraldo Leal,
qemu-s390x, John Snow, Daniel P. Berrangé, Richard Henderson,
Paolo Bonzini, qemu-arm, Peter Maydell, Jiaxun Yang, Joel Stanley,
Wainer dos Santos Moschetta, Mark Cave-Ayland, Aurelien Jarno,
Thomas Huth, Cleber Rosa, Markus Armbruster, Laurent Vivier
Philippe Mathieu-Daudé <philmd@linaro.org> writes:
> Hi Alex,
>
> On 21/5/24 14:53, Alex Bennée wrote:
>> There are a few more bits referencing centos8 in the tree which needed
>> cleaning up. After this we can remove the dedicated runner from the
>> gitlab registration. If we want to keep a dedicated Centos runner then
>> we can add back the bits needed to set it up (although arguably we
>> could have a single build-environment setup that handles all distros
>> and integrates with lcitool).
>
> Do you you mean we should generate
> scripts/ci/setup/build-environment.yml with lcitool?
> Otherwise should we update it?
Yes - I've got some WIP patches which I'll post soon.
>
>> Alex.
>> Alex Bennée (4):
>> ci: remove centos-steam-8 customer runner
>> docs/devel: update references to centos to later version
>> tests/vm: update centos.aarch64 image to 9
>> tests/vm: remove plain centos image
>> docs/devel/ci-jobs.rst.inc | 7 -
>> docs/devel/testing.rst | 8 +-
>> .gitlab-ci.d/custom-runners.yml | 1 -
>> .../custom-runners/centos-stream-8-x86_64.yml | 24 ---
>> .../org.centos/stream/8/build-environment.yml | 82 --------
>> .../ci/org.centos/stream/8/x86_64/configure | 198 ------------------
>> .../org.centos/stream/8/x86_64/test-avocado | 65 ------
>> scripts/ci/org.centos/stream/README | 17 --
>> tests/vm/Makefile.include | 1 -
>> tests/vm/centos | 51 -----
>> tests/vm/centos.aarch64 | 10 +-
>> 11 files changed, 9 insertions(+), 455 deletions(-)
>> delete mode 100644 .gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml
>> delete mode 100644 scripts/ci/org.centos/stream/8/build-environment.yml
>> delete mode 100755 scripts/ci/org.centos/stream/8/x86_64/configure
>> delete mode 100755 scripts/ci/org.centos/stream/8/x86_64/test-avocado
>> delete mode 100644 scripts/ci/org.centos/stream/README
>> delete mode 100755 tests/vm/centos
>>
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
^ permalink raw reply [flat|nested] 11+ messages in thread