Skip to content

nydusify: support target backend for copy subcommand#1930

Draft
imeoer wants to merge 1 commit into
dragonflyoss:masterfrom
imeoer:nydusify-copy-target-backend
Draft

nydusify: support target backend for copy subcommand#1930
imeoer wants to merge 1 commit into
dragonflyoss:masterfrom
imeoer:nydusify-copy-target-backend

Conversation

@imeoer
Copy link
Copy Markdown
Collaborator

@imeoer imeoer commented Apr 28, 2026

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 nydusify-copy-target-backend branch from 94fb2d2 to 451aec9 Compare April 28, 2026 12:14
Signed-off-by: yansong.ys <yansong.ys@antgroup.com>
@imeoer imeoer force-pushed the nydusify-copy-target-backend branch from 451aec9 to 293403c Compare April 28, 2026 12:15
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2026

Codecov Report

❌ Patch coverage is 9.72763% with 232 lines in your changes missing coverage. Please review.
✅ Project coverage is 69.96%. Comparing base (460bb74) to head (293403c).

Files with missing lines Patch % Lines
contrib/nydusify/pkg/copier/copier.go 8.73% 227 Missing and 3 partials ⚠️
contrib/nydusify/cmd/nydusify.go 60.00% 1 Missing and 1 partial ⚠️

❌ Your patch check has failed because the patch coverage (9.72%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.
❌ Your project check has failed because the head coverage (69.96%) is below the target coverage (70.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1930      +/-   ##
==========================================
+ Coverage   66.13%   69.96%   +3.82%     
==========================================
  Files         200      200              
  Lines       55859    56858     +999     
  Branches    48639    49403     +764     
==========================================
+ Hits        36941    39779    +2838     
+ Misses      17251    15133    -2118     
- Partials     1667     1946     +279     
Files with missing lines Coverage Δ
contrib/nydusify/cmd/nydusify.go 69.03% <60.00%> (+52.13%) ⬆️
contrib/nydusify/pkg/copier/copier.go 21.17% <8.73%> (+6.89%) ⬆️

... and 35 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