feature(sandbox-manager): add rate limiter to all apis and introduce a better limiter framework#172
Conversation
…a better limiter framework Signed-off-by: AiRanthem <zhongtianyun.zty@alibaba-inc.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #172 +/- ##
==========================================
+ Coverage 57.10% 57.27% +0.16%
==========================================
Files 94 96 +2
Lines 5761 5783 +22
==========================================
+ Hits 3290 3312 +22
Misses 2234 2234
Partials 237 237
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…a better limiter framework Signed-off-by: AiRanthem <zhongtianyun.zty@alibaba-inc.com>
…a better limiter framework Signed-off-by: AiRanthem <zhongtianyun.zty@alibaba-inc.com>
|
/lgtm |
|
@AiRanthem: PR needs rebase. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews