From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "Thomas Huth" <thuth@redhat.com>,
"QEMU Developers" <qemu-devel@nongnu.org>,
"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
"Willian Rampazzo" <wrampazz@redhat.com>,
"Alex Bennée" <alex.bennee@linaro.org>
Subject: Re: retrying failed gitlab CI external jobs (travis)
Date: Mon, 12 Jul 2021 12:08:34 +0100 [thread overview]
Message-ID: <YOwisuzpWCsPe/CT@redhat.com> (raw)
In-Reply-To: <CAFEAcA9ragK95AtCz0zCfxpDs0kF6SxXtq10smW44b4abCkTmQ@mail.gmail.com>
On Mon, Jul 12, 2021 at 12:03:02PM +0100, Peter Maydell wrote:
> On Mon, 12 Jul 2021 at 10:22, Daniel P. Berrangé <berrange@redhat.com> wrote:
> > WRT missing job for commit fc32b91a, I see there is a Travis stage
> > reported here:
> >
> > https://gitlab.com/qemu-project/qemu/-/pipelines/334907106/builds
> >
> > So I presume there was some delay in running the Travis jobs and
> > thus they only got reported after you sent the mail.
> >
> > The extra stage in the pipeline is not triggered/tracked by GitLab
> > itself. Rather it relies on Travis to see the changes, runs its job
> > and pushes information back to GitLab. This is completely asynchronous
> > to the rest of the normal GitLab pipeline, so unfortunately if Travis
> > hasn't even started the job yet, we see nothing :=(
>
> Hmm. If nothing shows up in the gitlab pipeline, it doesn't really
> work as a gating CI step, because I won't see if it fails...
Agreed, the Travis integration isn't ideal in this respect. We probably
have to consider to be merely a nice sanity check, rather than fully
gating CI.
The way I've integrated Cirrus CI into GitLab is more useful, because
that uses the GitLab job to directly trigger the Cirrus job, and so
we don't suffer from the asynchronous feedback loop going missing
entirely as with Travis.
It would be nice to integrate Traviss in the same way as Cirrus, but
I'm still not sure it is possible with Travis' REST API.
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 :|
prev parent reply other threads:[~2021-07-12 11:11 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-10 13:34 retrying failed gitlab CI external jobs (travis) Peter Maydell
2021-07-10 16:29 ` Peter Maydell
2021-07-12 9:22 ` Daniel P. Berrangé
2021-07-12 11:03 ` Peter Maydell
2021-07-12 11:08 ` Daniel P. Berrangé [this message]
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=YOwisuzpWCsPe/CT@redhat.com \
--to=berrange@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=f4bug@amsat.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.com \
--cc=wainersm@redhat.com \
--cc=wrampazz@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).