-
Notifications
You must be signed in to change notification settings - Fork 242
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
Support erasing inflights in CompleteMPU and AbortMPU #5719
base: development/8.8
Are you sure you want to change the base?
Conversation
- The action for abort is multipartDelete - Substract the total part size with the stored size in complete MPU API, so we can deduce the size we drop. The locations object does not hold the size. - Same approach for the AbortMPU API. Issue: CLDSRV-592
Issue: CLDSRV-592
Hello williamlardier,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Incorrect fix versionThe
Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:
Please check the |
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files
... and 14 files with indirect coverage changes @@ Coverage Diff @@
## development/8.8 #5719 +/- ##
===================================================
+ Coverage 60.55% 62.90% +2.35%
===================================================
Files 185 185
Lines 11617 11631 +14
===================================================
+ Hits 7035 7317 +282
+ Misses 4582 4314 -268
Flags with carried forward coverage won't be shown. Click here to find out more. |
7af350b
to
f1717f5
Compare
- Not all code is tested, but relevant parts for quota Issue: CLDSRV-592
f1717f5
to
e4df578
Compare
The added test ensure that the logic within the API is coherent, as unit testing is not possible here.