Skip to content
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

cache some badges for longer #9785

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

chris48s
Copy link
Member

@chris48s chris48s commented Dec 6, 2023

Having merged and deployed #9546 there's still a bunch of services where we're regularly hitting rate limits. Upping the caches again.

I'm a bit worried I might have completely smashed our sentry allowance for the month, but I wasn't able to immediately do something about this yesterday.

There are some other rate limiting issues which I will track elsewhere.

I'm going to just merge and deploy this one today.

@chris48s chris48s added operations Hosting, monitoring, and reliability for the production badge servers service-badge New or updated service badge labels Dec 6, 2023
Copy link
Contributor

github-actions bot commented Dec 6, 2023

Warnings
⚠️ This PR modified service code for bundlephobia but not its test code.
That's okay so long as it's refactoring existing code.
⚠️ This PR modified service code for discord but not its test code.
That's okay so long as it's refactoring existing code.
⚠️ This PR modified service code for opencollective but not its test code.
That's okay so long as it's refactoring existing code.
⚠️ This PR modified service code for pypi but not its test code.
That's okay so long as it's refactoring existing code.
Messages
📖 ✨ Thanks for your contribution to Shields, @chris48s!

Generated by 🚫 dangerJS against c9bf84c

@chris48s chris48s merged commit 57820b2 into badges:master Dec 6, 2023
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
operations Hosting, monitoring, and reliability for the production badge servers service-badge New or updated service badge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant