Skip to content

fix: run coglet wheel build after cog/sdk to avoid resource exhaustion#2977

Merged
michaeldwan merged 1 commit intomainfrom
md/sequential-build
Apr 29, 2026
Merged

fix: run coglet wheel build after cog/sdk to avoid resource exhaustion#2977
michaeldwan merged 1 commit intomainfrom
md/sequential-build

Conversation

@michaeldwan
Copy link
Copy Markdown
Member

The mise run build task was running all three component builds in parallel -- cog CLI, SDK wheel, and coglet cross-compile (maturin+zig). The coglet build is heavy enough that running it alongside the others exhausts resources.

Now cog and SDK build in parallel, then coglet builds sequentially after they finish.

The build task was running all three builds in parallel (cog CLI, SDK
wheel, coglet cross-compile). The coglet maturin+zig cross-compile is
heavy enough that running it alongside the other two exhausts resources.

Now cog and SDK build in parallel, then coglet builds after they finish.
@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented Apr 29, 2026

I'm Bonk, and I've done a quick review of your PR.

Summary: Changes the mise run build task to run build:cog and build:sdk in parallel, then sequentially run the heavy build:coglet:wheel:linux-x64 task afterward to prevent resource exhaustion during cross-compilation.

This is a straightforward fix that addresses the resource exhaustion issue described. The change correctly groups the lighter builds (Go CLI + Python SDK) in parallel, then runs the heavy maturin+zig cross-compilation sequentially. LGTM

github run

@michaeldwan michaeldwan marked this pull request as ready for review April 29, 2026 20:52
@michaeldwan michaeldwan requested a review from a team as a code owner April 29, 2026 20:52
@michaeldwan michaeldwan added this pull request to the merge queue Apr 29, 2026
Merged via the queue into main with commit 2a0755c Apr 29, 2026
43 checks passed
@michaeldwan michaeldwan deleted the md/sequential-build branch April 29, 2026 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants