.gitlab-ci.d/buildtest.yml | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+)
The device-crash-test script has been quite neglected in the past,
so that it bit-rot quite often. Let's add CI jobs that run this
script for at least some targets, so that this script does not
regress that easily anymore.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
.gitlab-ci.d/buildtest.yml | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
index 71d0f407ad..7e1cb0b3c2 100644
--- a/.gitlab-ci.d/buildtest.yml
+++ b/.gitlab-ci.d/buildtest.yml
@@ -100,6 +100,17 @@ avocado-system-debian:
IMAGE: debian-amd64
MAKE_CHECK_ARGS: check-avocado
+crash-test-debian:
+ extends: .native_test_job_template
+ needs:
+ - job: build-system-debian
+ artifacts: true
+ variables:
+ IMAGE: debian-amd64
+ script:
+ - cd build
+ - scripts/device-crash-test -q ./qemu-system-i386
+
build-system-fedora:
extends: .native_build_job_template
needs:
@@ -134,6 +145,18 @@ avocado-system-fedora:
IMAGE: fedora
MAKE_CHECK_ARGS: check-avocado
+crash-test-fedora:
+ extends: .native_test_job_template
+ needs:
+ - job: build-system-fedora
+ artifacts: true
+ variables:
+ IMAGE: fedora
+ script:
+ - cd build
+ - scripts/device-crash-test -q ./qemu-system-ppc
+ - scripts/device-crash-test -q ./qemu-system-riscv32
+
build-system-centos:
extends: .native_build_job_template
needs:
--
2.27.0
On Fri, Nov 26, 2021 at 05:27:24PM +0100, Thomas Huth wrote: > The device-crash-test script has been quite neglected in the past, > so that it bit-rot quite often. Let's add CI jobs that run this > script for at least some targets, so that this script does not > regress that easily anymore. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > .gitlab-ci.d/buildtest.yml | 23 +++++++++++++++++++++++ > 1 file changed, 23 insertions(+) > > diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml > index 71d0f407ad..7e1cb0b3c2 100644 > --- a/.gitlab-ci.d/buildtest.yml > +++ b/.gitlab-ci.d/buildtest.yml > @@ -100,6 +100,17 @@ avocado-system-debian: > IMAGE: debian-amd64 > MAKE_CHECK_ARGS: check-avocado > > +crash-test-debian: > + extends: .native_test_job_template > + needs: > + - job: build-system-debian > + artifacts: true > + variables: > + IMAGE: debian-amd64 > + script: > + - cd build > + - scripts/device-crash-test -q ./qemu-system-i386 > + > build-system-fedora: > extends: .native_build_job_template > needs: > @@ -134,6 +145,18 @@ avocado-system-fedora: > IMAGE: fedora > MAKE_CHECK_ARGS: check-avocado > > +crash-test-fedora: > + extends: .native_test_job_template > + needs: > + - job: build-system-fedora > + artifacts: true > + variables: > + IMAGE: fedora > + script: > + - cd build > + - scripts/device-crash-test -q ./qemu-system-ppc > + - scripts/device-crash-test -q ./qemu-system-riscv32 I'm curious why you picked all the 32-bit architecture targets to test this for, since 95% of usage these days will be on the 64-bit targets ? I guess both 32-bit & 64-bit targets probably have the same set of devices built mostly, but still wondering if there was a reason for your choice. 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 :|
On 29/11/2021 11.07, Daniel P. Berrangé wrote: > On Fri, Nov 26, 2021 at 05:27:24PM +0100, Thomas Huth wrote: >> The device-crash-test script has been quite neglected in the past, >> so that it bit-rot quite often. Let's add CI jobs that run this >> script for at least some targets, so that this script does not >> regress that easily anymore. >> >> Signed-off-by: Thomas Huth <thuth@redhat.com> >> --- >> .gitlab-ci.d/buildtest.yml | 23 +++++++++++++++++++++++ >> 1 file changed, 23 insertions(+) >> >> diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml >> index 71d0f407ad..7e1cb0b3c2 100644 >> --- a/.gitlab-ci.d/buildtest.yml >> +++ b/.gitlab-ci.d/buildtest.yml >> @@ -100,6 +100,17 @@ avocado-system-debian: >> IMAGE: debian-amd64 >> MAKE_CHECK_ARGS: check-avocado >> >> +crash-test-debian: >> + extends: .native_test_job_template >> + needs: >> + - job: build-system-debian >> + artifacts: true >> + variables: >> + IMAGE: debian-amd64 >> + script: >> + - cd build >> + - scripts/device-crash-test -q ./qemu-system-i386 >> + >> build-system-fedora: >> extends: .native_build_job_template >> needs: >> @@ -134,6 +145,18 @@ avocado-system-fedora: >> IMAGE: fedora >> MAKE_CHECK_ARGS: check-avocado >> >> +crash-test-fedora: >> + extends: .native_test_job_template >> + needs: >> + - job: build-system-fedora >> + artifacts: true >> + variables: >> + IMAGE: fedora >> + script: >> + - cd build >> + - scripts/device-crash-test -q ./qemu-system-ppc >> + - scripts/device-crash-test -q ./qemu-system-riscv32 > > I'm curious why you picked all the 32-bit architecture targets to test > this for, since 95% of usage these days will be on the 64-bit targets ? > > I guess both 32-bit & 64-bit targets probably have the same set of > devices built mostly, but still wondering if there was a reason for > your choice. There haven't been any firm reasons, it's mostly simply because the distros that I picked (Fedora and Debian) only offer the 32-bit versions here. I used those two distros since I wanted to pick a distro with a recent Python, and one with an older version of Python, so I first wanted to go with Fedora and CentOS 8. But then I remembered that CentOS 8 will soon be EOL, and we haven't decided what to do here yet (I guess we should switch to CentOS Stream?), so I used Debian instead. Thomas
On 11/26/21 17:27, Thomas Huth wrote: > The device-crash-test script has been quite neglected in the past, > so that it bit-rot quite often. Let's add CI jobs that run this > script for at least some targets, so that this script does not > regress that easily anymore. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > .gitlab-ci.d/buildtest.yml | 23 +++++++++++++++++++++++ > 1 file changed, 23 insertions(+) Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
© 2016 - 2024 Red Hat, Inc.