From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Alexander Bulekov <alxndr@bu.edu>
Cc: Peter Maydell <peter.maydell@linaro.org>,
Thomas Huth <thuth@redhat.com>,
QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [PULL 00/11] Optional gitlab-CI and doc fixes for -rc4
Date: Mon, 16 Aug 2021 12:44:09 +0100 [thread overview]
Message-ID: <YRpPiaazIrASHJUM@redhat.com> (raw)
In-Reply-To: <20210816113059.h2srf2tmvylzhhjp@mozz.bu.edu>
On Mon, Aug 16, 2021 at 07:30:59AM -0400, Alexander Bulekov wrote:
> On 210816 1211, Daniel P. Berrangé wrote:
> > On Mon, Aug 16, 2021 at 06:22:46AM -0400, Alexander Bulekov wrote:
> > > On 210816 1001, Peter Maydell wrote:
> > > > On Sat, 14 Aug 2021 at 07:10, Thomas Huth <thuth@redhat.com> wrote:
> > > > >
> > > > > Hi Peter,
> > > > >
> > > > > in case we're going to have an -rc4, here's a pull request that contains
> > > > > the fixes for getting the gitlab-CI green again. I also added some doc
> > > > > updates since they should be completely riskless. But if we won't have an
> > > > > rc4 due to other reasons, this pull request here certainly also does not
> > > > > justify another RC, so please ignore this PR in that case.
> > > > >
> > > > > The following changes since commit 703e8cd6189cf699c8d5c094bc68b5f3afa6ad71:
> > > > >
> > > > > Update version for v6.1.0-rc3 release (2021-08-10 19:08:09 +0100)
> > > > >
> > > > > are available in the Git repository at:
> > > > >
> > > > > https://gitlab.com/thuth/qemu.git tags/pull-request-2021-08-11
> > > > >
> > > > > for you to fetch changes up to 36b508993c4dcc6b3ef4b5c00e293ee9560926ee:
> > > > >
> > > > > docs/about/removed-features: Document removed machines from older QEMU versions (2021-08-11 15:39:09 +0200)
> > > > >
> > > > > CI run can be seen here:
> > > > >
> > > > > https://gitlab.com/thuth/qemu/-/pipelines/351602605
> > > > >
> > > > > ----------------------------------------------------------------
> > > > > * Fixes for the gitlab-CI (fix the hanging build-oss-fuzz pipeline)
> > > > > * Add documentation about features that have been removed in older versions
> > > > >
> > > >
> > > >
> > > > Applied, thanks.
> > > >
> > > > Please update the changelog at https://wiki.qemu.org/ChangeLog/6.1
> > > > for any user-visible changes.
> > >
> > > https://gitlab.com/qemu-project/qemu/-/jobs/1505950978
> > > Looks like build-oss-fuzz is still timing out, even without the issue
> > > in the vhost-usr-blk test. At this point the job should essentially just
> > > build + test qemu-system-i386 with some extra time spent on linking
> > > the fuzzer and briefly running through all the fuzzer configs. Maybe the
> > > only way to make this work is to split the job into a build + test
> > > stage?
> >
> > At this point I think we should just disable the job in gitlab entirely.
> > We've spent too long debugging this, while leaving CI red for everyone.
> >
> > Whomever is interested in this can then work to find a way to make it
> > reliable and request it be re-enabled once confident that it will work.
> >
>
> On my mirror the job succeeded in 41 minutes... I guess you have to get
> lucky with scheduling/ambient load.
> https://gitlab.com/a1xndr/qemu/-/jobs/1506197531
I don't think load would make that much of a difference. It smells like
there is still some non-deterministic bug in there that is causing a
hang of the job.
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 :|
next prev parent reply other threads:[~2021-08-16 11:46 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-14 6:09 [PULL 00/11] Optional gitlab-CI and doc fixes for -rc4 Thomas Huth
2021-08-14 6:09 ` [PULL 01/11] gitlab: exclude sparc-softmmu and riscv32-softmmu from cross builds Thomas Huth
2021-08-14 6:09 ` [PULL 02/11] gitlab: skip many more targets in windows " Thomas Huth
2021-08-15 15:47 ` Peter Maydell
2021-08-14 6:09 ` [PULL 03/11] storage-daemon: Add missing build dependency to the vhost-user-blk-test Thomas Huth
2021-08-14 6:09 ` [PULL 04/11] tests/qtest/vhost-user-blk-test: Check whether qemu-storage-daemon is available Thomas Huth
2021-08-14 6:09 ` [PULL 05/11] fuzz: avoid building twice, when running on gitlab Thomas Huth
2021-08-14 6:09 ` [PULL 06/11] docs/about/removed-features: Document removed CLI options from QEMU v2.12 Thomas Huth
2021-08-14 6:09 ` [PULL 07/11] docs/about/removed-features: Document removed CLI options from QEMU v3.0 Thomas Huth
2021-08-14 6:09 ` [PULL 08/11] docs/about/removed-features: Document removed CLI options from QEMU v3.1 Thomas Huth
2021-08-14 6:09 ` [PULL 09/11] docs/about/removed-features: Document removed HMP commands from QEMU v2.12 Thomas Huth
2021-08-14 6:09 ` [PULL 10/11] docs/about/removed-features: Document removed devices from older QEMU versions Thomas Huth
2021-08-14 6:09 ` [PULL 11/11] docs/about/removed-features: Document removed machines " Thomas Huth
2021-08-16 9:01 ` [PULL 00/11] Optional gitlab-CI and doc fixes for -rc4 Peter Maydell
2021-08-16 10:22 ` Alexander Bulekov
2021-08-16 11:11 ` Daniel P. Berrangé
2021-08-16 11:30 ` Alexander Bulekov
2021-08-16 11:40 ` Philippe Mathieu-Daudé
2021-08-16 12:00 ` Alexander Bulekov
2021-08-16 11:44 ` Daniel P. Berrangé [this message]
2021-08-17 7:23 ` Thomas Huth
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YRpPiaazIrASHJUM@redhat.com \
--to=berrange@redhat.com \
--cc=alxndr@bu.edu \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).