.gitlab-ci.d/container-cross.yml | 1 - 1 file changed, 1 deletion(-)
Revert "gitlab-ci: Disable the riscv64-debian-cross-container by default"
This reverts commit f51f90c65ed7706c3c4f7a889ce3d6b7ab75ef6a.
Hopefully it wont be very unstable.
Since riscv64 is now a release architecture in debian, we can try switching
to debian testing instead of debian sid. Also, debian-ports part of the
archive isn't needed anymore.
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
---
.gitlab-ci.d/container-cross.yml | 1 -
1 file changed, 1 deletion(-)
diff --git a/.gitlab-ci.d/container-cross.yml b/.gitlab-ci.d/container-cross.yml
index e3103940a0..dbffed3f21 100644
--- a/.gitlab-ci.d/container-cross.yml
+++ b/.gitlab-ci.d/container-cross.yml
@@ -77,7 +77,6 @@ riscv64-debian-cross-container:
allow_failure: true
variables:
NAME: debian-riscv64-cross
- QEMU_JOB_OPTIONAL: 1
s390x-debian-cross-container:
extends: .container_job_template
--
2.39.2
On 5/2/24 12:40, Michael Tokarev wrote: > Revert "gitlab-ci: Disable the riscv64-debian-cross-container by default" > This reverts commit f51f90c65ed7706c3c4f7a889ce3d6b7ab75ef6a. > > Hopefully it wont be very unstable. > > Since riscv64 is now a release architecture in debian, we can try switching > to debian testing instead of debian sid. Also, debian-ports part of the > archive isn't needed anymore. So... please update debian-riscv64-cross.docker. > > Signed-off-by: Michael Tokarev <mjt@tls.msk.ru> > --- > .gitlab-ci.d/container-cross.yml | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/.gitlab-ci.d/container-cross.yml b/.gitlab-ci.d/container-cross.yml > index e3103940a0..dbffed3f21 100644 > --- a/.gitlab-ci.d/container-cross.yml > +++ b/.gitlab-ci.d/container-cross.yml > @@ -77,7 +77,6 @@ riscv64-debian-cross-container: > allow_failure: true > variables: > NAME: debian-riscv64-cross > - QEMU_JOB_OPTIONAL: 1 But you can't do this until the above is done. r~
02.05.2024 23:05, Richard Henderson wrote: > On 5/2/24 12:40, Michael Tokarev wrote: >> Revert "gitlab-ci: Disable the riscv64-debian-cross-container by default" >> This reverts commit f51f90c65ed7706c3c4f7a889ce3d6b7ab75ef6a. >> >> Hopefully it wont be very unstable. >> >> Since riscv64 is now a release architecture in debian, we can try switching >> to debian testing instead of debian sid. Also, debian-ports part of the >> archive isn't needed anymore. > > So... please update debian-riscv64-cross.docker. debian-riscv64-cross.docker is generated by lcitool. It looks like lcitool has to be updated for this. >> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru> >> --- >> .gitlab-ci.d/container-cross.yml | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/.gitlab-ci.d/container-cross.yml b/.gitlab-ci.d/container-cross.yml >> index e3103940a0..dbffed3f21 100644 >> --- a/.gitlab-ci.d/container-cross.yml >> +++ b/.gitlab-ci.d/container-cross.yml >> @@ -77,7 +77,6 @@ riscv64-debian-cross-container: >> allow_failure: true >> variables: >> NAME: debian-riscv64-cross >> - QEMU_JOB_OPTIONAL: 1 > > But you can't do this until the above is done. I don't see a reason why not. Adding debian-ports archive does nothing and doesn't hurt either. Switching to testing is a different matter. This thing works now with sid/unstable, this is what this patch is about. Thanks, /mjt -- GPG Key transition (from rsa2048 to rsa4096) since 2024-04-24. New key: rsa4096/61AD3D98ECDF2C8E 9D8B E14E 3F2A 9DD7 9199 28F1 61AD 3D98 ECDF 2C8E Old key: rsa2048/457CE0A0804465C5 6EE1 95D1 886E 8FFB 810D 4324 457C E0A0 8044 65C5 Transition statement: http://www.corpit.ru/mjt/gpg-transition-2024.txt
On 02/05/2024 22.08, Michael Tokarev wrote: > 02.05.2024 23:05, Richard Henderson wrote: >> On 5/2/24 12:40, Michael Tokarev wrote: >>> Revert "gitlab-ci: Disable the riscv64-debian-cross-container by default" >>> This reverts commit f51f90c65ed7706c3c4f7a889ce3d6b7ab75ef6a. >>> >>> Hopefully it wont be very unstable. >>> >>> Since riscv64 is now a release architecture in debian, we can try switching >>> to debian testing instead of debian sid. Also, debian-ports part of the >>> archive isn't needed anymore. >> >> So... please update debian-riscv64-cross.docker. > > debian-riscv64-cross.docker is generated by lcitool. > It looks like lcitool has to be updated for this. Updating lcitool isn't too hard. Simply type "make lcitool-refresh" in your build directory - this should initialize the tests/lcitool/libvirt-ci submodule and regenerate the dockerfiles. Then you can do your change in tests/lcitool/libvirt-ci and run the refresh command again to update the dockerfiles. Once you're happy with the results, fork https://gitlab.com/libvirt/libvirt-ci on gitlab, push your changes into your fork and then open a merge request against the upstream libvirt-ci repository. >>> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru> >>> --- >>> .gitlab-ci.d/container-cross.yml | 1 - >>> 1 file changed, 1 deletion(-) >>> >>> diff --git a/.gitlab-ci.d/container-cross.yml >>> b/.gitlab-ci.d/container-cross.yml >>> index e3103940a0..dbffed3f21 100644 >>> --- a/.gitlab-ci.d/container-cross.yml >>> +++ b/.gitlab-ci.d/container-cross.yml >>> @@ -77,7 +77,6 @@ riscv64-debian-cross-container: >>> allow_failure: true >>> variables: >>> NAME: debian-riscv64-cross >>> - QEMU_JOB_OPTIONAL: 1 >> >> But you can't do this until the above is done. > > I don't see a reason why not. Adding debian-ports archive does nothing > and doesn't hurt either. > > Switching to testing is a different matter. > > This thing works now with sid/unstable, this is what this patch is about. But this doesn't match the patch description - there you talk about switching to testing and that the ports are not needed anymore, so this is confusing. I'd suggest to update at least the patch description if we continue with sid. Thomas
On 5/2/24 13:08, Michael Tokarev wrote: > This thing works now with sid/unstable, this is what this patch is about. It worked when first introduced, then failed, then worked... Sid is too unstable for our usage as a gating test. r~
02.05.2024 23:41, Richard Henderson wrote: > On 5/2/24 13:08, Michael Tokarev wrote: >> This thing works now with sid/unstable, this is what this patch is about. > > It worked when first introduced, then failed, then worked... > Sid is too unstable for our usage as a gating test. unstable can be unstable at times. Generally it works, and can fail during short periods. riscv64 didn't work since it got promoted to release arch last year, due to non-working multiarch because many riscv64 packages in debian had different version numbers compared to other architectures (due to rebuilds of binaries). Later on whole sid was broken by a large time64 transition and a lot of breakages in this area. This is now being completed and is migrated to testing, which is finally available for riscv64 too as a release arch. I wouldn't switch to testing just yet, because at this point, unstable will usually be more stable, or, rather, it will be broken for much shorter periods of time than testing, due to migration delays. It will take some more time for debian testing to become more or less stable in this context. But lack of riscv testing already cost me a *stable* release which failed to *build* (8.2.3 fails to build on riscv64). So I think it is more important to run this job on sid than not to run it at all. Thanks, /mjt -- GPG Key transition (from rsa2048 to rsa4096) since 2024-04-24. New key: rsa4096/61AD3D98ECDF2C8E 9D8B E14E 3F2A 9DD7 9199 28F1 61AD 3D98 ECDF 2C8E Old key: rsa2048/457CE0A0804465C5 6EE1 95D1 886E 8FFB 810D 4324 457C E0A0 8044 65C5 Transition statement: http://www.corpit.ru/mjt/gpg-transition-2024.txt
On Thu, May 02, 2024 at 11:54:15PM +0300, Michael Tokarev wrote: > 02.05.2024 23:41, Richard Henderson wrote: > > On 5/2/24 13:08, Michael Tokarev wrote: > > > This thing works now with sid/unstable, this is what this patch is about. > > > > It worked when first introduced, then failed, then worked... > > Sid is too unstable for our usage as a gating test. > > unstable can be unstable at times. Generally it works, and can fail > during short periods. > > riscv64 didn't work since it got promoted to release arch last year, due > to non-working multiarch because many riscv64 packages in debian had > different version numbers compared to other architectures (due to rebuilds > of binaries). Later on whole sid was broken by a large time64 transition > and a lot of breakages in this area. This is now being completed and > is migrated to testing, which is finally available for riscv64 too as > a release arch. > > I wouldn't switch to testing just yet, because at this point, unstable > will usually be more stable, or, rather, it will be broken for much > shorter periods of time than testing, due to migration delays. > > It will take some more time for debian testing to become more or less > stable in this context. But lack of riscv testing already cost me a > *stable* release which failed to *build* (8.2.3 fails to build on > riscv64). So I think it is more important to run this job on sid > than not to run it at all. JOB_OPTIONAL requires a job to be manually started, but is non-gating. We need something intermediate, that always starts the job, but still keeps it non-gating. You'll see if the pipeline job for riscv is green or yellow (failure), and can then check if its a real failure of a Debian failure. 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 :|
https://gitlab.com/mjt0k/qemu/-/jobs/6768635797 fwiw. /mjt -- GPG Key transition (from rsa2048 to rsa4096) since 2024-04-24. New key: rsa4096/61AD3D98ECDF2C8E 9D8B E14E 3F2A 9DD7 9199 28F1 61AD 3D98 ECDF 2C8E Old key: rsa2048/457CE0A0804465C5 6EE1 95D1 886E 8FFB 810D 4324 457C E0A0 8044 65C5 Transition statement: http://www.corpit.ru/mjt/gpg-transition-2024.txt
© 2016 - 2024 Red Hat, Inc.