[PATCH v3 1/2] CI: Move default image under global defaults

Simon Glass sjg at chromium.org
Sun Feb 16 20:39:34 CET 2025


Hi Tom,

On Sun, 16 Feb 2025 at 09:07, Tom Rini <trini at konsulko.com> wrote:
>
> On Sun, Feb 16, 2025 at 07:10:12AM -0700, Simon Glass wrote:
> > Hi Tom,
> >
> > On Sat, 15 Feb 2025 at 11:12, Tom Rini <trini at konsulko.com> wrote:
> > >
> > > On Sat, Feb 15, 2025 at 10:21:16AM -0700, Simon Glass wrote:
> > > > Hi Tom,
> > > >
> > > > On Sat, 15 Feb 2025 at 07:41, Tom Rini <trini at konsulko.com> wrote:
> > > > >
> > > > > On Sat, Feb 15, 2025 at 04:59:40AM -0700, Simon Glass wrote:
> > > > > > Hi Tom,
> > > > > >
> > > > > > On Mon, 10 Feb 2025 at 09:25, Tom Rini <trini at konsulko.com> wrote:
> > > > > > >
> > > > > > > On Thu, Feb 06, 2025 at 03:38:55PM -0700, Simon Glass wrote:
> > > > > > >
> > > > > > > > This is a global default, so put it under 'default' like the tags.
> > > > > > > >
> > > > > > > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > > > > > > Suggested-by: Tom Rini <trini at konsulko.com>
> > > > > > > > Reviewed-by: Tom Rini <trini at konsulko.com>
> > > > > > >
> > > > > > > Please make v4 include the way you redid the second patch and be on top
> > > > > > > of mainline, thanks.
> > > > > >
> > > > > > That's enough versions for me, so I'll let you do that, if you'd like.
> > > > > > It probably doesn't affect your tree as not as much is done in
> > > > > > parallel.
> > > > >
> > > > > I am disappointed.
> > > >
> > > > I'm sorry to disappoint you.
> > > >
> > > > The background is that I looked at the difference between our trees
> > > > and the gitlab files are quite different. My CI runs take about 35
> > > > mins and it seems that yours is around 90 mins. I would like to reduce
> > > > / remove the delta (for time and patch diff), but I'm not sure how.
> > > >
> > > > My goal is to get CI runs to below 20 minutes, best case.
> > >
> > > I'm sure CI could be quicker still with a number of faster runners. But
> > > if you can't be bothered to make changes against mainline, what is the
> > > point?
> >
> > If you recall, I was working with your tree and had various ideas to
> > speed things up, but you didn't like it. So I've had to do it in my
> > tree. This is not about more runners (although I might have another
> > one soon). It is about running jobs in parallel.
>
> And I wasn't sure more runners in parallel would help (as it would slow
> down the fast runner which is what keeps the long jobs from being even
> longer) as much as adding more regular runners would (which we've done)
> and noted that in the end it's a configuration on the runner side so to
> go ahead. And I reviewed and ack'd the patches here which exposed the
> issues your path revealed. I just can't apply them because they need to
> be rebased (and squashed).

You have already added tags for things, but (IIUC) they are around the
other way from what I have added.

I have a tag called 'single' which means that the machine is only
allowed to one of those jobs. The world-build jobs are marked with
'single'.

For other jobs, I allow the runners to pick up some in parallel
depending on their performance (for moa and tui that is 10).

So at most, there is a 'world build' and 10 test.py jobs running on
the same machine. It seems to work fine in practice, although I would
rather be able to make these two types of jobs mutually exclusive, so
that a runner is either running 10 parallel jobs or 1 'single' job,
but not both. I'm not sure how to do that.

Regards,
Simon


More information about the U-Boot mailing list