-
Notifications
You must be signed in to change notification settings - Fork 12.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bump max builds to 3 on bots #17395
Comments
We might also switch back to high-mem instances, which have 30GB RAM, but slightly slower CPU's. That would allow us many more builds. |
I've bumped the builds per slave to 3. |
lnicola
pushed a commit
to lnicola/rust
that referenced
this issue
Jul 28, 2024
…oject-notification, r=Veykril chore: remove `UnindexinedProject` notification This PR is split out from rust-lang/rust-analyzer#17246 (and contains its changes, which is a little annoying from a review perspective...). I'd like to land this change a week or so after rust-lang#17246 lands in order to give any users of the unindexed project notification time to adopt migrate.
RalfJung
pushed a commit
to RalfJung/rust
that referenced
this issue
Aug 1, 2024
…oject-notification, r=Veykril chore: remove `UnindexinedProject` notification This PR is split out from rust-lang/rust-analyzer#17246 (and contains its changes, which is a little annoying from a review perspective...). I'd like to land this change a week or so after rust-lang#17246 lands in order to give any users of the unindexed project notification time to adopt migrate.
lnicola
pushed a commit
to lnicola/rust
that referenced
this issue
Aug 29, 2024
…, r=lnicola Remove rust-analyzer.workspace.discoverProjectRunner The functionality for this vscode config option was removed in rust-lang#17395, so it doesn't do anything anymore.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now we are running 8-core/15GM VMs but only 2 builds at a time, usually with -j2. It seems like we could easily bump this to 3 builds at a time to improve our utilization.
The text was updated successfully, but these errors were encountered: