Skip to content

repackage: reduce i/o load so we can run in parallel to builds#3344

Merged
GuillaumeGomez merged 1 commit into
rust-lang:mainfrom
syphar:repackage-io
May 21, 2026
Merged

repackage: reduce i/o load so we can run in parallel to builds#3344
GuillaumeGomez merged 1 commit into
rust-lang:mainfrom
syphar:repackage-io

Conversation

@syphar
Copy link
Copy Markdown
Member

@syphar syphar commented May 20, 2026

I saw that build speed was impacted when I ran the repackage command. My guess is that this is because of i/o load.

This PR:

  • switches from sync_all to flush in one case. (difference only important in case of system crashes).
  • makes download / storage concurrency configurable.

in production I then want to run the comment via:

 ionice -c3 docs-rs-admin repackage ...

@syphar syphar requested a review from a team as a code owner May 20, 2026 13:15
@syphar syphar self-assigned this May 20, 2026
@github-actions github-actions Bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label May 20, 2026
@GuillaumeGomez GuillaumeGomez merged commit effecca into rust-lang:main May 21, 2026
11 checks passed
@GuillaumeGomez GuillaumeGomez deleted the repackage-io branch May 21, 2026 17:14
@github-actions github-actions Bot added S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants