public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Carlos O'Donell <carlos@redhat.com>
To: DJ Delorie <dj@redhat.com>
Cc: Siddhesh Poyarekar <siddhesh@gotplt.org>, libc-alpha@sourceware.org
Subject: Re: Run TryBot-apply_patch on the full queue?
Date: Mon, 19 Sep 2022 15:46:24 -0400	[thread overview]
Message-ID: <YyjHEASa9GH83wP4@fedora> (raw)
In-Reply-To: <xn5yhj1f94.fsf@greed.delorie.com>

On Mon, Sep 19, 2022 at 12:49:11PM -0400, DJ Delorie via Libc-alpha wrote:
> I'd have to write a whole new curator/runner to enqueue the jobs for the
> trybot.  So, not impossible, but not a tweak of existing software.  The
> existing system would end up doing a full build of every patch every
> day.

The curator/runners are a more advanced "patchwork bot" framework that
can do very specific things.

Currently the curator reads trusted sources for information.
Then that information is processed and runners carry out the actions.

Today the re-triggering CI due to an outage is going to require a developer
or bot to send an email to the list saying "retry CI" (signed command)
and then that will redo CI for thos failed jobs (for all curators and
runners run by all ISVs/IHVs/users/etc that listen to such a command)

We don't want to do this every day for all the patches currently in
the list that apply. This would potentially cause all listeners to
redo their patch application and builds.

However, we do want to emulate the current best practice that if a 
series doesn't rebase then we need to mark it with failed CI.

I think the best design is going to be:

- Write a custom pre-commit CI patchwork bot whose job is *only* to
  apply patches and update the patches SWF with a fail.
  - Run this daily.

- Write a patchwork bot whose job is only to mark F-fails as Failed CI.
  - Run this daily.

- Write a dashboard where the bots can report that they ran?
  - This way we can track all of our bots?

> As for notifications:
> https://github.com/getpatchwork/patchwork/issues/425

I think we'll have to eventually implment notifications via a patchwork
bot that trawls for state changes and emails developers.

Cheers,
Carlos.


  reply	other threads:[~2022-09-19 19:46 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-19 13:31 Siddhesh Poyarekar
2022-09-19 13:38 ` Siddhesh Poyarekar
2022-09-19 16:49 ` DJ Delorie
2022-09-19 19:46   ` Carlos O'Donell [this message]
2022-09-19 19:55     ` DJ Delorie
2022-09-20 10:21       ` Carlos O'Donell
2022-09-20 14:26         ` DJ Delorie
2022-09-20 14:39           ` Siddhesh Poyarekar

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=YyjHEASa9GH83wP4@fedora \
    --to=carlos@redhat.com \
    --cc=dj@redhat.com \
    --cc=libc-alpha@sourceware.org \
    --cc=siddhesh@gotplt.org \
    /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 read-only IMAP folder(s) and NNTP newsgroup(s).