qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: Kevin Wolf <kwolf@redhat.com>
Cc: Jason Wang <jasowang@redhat.com>,
	Gerd Hoffmann <kraxel@redhat.com>,
	QEMU Developers <qemu-devel@nongnu.org>,
	Markus Armbruster <armbru@redhat.com>
Subject: Re: please try to avoid sending pullreqs late on release-candidate day
Date: Wed, 22 Jul 2020 12:50:20 +0100	[thread overview]
Message-ID: <CAFEAcA93OY0omjoYW7ofH7N6iRFHfboa1DGO1ra3qha0SasVVg@mail.gmail.com> (raw)
In-Reply-To: <20200722093621.GA4838@linux.fritz.box>

On Wed, 22 Jul 2020 at 10:36, Kevin Wolf <kwolf@redhat.com> wrote:
>
> Am 21.07.2020 um 17:56 hat Peter Maydell geschrieben:
> > It is not helpful if everybody sends their pullrequests late
> > on the Tuesday afternoon, as there just isn't enough time in the
> > day to merge test and apply them all before I have to cut the tag.
> > Please, if you can, try to send pullrequests earlier, eg Monday.
>
> I sent the majority of my fixes for -rc1 on Friday, not the least to
> give us some time in case we get a testing failure. However, the earlier
> you send the pull request, the greater the chance that you get some new
> patches after the pull request. In this case, the patches were only
> ready Tuesday afternoon, so even sending on Monday instead of Friday
> wouldn't have helped.

Patches that arrive and are only ready Tuesday afternoon are
naturally at risk of slipping into the next RC. That's OK.
Though when we get to rc2/rc3 you should warn me when you
expect that so I can make a decision about whether it's better
to slip the rc by a day to wait for them.

> The alternative would have been letting them wait for -rc2. I suppose
> you can always says "too late" and make that decision for me, but I
> wouldn't want to unnecessarily move things to a later RC. Do you think
> we shouldn't send a pull request in case of doubt?

Mostly what I mean is "don't assume that because RC day is Tuesday
that you can send a pullreq on Tuesday and have it get into the RC".
If it turns out that you have to do that, that's not a big problem.
What is a problem is if half a dozen submaintainers all send a
pullreq at once on the Tuesday afternoon. So in the situation where
you don't anticipate anything much late arriving then send it earlier.

> Can you test multiple pull requests at once?

I could in theory I guess, but my scripting assumes one at once.

thanks
-- PMM


  reply	other threads:[~2020-07-22 11:51 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-21 15:56 please try to avoid sending pullreqs late on release-candidate day Peter Maydell
2020-07-21 21:16 ` Gerd Hoffmann
2020-07-22  8:05   ` Peter Maydell
2020-07-22  3:34 ` Jason Wang
2020-07-22  9:36 ` Kevin Wolf
2020-07-22 11:50   ` Peter Maydell [this message]
2020-07-22 12:16   ` Alex Bennée
2020-07-23  6:28     ` Markus Armbruster
2020-07-23 10:26       ` Philippe Mathieu-Daudé
2020-07-23 11:12         ` Markus Armbruster
2020-07-23 16:36         ` Alex Bennée

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=CAFEAcA93OY0omjoYW7ofH7N6iRFHfboa1DGO1ra3qha0SasVVg@mail.gmail.com \
    --to=peter.maydell@linaro.org \
    --cc=armbru@redhat.com \
    --cc=jasowang@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=qemu-devel@nongnu.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 NNTP newsgroup(s).