scheduler: Add hook plugins logic in ReplaceQuotas and OnQuotaUpdate methods of ElasticQuota plugin.#2465
Conversation
…methods of ElasticQuota plugin. Signed-off-by: Tao Yang <taoyang@apache.org>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2465 +/- ##
==========================================
+ Coverage 65.93% 65.95% +0.02%
==========================================
Files 477 477
Lines 56194 56214 +20
==========================================
+ Hits 37049 37078 +29
+ Misses 16461 16452 -9
Partials 2684 2684
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:
|
Signed-off-by: Tao Yang <taoyang@apache.org>
|
@shaloulcy @saintube Please help to review this PR, Thanks. |
|
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: saintube The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Ⅰ. Describe what this PR does
There are 2 locations in quota_handler.go that need to be made compatible with hook plugins:
Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist
make test