Skip to content

Conversation

@syphar
Copy link
Member

@syphar syphar commented Jan 20, 2026

there are 362_872 releases out of 1_913_132 left that still store single files on the S3 bucket, and not a zip file.

Since rebuilding would take quite some time, and also wouldn't be guarenteed to work for old releases, I think the best idea is to repackage the files.

So

  • download all files
  • create & upload the ZIP
  • switch the archive-storage marker
  • delete the single files

I'll try the migration one-by-one first and manually check if there are any issues.

After all old releases are migrated, we can drop a whole lot of code related to the old storage, G
also we'll be able to provide ZIP downloads for these old releases too.

@syphar syphar requested a review from a team as a code owner January 20, 2026 17:57
@github-actions github-actions bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Jan 20, 2026
@syphar syphar self-assigned this Jan 20, 2026
@GuillaumeGomez
Copy link
Member

Please add instructions about it (in a README I suppose).

@syphar
Copy link
Member Author

syphar commented Jan 21, 2026

Please add instructions about it (in a README I suppose).

For whom?

This a temporary command, and I'll drop it when the run is finished, probably in a couple of weeks.

Or am I missing something?

@GuillaumeGomez
Copy link
Member

No, I was missing something. Nevermind then! Thanks. :)

@GuillaumeGomez GuillaumeGomez merged commit 92056ed into rust-lang:main Jan 21, 2026
15 checks passed
@GuillaumeGomez GuillaumeGomez deleted the migrate-archive-storage branch January 21, 2026 14:06
@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 Jan 21, 2026
@syphar
Copy link
Member Author

syphar commented Jan 21, 2026

No, I was missing something. Nevermind then! Thanks. :)

re-reading the PR i see I forgot to mention that in the PR descrition.

" temporary" is only mentioned in the CLI structs / docstring

@syphar syphar removed the S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it label Jan 22, 2026
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