Add Concurrency Limiter

sargun commented 5 years ago

Description

This enables limiting concurrency. It is a naive approach which will reject requests as soon as they exceed the maximum number of in-flight requests.

Motivation and Context

[X] I have raised an issue to propose this change (https://github.com/openfaas-incubator/of-watchdog/issues/52)

How Has This Been Tested?

There are tests in the code that test for all of the obvious possible variants. The code is written in such a way that testing is easy.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I've read the CONTRIBUTION guide
[X] I have signed-off my commits with git commit -s
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

alexellis commented 5 years ago

Derek close

alexellis commented 5 years ago

Derek add label: invalid

alexellis commented 5 years ago

Hi @sargun thank you so much for your interest in OpenFaaS.

I'm going to have to close your PR because you've not followed the contribution process, which is very clear.

Regards,

Alex

From the contributing guide:

https://github.com/openfaas/faas/blob/master/CONTRIBUTING.md#i-have-a-great-idea

Please do not raise a proposal after doing the work - this is counter to the spirit of the project. It is hard to be objective about something which has already been done

alexellis commented 5 years ago

Derek reopen

openfaas / of-watchdog