Skip to content

Conversation

@syphar
Copy link
Member

@syphar syphar commented Dec 5, 2025

I'm sorry for the whole back and forth here.

With the time I have difficulties reproducing the load-issue locally, so it's testing in production :)
( hopefully with little impact).

This sentry error pointed me to

  1. I didn't add the "fallback" that protects against some index edge cases to all places where we work with the index. Thats in this PR.
  2. I went back to a simpler solution for the download-or-fetch problem, between the old and the new way.

With a warm local cache all should be fine, I have to find a good time when I just test the cold cache.

the whole "cold cache" issue will bite us when we migrate our webserver to ecs, I assume there we'll have a fresh filesystem for every deploy

@syphar syphar self-assigned this Dec 5, 2025
@syphar syphar requested a review from a team as a code owner December 5, 2025 19:00
@github-actions github-actions bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Dec 5, 2025
@syphar syphar force-pushed the different-logkcig branch from 6b22574 to 429f9b2 Compare December 5, 2025 19:04
@syphar syphar requested a review from GuillaumeGomez December 7, 2025 08:51
@GuillaumeGomez
Copy link
Member

Thanks!

@syphar syphar merged commit b6b9740 into rust-lang:main Dec 7, 2025
22 of 25 checks passed
@syphar syphar deleted the different-logkcig branch December 7, 2025 10:58
@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 Dec 7, 2025
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