Skip to content

Revise memory headroom calculation to reduce OOM#1128

Merged
xu282934741 merged 2 commits into
kubewharf:mainfrom
jinxin32:memory_headroom
Apr 23, 2026
Merged

Revise memory headroom calculation to reduce OOM#1128
xu282934741 merged 2 commits into
kubewharf:mainfrom
jinxin32:memory_headroom

Conversation

@jinxin32
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:

Related PR: kubewharf/katalyst-api#182

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 20, 2026

Codecov Report

❌ Patch coverage is 93.05556% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.56%. Comparing base (55ffd45) to head (718d604).
⚠️ Report is 8 commits behind head on main.

Files with missing lines Patch % Lines
...esource/memory/headroompolicy/policy_numa_aware.go 93.87% 2 Missing and 1 partial ⚠️
...qos/reclaimedresource/memoryheadroom/util_based.go 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1128      +/-   ##
==========================================
- Coverage   61.57%   61.56%   -0.02%     
==========================================
  Files         783      783              
  Lines       73224    73285      +61     
==========================================
+ Hits        45091    45120      +29     
- Misses      23129    23154      +25     
- Partials     5004     5011       +7     
Flag Coverage Δ
unittest 61.56% <93.05%> (-0.02%) ⬇️

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.

@xu282934741 xu282934741 merged commit 80d9615 into kubewharf:main Apr 23, 2026
26 of 30 checks passed
@jinxin32 jinxin32 deleted the memory_headroom branch April 23, 2026 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workflow/need-review review: test succeeded, need to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants