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.

Sign up for GitHub

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

Jump to bottom

Manager adds ratelimit for CreateJob and GetJobs. #3606

Open
gaius-qi opened this issue Oct 24, 2024 · 0 comments · Fixed by #3638
Open

Manager adds ratelimit for CreateJob and GetJobs. #3606

gaius-qi opened this issue Oct 24, 2024 · 0 comments · Fixed by #3638
Labels
enhancement New feature or request help wanted Extra attention is needed
Milestone
v2.2.0

Comments

@gaius-qi
Copy link
Member

Feature request:

  • The rate limiter using a distributed manager services.
    • Need to store the values of rate limiter in redis.
  • Different levels of ratelimit need to be designed and can be configured in the console.
    • Levels include all of the clusters and single of the cluster.

Use case:

UI Example:
@gaius-qi gaius-qi added enhancement New feature or request help wanted Extra attention is needed labels Oct 24, 2024
@gaius-qi gaius-qi added this to the v2.2.0 milestone Oct 24, 2024
@gaius-qi gaius-qi linked a pull request Nov 8, 2024 that will close this issue
feat: add rate limit for job open api by cluster #3638
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Milestone
v2.2.0
Development

Successfully merging a pull request may close this issue.

feat: add rate limit for job open api by cluster dragonflyoss/Dragonfly2
1 participant
@gaius-qi