Skip to content

ADR for concurrent request rate limiting - #5435

Open
stephanme wants to merge 2 commits into
mainfrom
rate-limiter-adr
Open

stephanme wants to merge 2 commits into
mainfrom
rate-limiter-adr

Conversation

@stephanme

Copy link
Copy Markdown
Member

Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

  • A short explanation of the proposed change:

  • An explanation of the use cases your change solves

  • Links to any other associated PRs

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

Comment thread decisions/0016-rate-limiting.md Outdated
Comment thread decisions/0016-rate-limiting.md Outdated
- implementation on #5361 was improved and reaches more throughput now
- all measurements run for 300s
- more detailed explanation how to determine max throughput
- some more arguments for replacing nginx by a dedicated CAPI router
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants