search

kubewharf / katalyst-core

Katalyst aims to provide a universal solution to help improve resource utilization and optimize the overall costs in the cloud. This is the core components in Katalyst system, including multiple agents and centralized components
Apache License 2.0
394 stars 91 forks source link

feat: adding support to io.cost setting #496

Closed lubinszARM closed 3 months ago

lubinszARM commented 4 months ago

What type of PR is this?

Features

What this PR does / why we need it:

This patch provied the feature of io.cost setting.

Which issue(s) this PR fixes:

In production area, some critical services and online pods were affected by busy io. This patch provieded the io qos feature

Special notes for your reviewer:

None

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 46.50350% with 153 lines in your changes are missing coverage. Please review.

Project coverage is 55.53%. Comparing base (b122f7b) to head (8ddf4ef). Report is 3 commits behind head on main.

:exclamation: Current head 8ddf4ef differs from pull request most recent head 48ad87d. Consider uploading reports for the commit 48ad87d to get more accurate results

Files Patch % Lines
...ent/qrm-plugins/io/handlers/iocost/iocost_linux.go 33.87% 121 Missing :warning:
pkg/agent/qrm-plugins/io/handlers/iocost/utils.go 71.25% 17 Missing and 6 partials :warning:
cmd/katalyst-agent/app/options/qrm/io_plugin.go 57.14% 6 Missing :warning:
pkg/agent/qrm-plugins/io/staticpolicy/policy.go 50.00% 2 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #496 +/- ## ========================================== - Coverage 55.67% 55.53% -0.14% ========================================== Files 552 554 +2 Lines 59419 59751 +332 ========================================== + Hits 33083 33185 +102 - Misses 22899 23130 +231 + Partials 3437 3436 -1 ``` | [Flag](https://app.codecov.io/gh/kubewharf/katalyst-core/pull/496/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kubewharf) | Coverage Δ | | |---|---|---| | [unittest](https://app.codecov.io/gh/kubewharf/katalyst-core/pull/496/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kubewharf) | `55.53% <46.50%> (-0.14%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kubewharf#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.