git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [RFC] How to accellerate the patch flow (or should we?)

From
Patrick Steinhardt <ps@pks.im>
Date
Sep 29, 2025, 22:23 UTC
Message-ID
<aNsG9kmkfoToSmAD@pks.im>
In-Reply-To
<b583b17e-96a5-4f22-8cc4-acbd3dfec82b@app.fastmail.com>
On Mon, Sep 29, 2025 at 10:12:31PM +0200, Kristoffer Haugsbakk wrote:
Show 33 quoted lines
> On Sat, Sep 27, 2025, at 23:32, Taylor Blau wrote:
> > On Fri, Sep 26, 2025 at 03:24:04PM -0700, Junio C Hamano wrote:
> >>  4. While the above cycle is running, the maintainer may queue it in
> >>     'seen', for two purposes.  (1) not to lose sight and forget
> >>     about the change.  (2) to catch potential conflicts and overlaps
> >>     with other in-flight topics to keep their interaction manageable.
> >
> > Perhaps a third purpose is to let the maintainer (or those who use
> > and/or build off of 'seen' as their daily driver) detect any bugs in
> > that topic, or via interaction with other topics in 'seen'.
> >
> >> The time taken during 7. is pretty much fixed and unless we are
> >> willing to sacrifice the quality of the end result, cannot
> >> reasonably be shortened (note that this is based on the assumption
> >> that "find any remaining bugs while it is in 'next' before it hits
> >> 'master'" philosophy is working, but we have never run experiments
> >> to shorten this to say 3 days to see if we see more bugs on 'master'
> >> yet).
> >
> > I have mixed feelings about this. On the one hand, I am a little
> > uncomfortable with the idea of shortening the time in 'next' to fewer
> > than 7 days. I, too, have the feeling that having more time in 'next'
> > gives us a greater chance of spotting bugs in a topic that is otherwise
> > destined for 'master'.
> >
> > On the other hand, how many people are using 'next' as their daily
> > driver? Of those, how many are actively looking for bugs in the topics
> > that are in master..next. And of those, how many are actually triggering
> > unique code paths that would expose those bugs in the first place?
> 
> Theoretically all the projects that make heavy use of git(1) could run
> `next` (and `master`) as an alternative configuration of their
> integration tests.

At GitLab we do: we have nightly CI runs that execute our integration tests with both `next` and `master`. This has already surfaced issues at several points in time. We also have plans to roll out `next` to all of our developers, but we haven't gotten around to this one yet.

Given the daily schedule of the CI runs it would be _okayish_ to reduce the 7 days to something shorter. But realistically, not every pipeline failure is noticed immediately, and sometimes we have different priorities before we can dive into the pipeline failure.

The same is basically true for those companies that deploy to their devs from `next`. There is a certain roundtrip time: the new version has to be deployed, people have to start using it, and the bug reports have to trickle back to the Git teams before they become actionable.

So I don't really feel like we can significantly reduce these 7 days without losing some of the safety.

Patrick
Previous: Ben KnobleNext: Patrick Steinhardt
Message 13 of 17 in “[RFC] How to accellerate the patch flow (or should we?)”
  1. Junio C HamanoSep 26, 2025
  2. Taylor BlauSep 27, 2025
  3. Junio C HamanoSep 28, 2025
  4. Taylor BlauSep 28, 2025
  5. Patrick SteinhardtSep 29, 2025
  6. Junio C HamanoSep 29, 2025
  7. Patrick SteinhardtSep 29, 2025
  8. Junio C HamanoOct 1, 2025
  9. Taylor BlauSep 30, 2025
  10. Junio C HamanoSep 30, 2025
  11. Kristoffer HaugsbakkSep 29, 2025
  12. Ben KnobleSep 29, 2025
  13. Patrick SteinhardtSep 29, 2025
  14. Patrick SteinhardtSep 29, 2025
  15. Taylor BlauSep 30, 2025
  16. Kristoffer HaugsbakkSep 29, 2025
  17. Junio C HamanoSep 29, 2025

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.