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

[Tom Rini]

On Sun, Feb 16, 2025 at 12:39:34PM -0700, Simon Glass wrote:
> 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.

So unless I'm missing something, in both cases the bottleneck is that
for world build jobs you don't want anything else going on with the
underlying build host. You could register 10 "all" runners and 1 "fast
amd64" runner (and something similar but smaller for alexandra). If you
update the registrations on source.denx.de can you then shut down your
gitlab instance?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20250216/8de1d303/attachment.sig>