Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added option to enable data verification during migration #1415

Merged
merged 1 commit into from
Apr 1, 2024
Merged

Added option to enable data verification during migration #1415

merged 1 commit into from
Apr 1, 2024

Conversation

peng9808
Copy link
Collaborator

@peng9808 peng9808 commented Mar 8, 2024

What this PR does / why we need it:

#1398

Special notes for your reviewer:

Added an optional parameter to provide validation

Does this PR introduce a user-facing change?

NONE

Signed-off-by: peng9808 <peng.lai@daocloud.io>
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 28.67%. Comparing base (8024fee) to head (2713559).
Report is 1 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1415      +/-   ##
==========================================
- Coverage   28.74%   28.67%   -0.07%     
==========================================
  Files          34       34              
  Lines        3079     3079              
==========================================
- Hits          885      883       -2     
- Misses       2077     2078       +1     
- Partials      117      118       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SSmallMonster SSmallMonster merged commit 4f623b4 into hwameistor:main Apr 1, 2024
3 checks passed
@peng9808 peng9808 deleted the lvmigrate branch April 2, 2024 05:58
@peng9808 peng9808 restored the lvmigrate branch April 2, 2024 05:58
@peng9808 peng9808 deleted the lvmigrate branch April 2, 2024 05:58
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.

3 participants