Skip to content

feat(sriov): skip dynamic VF allocation by pod annotations#1112

Open
junyu-peng wants to merge 1 commit intokubewharf:mainfrom
junyu-peng:dev/sriov
Open

feat(sriov): skip dynamic VF allocation by pod annotations#1112
junyu-peng wants to merge 1 commit intokubewharf:mainfrom
junyu-peng:dev/sriov

Conversation

@junyu-peng
Copy link
Copy Markdown
Collaborator

add pod filter to skip VF allocation for pods using host network update test cases to verify host network pod filtering

What type of PR is this?

Enhancements

add filter for pods to skip VF allocation

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for your reviewer:

@junyu-peng junyu-peng requested a review from luomingmeng as a code owner March 30, 2026 06:42
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

❌ Patch coverage is 73.33333% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.49%. Comparing base (95c2c56) to head (62d13d4).
⚠️ Report is 12 commits behind head on main.

Files with missing lines Patch % Lines
pkg/agent/qrm-plugins/sriov/policy/dynamic.go 74.07% 5 Missing and 2 partials ⚠️
cmd/katalyst-agent/app/options/qrm/sriov_plugin.go 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1112      +/-   ##
==========================================
+ Coverage   61.46%   61.49%   +0.02%     
==========================================
  Files         783      783              
  Lines       72974    73186     +212     
==========================================
+ Hits        44853    45005     +152     
- Misses      23139    23173      +34     
- Partials     4982     5008      +26     
Flag Coverage Δ
unittest 61.49% <73.33%> (+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.

@junyu-peng junyu-peng force-pushed the dev/sriov branch 2 times, most recently from 7f49885 to 348cd62 Compare April 1, 2026 09:45
@junyu-peng junyu-peng changed the title feat(sriov): skip VF allocation for host network pods (WIP)feat(sriov): skip VF allocation for host network pods Apr 3, 2026
@junyu-peng junyu-peng changed the title (WIP)feat(sriov): skip VF allocation for host network pods (WIP): skip VF allocation for host network pods Apr 3, 2026
@junyu-peng junyu-peng changed the title (WIP): skip VF allocation for host network pods feat(sriov): skip VF allocation for host network pods Apr 3, 2026
@junyu-peng junyu-peng changed the title feat(sriov): skip VF allocation for host network pods feat(sriov): skip dynamic VF allocation by pod annotations Apr 8, 2026
@junyu-peng junyu-peng added the workflow/need-review review: test succeeded, need to review label Apr 13, 2026
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.

1 participant