Skip to content

Conversation

richiejp
Copy link
Collaborator

Signed-off-by: Richard Palethorpe [email protected]

Description

Possibly the CI CPU or RAM is being over subscribed and either the runner process is not able to report progress or is killed by OOM.

The original Purego PR accidentally set CMake to use all the cores, now it defaults to nproc - 1

Compiling locally it doesn't appear that JOBS=4 uses more than 16GB of RAM (with JOBS=20 it certainly does), so I would guess that CPU is the issue. An alternative is to use nice (possibly from Docker https://github.com/docker/build-push-action?tab=readme-ov-file#environment-variables).

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Copy link

netlify bot commented Aug 15, 2025

Deploy Preview for localai ready!

Name Link
🔨 Latest commit 361e1a6
🔍 Latest deploy log https://app.netlify.com/projects/localai/deploys/68a02e96784169000803e39b
😎 Deploy Preview https://deploy-preview-6067--localai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@richiejp
Copy link
Collaborator Author

I'm trying to figure out a sensible way to run the backend build/tests for the particular backend that was changed which possibly means using a dynamic matrix.

In the meantime I'm just rebuilding all of the backends on my fork's CI: https://github.com/richiejp/LocalAI/actions/runs/16999488324

@richiejp
Copy link
Collaborator Author

Let's try that again: https://github.com/richiejp/LocalAI/actions/runs/17005231171/job/48213847468

@mudler
Copy link
Owner

mudler commented Aug 16, 2025

I'm trying to figure out a sensible way to run the backend build/tests for the particular backend that was changed which possibly means using a dynamic matrix.

In the meantime I'm just rebuilding all of the backends on my fork's CI: https://github.com/richiejp/LocalAI/actions/runs/16999488324

yup, as a one-off I usually edit

and add the pull_request event

@richiejp
Copy link
Collaborator Author

OK, I was a bit worried about doing that on the main repo in case it counted towards some usage limits, but it's probably fine in that case.

Seem to be having some success: https://github.com/richiejp/LocalAI/actions/runs/17005231171/job/48213847641

@richiejp
Copy link
Collaborator Author

and https://github.com/richiejp/LocalAI/actions/runs/17005231171/job/48213847442

There's another bug I noticed which could effect OSX, so I'll submit a fix for that as well.

@richiejp richiejp marked this pull request as ready for review August 16, 2025 07:09
Copy link
Owner

@mudler mudler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice! let's see how it behaves on master

@mudler mudler merged commit 784bd5d into mudler:master Aug 16, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants