Skip to content

feat: make the siso remote job count configurable - #921

Merged
MarshallOfSound merged 1 commit into
electron:mainfrom
nornagon:feat/siso-remote-jobs
Oct 23, 2026
Merged

MarshallOfSound merged 1 commit into
electron:mainfrom
nornagon:feat/siso-remote-jobs

Conversation

@nornagon

Copy link
Copy Markdown
Contributor

e build always ran siso with -remote_jobs 200. Passing the flag yourself had no effect: build-tools adds its own flags after the user's and siso keeps the last value, so e build -remote_jobs 500 still ran with 200.

A -remote_jobs flag on the command line is now respected, the same way -j already is for reclient builds. A new rbeRemoteJobs config option sets the value permanently, for backends with more or less capacity than the default. The default stays at 200.

@nornagon
nornagon requested review from a team and ckerr as code owners October 23, 2026 15:51
`e build` always ran siso with `-remote_jobs 200`. Passing the flag yourself
had no effect: build-tools adds its own flags after the user's and siso keeps
the last value, so `e build -remote_jobs 500` still ran with 200.

A `-remote_jobs` flag on the command line is now respected, the same way `-j`
already is for reclient builds. A new `rbeRemoteJobs` config option sets the
value permanently, for backends with more or less capacity than the default.
The default stays at 200.
@nornagon
nornagon force-pushed the feat/siso-remote-jobs branch from aadfe6f to 1b14491 Compare October 23, 2026 17:43
@MarshallOfSound
MarshallOfSound enabled auto-merge (squash) October 23, 2026 18:02
@MarshallOfSound
MarshallOfSound merged commit e3edc2e into electron:main Oct 23, 2026
13 checks passed
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