Re: [PATCH] ci: avoid unnecessary builds
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Nov 7, 2022, 19:53 UTC
- Message-ID
- <Y2liOzOFLyz4uzd0@nand.local>
- In-Reply-To
- <f975f57e-71e2-3227-8039-14dff82f04db@github.com>
On Mon, Nov 07, 2022 at 02:45:24PM -0500, Derrick Stolee wrote:
Show 19 quoted lines
> On 11/3/22 9:34 AM, Johannes Schindelin via GitGitGadget wrote: > > From: Johannes Schindelin <johannes.schindelin@gmx.de> > > > > Whenever a branch is pushed to a repository which has GitHub Actions > > enabled, a bunch of new workflow runs are started. > > > > We sometimes see contributors push multiple branch updates in rapid > > succession, which in conjunction with the impressive time swallowed by > > even just a single CI build frequently leads to many queued-up runs. > > > > This is particularly problematic in the case of Pull Requests where a > > single contributor can easily (inadvertently) prevent timely builds for > > other contributors. > > As someone who is both the cause and the victim of this, I > thank you for finding a way to reduce wasted CPU time. This > patch looks good to me, though I'll need to trust the docs > and your testing to be sure it will work. We will definitely > see it in place as it merges into 'next' and 'main'.
I wonder how we should treat Ævar's concerns in this thread. I suspect that the vast majority of workflows wouldn't be affected, but I don't want to completely break Ævar's workflow, either ;-).
Some kind of configuration mechanism like I proposed might be nice. Thoughts?
Thanks, Taylor