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

[resmgr RDT+blkio 2/2]: set initial classes adjustment. #479

Closed

Conversation

klihub
Copy link
Collaborator

@klihub klihub commented Mar 11, 2025

Note #1: this PR is stacked on #478.
Note #2: this PR shouldn't be merged before a fix is merged to containerd to interpret an initial empty blockio config file as a soft disable for blockio configuration.

The current code sets RDT and block I/O for container updates, but not for initial adjustment. This PR implements the missing adjustment bits.

klihub added 2 commits March 11, 2025 08:29
Resolve default default RDT class (/PodQos) to the pods QoS class
when genering an initial sync response.

Signed-off-by: Krisztian Litkey <[email protected]>
Set up initial RDT and block I/O classes on creation. Resolve
classes identically to updates.

Signed-off-by: Krisztian Litkey <[email protected]>
@klihub klihub changed the title [resmgr 2/2]: set initial RDT and block I/O classes in container adjustment. [resmgr RDT+blkio 2/2]: set initial classes adjustment. Mar 11, 2025
@klihub
Copy link
Collaborator Author

klihub commented Mar 12, 2025

Closing in favor of #481 and #482.

@klihub klihub closed this Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant