CLDSRV-986: Pass overhead fields when abortMPU cleans up after completeMPU (hotfix/9.0.32) - #6271
Conversation
…teMPU The cleanup deletes the object or version an earlier CompleteMultipartUpload created; without the overhead fields the resulting oplog entry carries no owner id, which makes SUR hit an assert. (cherry picked from commit 4c27883)
Hello dvasilas,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 expected to find at least:
Please check the |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files
... and 1 file with indirect coverage changes @@ Coverage Diff @@
## hotfix/9.0.32 #6271 +/- ##
=================================================
- Coverage 83.38% 83.37% -0.01%
=================================================
Files 189 189
Lines 12199 12200 +1
=================================================
Hits 10172 10172
- Misses 2027 2028 +1
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
/approve |
Build failedThe build for commit did not succeed in branch bugfix/CLDSRV-986-abortmpu-delete-overhead-fields-9.0.32 The following options are set: approve |
Issue: CLDSRV-806 (cherry picked from commit 55ea857)
e5525e2 to
4ffb60a
Compare
ConflictThere is a conflict between your branch Please resolve the conflict on the feature branch ( git fetch && \
git checkout origin/bugfix/CLDSRV-986-abortmpu-delete-overhead-fields-9.0.32 && \
git merge origin/hotfix/9.0.32Resolve merge conflicts and commit git push origin HEAD:bugfix/CLDSRV-986-abortmpu-delete-overhead-fields-9.0.32The following options are set: approve |
4ffb60a to
4486986
Compare
Build failedThe build for commit did not succeed in branch bugfix/CLDSRV-986-abortmpu-delete-overhead-fields-9.0.32 The following options are set: approve |
The test was flaky due to stricter tolerance on load distribution across KMIP nodes. Enhanced error message to include deviation details for better debugging. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> (cherry picked from commit 95a57b3)
4486986 to
ebd2abb
Compare
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue CLDSRV-986. Goodbye dvasilas. The following options are set: approve |
Cherry-picked from #6268.