Skip to content

feat: support sriov vf#1064

Merged
xu282934741 merged 20 commits intokubewharf:mainfrom
junyu-peng:dev/sriov
Mar 26, 2026
Merged

feat: support sriov vf#1064
xu282934741 merged 20 commits intokubewharf:mainfrom
junyu-peng:dev/sriov

Conversation

@junyu-peng
Copy link
Copy Markdown
Collaborator

What type of PR is this?

Features

add sriov vf resource plugin

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for your reviewer:

@junyu-peng junyu-peng added the workflow/need-review review: test succeeded, need to review label Jan 27, 2026
@junyu-peng junyu-peng changed the title feat: add sriov vf resource feat: support sriov vf Jan 27, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 27, 2026

Codecov Report

❌ Patch coverage is 61.17412% with 582 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.10%. Comparing base (8e017a8) to head (541e092).
⚠️ Report is 30 commits behind head on main.

Files with missing lines Patch % Lines
pkg/agent/qrm-plugins/sriov/policy/static.go 59.24% 107 Missing and 12 partials ⚠️
pkg/util/machine/sriov_linux.go 58.25% 65 Missing and 26 partials ⚠️
...agent/qrm-plugins/sriov/handler/statereconciler.go 47.64% 75 Missing and 14 partials ⚠️
pkg/agent/qrm-plugins/sriov/policy/dynamic.go 61.90% 53 Missing and 19 partials ⚠️
.../agent/qrm-plugins/sriov/state/state_checkpoint.go 53.38% 49 Missing and 6 partials ⚠️
pkg/agent/qrm-plugins/sriov/state/state.go 74.83% 34 Missing and 4 partials ⚠️
pkg/agent/qrm-plugins/sriov/state/state_mem.go 60.65% 22 Missing and 2 partials ⚠️
cmd/katalyst-agent/app/options/qrm/sriov_plugin.go 70.90% 16 Missing ⚠️
cmd/katalyst-agent/app/agent/qrm/sriov_plugin.go 12.50% 14 Missing ⚠️
pkg/agent/qrm-plugins/sriov/policy/base.go 87.03% 7 Missing and 7 partials ⚠️
... and 7 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1064      +/-   ##
==========================================
- Coverage   61.10%   61.10%   -0.01%     
==========================================
  Files         739      754      +15     
  Lines       69760    71254    +1494     
==========================================
+ Hits        42628    43540     +912     
- Misses      22383    22862     +479     
- Partials     4749     4852     +103     
Flag Coverage Δ
unittest 61.10% <61.17%> (-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.

Comment thread pkg/agent/qrm-plugins/sriov/handler/statereconciler.go Outdated
Comment thread pkg/agent/qrm-plugins/sriov/handler/statereconciler.go
Comment thread pkg/agent/qrm-plugins/sriov/handler/statereconciler.go Outdated
Comment thread pkg/agent/qrm-plugins/sriov/state/state_mem.go Outdated
Comment thread pkg/agent/qrm-plugins/sriov/policy/dynamic.go Outdated
Comment thread pkg/util/machine/sriov_linux.go
Comment thread pkg/util/machine/sriov_linux.go
luomingmeng
luomingmeng previously approved these changes Mar 24, 2026
@xu282934741 xu282934741 merged commit c811e77 into kubewharf:main Mar 26, 2026
10 checks passed
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