Skip to content

storage: remove unnecessary vmm related crates#1944

Draft
imeoer wants to merge 1 commit into
dragonflyoss:masterfrom
imeoer:storage-crate-clean
Draft

storage: remove unnecessary vmm related crates#1944
imeoer wants to merge 1 commit into
dragonflyoss:masterfrom
imeoer:storage-crate-clean

Conversation

@imeoer

@imeoer imeoer commented May 15, 2026

Copy link
Copy Markdown
Collaborator

To keep the storage crate tidy.

Overview

Please briefly describe the changes your pull request makes.

Related Issues

Please link to the relevant issue. For example: Fix #123 or Related #456.

Change Details

Please describe your changes in detail:

Test Results

If you have any relevant screenshots or videos that can help illustrate your changes, please add them here.

Change Type

Please select the type of change your pull request relates to:

  • Bug Fix
  • Feature Addition
  • Documentation Update
  • Code Refactoring
  • Performance Improvement
  • Other (please describe)

Self-Checklist

Before submitting a pull request, please ensure you have completed the following:

  • I have run a code style check and addressed any warnings/errors.
  • I have added appropriate comments to my code (if applicable).
  • I have updated the documentation (if applicable).
  • I have written appropriate unit tests.

@imeoer imeoer force-pushed the storage-crate-clean branch from 197b9fc to bd70ac4 Compare May 15, 2026 14:10
To keep the storage crate tidy.

Signed-off-by: yansong.ys <yansong.ys@antgroup.com>
@imeoer imeoer force-pushed the storage-crate-clean branch from bd70ac4 to 0e5721e Compare May 18, 2026 02:42
@codecov

codecov Bot commented May 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 34.48276% with 57 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.87%. Comparing base (6c11431) to head (0e5721e).

Files with missing lines Patch % Lines
rafs/src/fs.rs 0.00% 46 Missing ⚠️
storage/src/volatile.rs 65.00% 7 Missing ⚠️
storage/src/device.rs 0.00% 3 Missing ⚠️
storage/src/utils.rs 90.90% 0 Missing and 1 partial ⚠️

❌ Your patch check has failed because the patch coverage (34.48%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1944      +/-   ##
==========================================
+ Coverage   67.71%   71.87%   +4.15%     
==========================================
  Files         208      209       +1     
  Lines       60352    60418      +66     
  Branches    53132    53198      +66     
==========================================
+ Hits        40866    43423    +2557     
+ Misses      17753    15007    -2746     
- Partials     1733     1988     +255     
Files with missing lines Coverage Δ
rafs/src/prefetch.rs 69.38% <ø> (ø)
storage/src/backend/localdisk.rs 49.80% <ø> (ø)
storage/src/backend/localfs.rs 88.44% <ø> (ø)
storage/src/backend/mod.rs 85.58% <ø> (ø)
storage/src/cache/cachedfile.rs 44.64% <ø> (-0.07%) ⬇️
storage/src/cache/dedup/db.rs 78.07% <ø> (ø)
storage/src/cache/dedup/mod.rs 89.54% <ø> (ø)
storage/src/cache/dummycache.rs 93.83% <ø> (ø)
storage/src/cache/filecache/mod.rs 69.90% <100.00%> (ø)
storage/src/cache/mod.rs 57.96% <ø> (ø)
... and 12 more

... and 31 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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