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 docker badges for longer (take 3) #9369

Merged
merged 1 commit into from Jul 12, 2023

Conversation

chris48s
Copy link
Member

Refs #9342

It looks like the magic number we need to stay under is 180 reqs/minute.

We're still going over that at peak times, so I am going to hit it again with the only hammer we've got.

I'm going to leave build status and version badges on the default max-ages for those categories though.

@chris48s chris48s added the service-badge Accepted and actionable changes, features, and bugs label Jul 12, 2023
@github-actions
Copy link
Contributor

Warnings
⚠️ This PR modified service code for docker 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 a33004e

@chris48s chris48s merged commit 83c22cc into badges:master Jul 12, 2023
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service-badge Accepted and actionable changes, features, and bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant