Hi Tom, On Sun, 16 Feb 2025 at 14:52, Tom Rini <[email protected]> wrote: > > 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 <[email protected]> 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 <[email protected]> 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 <[email protected]> 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 <[email protected]> > > > > > > > > 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 <[email protected]> > > > > > > > > > > Suggested-by: Tom Rini <[email protected]> > > > > > > > > > > Reviewed-by: Tom Rini <[email protected]> > > > > > > > > > > > > > > > > > > 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?
I've put a tag of 'single' on things that should run on the single-job runner. Everything else can run concurrently, e.g. up to 10 jobs. So I have two runners on the same host. E.g. tui-single has 'limit = 1', but 'tui' has no limit and is just governed by the 'concurrent = 10' at the top of the file. >From my side, I have found it helpful and refreshing to have a gitlab instance which I can control, e.g. it runs in half the time and if my patches are completely blocked by Linaro, etc., I have an escape valve. Regards, Simon
