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

chore: fix delete backup hang #8321

Merged
merged 1 commit into from
Oct 23, 2024
Merged

Conversation

wangyelei
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines. label Oct 23, 2024
@apecloud-bot apecloud-bot added the approved PR Approved Test label Oct 23, 2024
@wangyelei wangyelei merged commit 4334926 into release-0.9 Oct 23, 2024
38 checks passed
@wangyelei wangyelei deleted the bugfix/backup_delete_hang branch October 23, 2024 08:58
@github-actions github-actions bot added this to the Release 0.9.2 milestone Oct 23, 2024
Copy link

codecov bot commented Oct 23, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 64.95%. Comparing base (c59de8c) to head (1a435bf).
Report is 48 commits behind head on release-0.9.

Files with missing lines Patch % Lines
pkg/dataprotection/backup/deleter.go 50.00% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           release-0.9    #8321      +/-   ##
===============================================
- Coverage        65.20%   64.95%   -0.26%     
===============================================
  Files              356      358       +2     
  Lines            44731    45308     +577     
===============================================
+ Hits             29169    29429     +260     
- Misses           12979    13274     +295     
- Partials          2583     2605      +22     
Flag Coverage Δ
unittests 64.95% <50.00%> (-0.26%) ⬇️

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved PR Approved Test size/XS Denotes a PR that changes 0-9 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants