Skip to content

feat(resctrl): add manager for resctrl filesystem operations#1088

Open
luomingmeng wants to merge 1 commit intokubewharf:mainfrom
luomingmeng:dev/refactor-resctrl-hinter
Open

feat(resctrl): add manager for resctrl filesystem operations#1088
luomingmeng wants to merge 1 commit intokubewharf:mainfrom
luomingmeng:dev/refactor-resctrl-hinter

Conversation

@luomingmeng
Copy link
Copy Markdown
Collaborator

What type of PR is this?

Enhancements

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for your reviewer:

@luomingmeng luomingmeng force-pushed the dev/refactor-resctrl-hinter branch from edafed4 to 7a69030 Compare March 10, 2026 02:38
Add resctrl group lifecycle management feature with periodic cleanup of unused groups.
Introduce new manager component to handle resctrl filesystem operations and integrate with existing state.
Add platform-specific scanner implementations to find resctrl mountpoint.
Extend configuration to enable/disable the feature and add tests for new functionality.
@luomingmeng luomingmeng force-pushed the dev/refactor-resctrl-hinter branch from 7a69030 to dbffbe5 Compare March 10, 2026 02:52
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 10, 2026

Codecov Report

❌ Patch coverage is 56.28743% with 73 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.71%. Comparing base (afadd4d) to head (dbffbe5).
⚠️ Report is 206 commits behind head on main.

Files with missing lines Patch % Lines
...rm-plugins/memory/dynamicpolicy/resctrl/manager.go 61.34% 29 Missing and 17 partials ⚠️
...gins/memory/dynamicpolicy/resctrl/scanner_linux.go 0.00% 15 Missing ⚠️
...qrm-plugins/memory/dynamicpolicy/resctrl_hinter.go 64.70% 4 Missing and 2 partials ⚠️
...md/katalyst-agent/app/options/qrm/memory_plugin.go 66.66% 3 Missing ⚠️
...nt/qrm-plugins/memory/dynamicpolicy/state/state.go 50.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1088      +/-   ##
==========================================
- Coverage   60.71%   60.71%   -0.01%     
==========================================
  Files         700      741      +41     
  Lines       66856    69918    +3062     
==========================================
+ Hits        40591    42448    +1857     
- Misses      21715    22680     +965     
- Partials     4550     4790     +240     
Flag Coverage Δ
unittest 60.71% <56.28%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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