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

Update du/dd progress on completion #8608

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Lyndon-Li
Copy link
Contributor

Fix issue #8497, update du/dd progress on completion

@Lyndon-Li Lyndon-Li force-pushed the update-du-dd-progress-when-terminal-event-is-missing branch from 05858fa to 411469b Compare January 13, 2025 10:33
@Lyndon-Li Lyndon-Li force-pushed the update-du-dd-progress-when-terminal-event-is-missing branch 2 times, most recently from 3eae9b6 to 7bbef23 Compare January 13, 2025 11:04
@Lyndon-Li Lyndon-Li force-pushed the update-du-dd-progress-when-terminal-event-is-missing branch from 7bbef23 to 97ce566 Compare January 13, 2025 11:18
Copy link

codecov bot commented Jan 13, 2025

Codecov Report

Attention: Patch coverage is 69.04762% with 13 lines in your changes missing coverage. Please review.

Project coverage is 59.16%. Comparing base (5f7bf64) to head (97ce566).

Files with missing lines Patch % Lines
pkg/uploader/provider/restic.go 66.66% 5 Missing ⚠️
pkg/datapath/micro_service_watcher.go 69.23% 2 Missing and 2 partials ⚠️
pkg/uploader/provider/kopia.go 60.00% 4 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8608   +/-   ##
=======================================
  Coverage   59.15%   59.16%           
=======================================
  Files         370      370           
  Lines       39777    39789   +12     
=======================================
+ Hits        23532    23543   +11     
  Misses      14747    14747           
- Partials     1498     1499    +1     

☔ 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant