Skip to content

INTEGRATION [PR#6222 > development/9.4] Handle BucketAlreadyExists races and bump arsenal for MPU afterEach fix#6234

Open
bert-e wants to merge 5 commits into
development/9.4from
w/9.4/improvement/CLDSRV-919-afterEach-flakiness-internal-error-fix
Open

INTEGRATION [PR#6222 > development/9.4] Handle BucketAlreadyExists races and bump arsenal for MPU afterEach fix#6234
bert-e wants to merge 5 commits into
development/9.4from
w/9.4/improvement/CLDSRV-919-afterEach-flakiness-internal-error-fix

Conversation

@bert-e

@bert-e bert-e commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

This pull request has been created automatically.
It is linked to its parent pull request #6222.

Do not edit this pull request directly.
If you need to amend/cancel the changeset on branch
w/9.4/improvement/CLDSRV-919-afterEach-flakiness-internal-error-fix, please follow this
procedure:

 git fetch
 git checkout w/9.4/improvement/CLDSRV-919-afterEach-flakiness-internal-error-fix
 # <amend or cancel the changeset by _adding_ new commits>
 git push origin w/9.4/improvement/CLDSRV-919-afterEach-flakiness-internal-error-fix

Please always comment pull request #6222 instead of this one.

Treat a concurrent creator as the owner of normal bucket cleanup and return the local minimal MPU bucket without another metadata read.

Issue: CLDSRV-919
Keep the Prettier migration isolated from the functional race fix so reviewers can inspect the behavior independently.

Issue: CLDSRV-919
…-flakiness-internal-error-fix' into w/9.4/improvement/CLDSRV-919-afterEach-flakiness-internal-error-fix
Comment on lines +232 to -243
if (err) {
return callback(err);
}
return callback(null, data);
});
},
},
},
// Function to run upon finishing both parallel requests
(err, results) => {
if (err) {
return cb(err);
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.16129% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.09%. Comparing base (0d08680) to head (639cc42).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
lib/api/apiUtils/bucket/bucketCreation.js 94.91% 3 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

Files with missing lines Coverage Δ
lib/services.js 88.14% <100.00%> (+0.09%) ⬆️
lib/api/apiUtils/bucket/bucketCreation.js 93.54% <94.91%> (+0.15%) ⬆️

... and 1 file with indirect coverage changes

@@                 Coverage Diff                 @@
##           development/9.4    #6234      +/-   ##
===================================================
- Coverage            86.10%   86.09%   -0.01%     
===================================================
  Files                  212      212              
  Lines                14483    14489       +6     
===================================================
+ Hits                 12470    12474       +4     
- Misses                2013     2015       +2     
Flag Coverage Δ
file-ft-tests 70.14% <62.90%> (-0.02%) ⬇️
file-ft-tests-null-compat 70.57% <62.90%> (-0.07%) ⬇️
kmip-ft-tests 27.93% <51.61%> (+<0.01%) ⬆️
mongo-v0-ft-tests 71.17% <66.12%> (-0.06%) ⬇️
mongo-v1-ft-tests 71.17% <62.90%> (-0.03%) ⬇️
multiple-backend 36.13% <50.00%> (-0.01%) ⬇️
s3c-ft-tests-v0 65.12% <75.80%> (-0.02%) ⬇️
s3c-ft-tests-v0-null-compat 65.18% <75.80%> (-0.02%) ⬇️
s3c-ft-tests-v1 65.10% <75.80%> (-0.02%) ⬇️
sur-tests 34.81% <51.61%> (-0.86%) ⬇️
sur-tests-inflights 37.54% <51.61%> (-0.03%) ⬇️
unit 73.03% <95.16%> (+0.02%) ⬆️
utapi-v2-tests 35.10% <64.51%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread lib/api/apiUtils/bucket/bucketCreation.js
Comment thread tests/unit/bucket/bucketCreation.js
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.

4 participants