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.

Sign up for GitHub

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

Jump to bottom

cache some badges for longer #9785

Merged
merged 1 commit into from Dec 6, 2023
Merged

cache some badges for longer #9785

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 Accepted and actionable changes, features, and bugs labels Dec 6, 2023
@github-actions GitHub Actions
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
@chris48s chris48s mentioned this pull request Dec 6, 2023
Merged
@chris48s chris48s mentioned this pull request Dec 31, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
operations Hosting, monitoring, and reliability for the production badge servers service-badge Accepted and actionable changes, features, and bugs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@chris48s