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

Reduce abusers rate limit to 40 rps and add rate limit on versions #2634

Merged
merged 1 commit into from
Feb 8, 2021

Commits on Feb 6, 2021

  1. Reduce abusers rate limit to 40 rps and add rate limit on versions

    Fixes: CPU spikes on postgres.
    
    We had increased limits to 100 rps when we were having issues with
    info endpoint. we don't rate limit info at the moment, there limit
    need not be this high. Further, we had 100 rps at the time when
    we were running single nginx. With nginx on every pod, our rate limit
    was 100 * number of pods (replicas).
    
    versions was missing rate limit. added it for good measure.
    
    I have verified that our normal traffic doesn't need this high limits.
    Our original rate limits were 10 rps:
    https://github.com/rubygems/rubygems-chef/blob/master/cookbooks/rubygems-balancer/templates/default/site.conf.erb#L14
    sonalkr132 committed Feb 6, 2021
    Configuration menu
    Copy the full SHA
    302addd View commit details
    Browse the repository at this point in the history