Open WangZzzhe opened 3 months ago
Attention: Patch coverage is 71.05263%
with 132 lines
in your changes are missing coverage. Please review.
Project coverage is 56.26%. Comparing base (
74b648c
) to head (4db7acc
). Report is 11 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
- what's the difference between this eviction plugin and cpu-load && cpu-suppression eviction plugins?
- why don't we implement this eviction in qrm but in eviction-manager? will it be conflict with qrm hybrid logic?
What type of PR is this?
Enhancements
What this PR does / why we need it:
Which issue(s) this PR fixes:
https://github.com/kubewharf/katalyst-core/issues/492 PR in katalyst-api: https://github.com/kubewharf/katalyst-api/pull/74 go.mod should be updated after api PR merged