Skip to content
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

[proposal] move LLC isolation plugin as runtime hooks plugin #756

Closed
Tracked by #174
zwzhang0107 opened this issue Oct 28, 2022 · 1 comment · Fixed by #1798
Closed
Tracked by #174

[proposal] move LLC isolation plugin as runtime hooks plugin #756

zwzhang0107 opened this issue Oct 28, 2022 · 1 comment · Fixed by #1798
Labels
area/koordlet help wanted Extra attention is needed kind/proposal Create a report to help us improve
Milestone

Comments

@zwzhang0107
Copy link
Contributor

zwzhang0107 commented Oct 28, 2022

What is your proposal:
Plugins in qos manager can be classified into two types: static and dynamic.

For dynamic type, pod will be adjusted dynamically according to node running state, such as cpu suppress, cpu burst, memory evict. Dynamic type strategies has been refactored as plugin in #1500.

For static type, advanced parameters like NodeSLO will be injected according to pod QoS and Priority, such as cpu qos, memory qos,

Why is this needed:
move static type to runtime hook to make sure that qos parameters in can be set ASAP once container created.

Is there a suggested solution, if so, please add it:

@zwzhang0107 zwzhang0107 added the kind/proposal Create a report to help us improve label Oct 28, 2022
@zwzhang0107 zwzhang0107 changed the title [proposal] move LLC plugin to qos-manager [proposal] move LLC isolation plugin to qos-manager Oct 28, 2022
@zwzhang0107
Copy link
Contributor Author

/area koordlet
/assign stormgbs
/milestone v1.1

@zwzhang0107 zwzhang0107 changed the title [proposal] move LLC isolation plugin to qos-manager [proposal] move LLC isolation plugin as runtime hooks plugin Oct 28, 2022
@zwzhang0107 zwzhang0107 added the help wanted Extra attention is needed label Jul 27, 2023
@zwzhang0107 zwzhang0107 added this to the someday milestone Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/koordlet help wanted Extra attention is needed kind/proposal Create a report to help us improve
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants