Skip to content

feat(metaserver): add resource package manager to meta server#984

Merged
luomingmeng merged 4 commits intokubewharf:mainfrom
XudongLiuHarold:dev/xudongl/add-resource-package-manager-to-meta-server
Oct 27, 2025
Merged

feat(metaserver): add resource package manager to meta server#984
luomingmeng merged 4 commits intokubewharf:mainfrom
XudongLiuHarold:dev/xudongl/add-resource-package-manager-to-meta-server

Conversation

@XudongLiuHarold
Copy link
Copy Markdown
Collaborator

What type of PR is this?

  • add resource package manager interface in meta server

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for your reviewer:

@XudongLiuHarold XudongLiuHarold force-pushed the dev/xudongl/add-resource-package-manager-to-meta-server branch 3 times, most recently from 4040057 to ec56bb7 Compare September 19, 2025 23:27
@XudongLiuHarold XudongLiuHarold force-pushed the dev/xudongl/add-resource-package-manager-to-meta-server branch 4 times, most recently from 529e307 to 6282d4b Compare September 22, 2025 23:31
@XudongLiuHarold XudongLiuHarold marked this pull request as ready for review September 22, 2025 23:31
@XudongLiuHarold XudongLiuHarold force-pushed the dev/xudongl/add-resource-package-manager-to-meta-server branch from 6282d4b to 416122c Compare September 22, 2025 23:32
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 22, 2025

Codecov Report

❌ Patch coverage is 76.61692% with 47 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.71%. Comparing base (85ac1d3) to head (b65aaf0).
⚠️ Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
pkg/metaserver/npd/fetcher.go 74.46% 8 Missing and 4 partials ⚠️
pkg/util/resource-package/convert.go 87.17% 5 Missing and 5 partials ⚠️
pkg/metaserver/metaserver.go 18.18% 8 Missing and 1 partial ⚠️
pkg/util/cnr.go 33.33% 6 Missing and 2 partials ⚠️
pkg/metaserver/resourcepackage/manager.go 73.68% 3 Missing and 2 partials ⚠️
pkg/util/cnr_topology.go 50.00% 1 Missing and 1 partial ⚠️
...isor/plugin/qosaware/reporter/headroom_reporter.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #984      +/-   ##
==========================================
+ Coverage   58.68%   59.71%   +1.03%     
==========================================
  Files         679      682       +3     
  Lines       77274    63844   -13430     
==========================================
- Hits        45347    38125    -7222     
+ Misses      27540    21315    -6225     
- Partials     4387     4404      +17     
Flag Coverage Δ
unittest 59.71% <76.61%> (+1.03%) ⬆️

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.

@XudongLiuHarold XudongLiuHarold added the workflow/need-review review: test succeeded, need to review label Sep 23, 2025
@XudongLiuHarold XudongLiuHarold force-pushed the dev/xudongl/add-resource-package-manager-to-meta-server branch from 416122c to 45d59bb Compare September 23, 2025 00:08
@XudongLiuHarold XudongLiuHarold force-pushed the dev/xudongl/add-resource-package-manager-to-meta-server branch 10 times, most recently from 6e72604 to 378e8ee Compare September 26, 2025 23:09
@XudongLiuHarold XudongLiuHarold force-pushed the dev/xudongl/add-resource-package-manager-to-meta-server branch from 378e8ee to 49ee0d8 Compare September 26, 2025 23:23
@XudongLiuHarold XudongLiuHarold force-pushed the dev/xudongl/add-resource-package-manager-to-meta-server branch 12 times, most recently from 1b7637c to 8018036 Compare October 23, 2025 21:36
@XudongLiuHarold XudongLiuHarold force-pushed the dev/xudongl/add-resource-package-manager-to-meta-server branch 2 times, most recently from 5ea5efe to 5d9c47e Compare October 24, 2025 00:03
Signed-off-by: xudong.l <xudong.l@bytedance.com>
Signed-off-by: xudong.l <xudong.l@bytedance.com>
Signed-off-by: xudong.l <xudong.l@bytedance.com>
@XudongLiuHarold XudongLiuHarold force-pushed the dev/xudongl/add-resource-package-manager-to-meta-server branch from 5d9c47e to 7c47e82 Compare October 24, 2025 00:20
Signed-off-by: xudong.l <xudong.l@bytedance.com>
@luomingmeng luomingmeng merged commit 4bbf9a0 into kubewharf:main Oct 27, 2025
11 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.

2 participants