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

doc(*): add proposal for enhance orm by nri #525

Closed Airren closed 2 months ago

Airren commented 3 months ago

What type of PR is this?

Enhancements

What this PR does / why we need it:

Leverage NRI to enhance ORM, thus avoiding intrusive modifications to upstream Kubernetes components. This facilitates synchronous configuration of QoS policies and seamless injection of environment variables

Which issue(s) this PR fixes:

https://github.com/kubewharf/katalyst-core/issues/430

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 55.48%. Comparing base (2d6e0a3) to head (6914502). Report is 67 commits behind head on main.

:exclamation: Current head 6914502 differs from pull request most recent head 776b501. Consider uploading reports for the commit 776b501 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #525 +/- ## ========================================== - Coverage 55.52% 55.48% -0.04% ========================================== Files 552 552 Lines 59056 59415 +359 ========================================== + Hits 32788 32964 +176 - Misses 22842 23011 +169 - Partials 3426 3440 +14 ``` | [Flag](https://app.codecov.io/gh/kubewharf/katalyst-core/pull/525/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/525/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kubewharf) | `55.48% <ø> (-0.04%)` | :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.

CLAassistant commented 3 months ago

CLA assistant check
All committers have signed the CLA.

caohe commented 3 months ago

@Airren Thanks for your contribution! This proposal is very detailed and well thought out. Could you sign the CLA required by the community?

Airren commented 3 months ago

@Airren Thanks for your contribution! This proposal is very detailed and well thought out. Could you sign the CLA required by the community?

sure, I have finished the CLA signed.