-
Notifications
You must be signed in to change notification settings - Fork 119
feat: add loadaware scheduler plugin and npd controller plugin #654
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.
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
base: main
Are you sure you want to change the base?
Conversation
d86d580 to
2949fc3
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #654 +/- ##
==========================================
+ Coverage 57.00% 57.04% +0.04%
==========================================
Files 552 567 +15
Lines 52838 53918 +1080
==========================================
+ Hits 30118 30756 +638
- Misses 18941 19333 +392
- Partials 3779 3829 +50
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:
|
5545830 to
4375250
Compare
4375250 to
7bdce6a
Compare
| ) | ||
|
|
||
| type NPDOptions struct { | ||
| NPDIndicatorPlugins []string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NPDMetricsPlugins
What type of PR is this?
Enhancements
What this PR does / why we need it:
loadAware scheduler plugin:
schedule pod by NPD and SPD portrait metrics
npd controller indicator plugin:
fetch metrics from metrics server and patch aggregated data to NPD status
Special notes for your reviewer:
pr should be rebased after kubewharf/katalyst-api#94 and #633 merged