Skip to content

INTEGRATION [PR#5855 > development/9.1] CLDSRV-674: handle cross-account bucket policies#5886

Merged
bert-e merged 12 commits intodevelopment/9.1from
w/9.1/bugfix/CLDSRV-674
Jul 23, 2025
Merged

INTEGRATION [PR#5855 > development/9.1] CLDSRV-674: handle cross-account bucket policies#5886
bert-e merged 12 commits intodevelopment/9.1from
w/9.1/bugfix/CLDSRV-674

Conversation

@bert-e
Copy link
Copy Markdown
Contributor

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

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

Do not edit this pull request directly.
If you need to amend/cancel the changeset on branch
w/9.1/bugfix/CLDSRV-674, please follow this
procedure:

 $ git fetch
 $ git checkout w/9.1/bugfix/CLDSRV-674
 $ # <amend or cancel the changeset by _adding_ new commits>
 $ git push origin w/9.1/bugfix/CLDSRV-674

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

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 23, 2025

Codecov Report

Attention: Patch coverage is 93.47826% with 6 lines in your changes missing coverage. Please review.

Project coverage is 83.20%. Comparing base (90f0dfb) to head (fa33ce2).

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
lib/api/apiUtils/authorization/permissionChecks.js 92.30% 6 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

Files with missing lines Coverage Δ
lib/api/api.js 89.52% <100.00%> (+0.05%) ⬆️
lib/api/objectGet.js 86.14% <ø> (ø)
lib/metadata/metadataUtils.js 95.23% <100.00%> (+0.45%) ⬆️
lib/api/apiUtils/authorization/permissionChecks.js 94.35% <92.30%> (+2.56%) ⬆️

... and 1 file with indirect coverage changes

@@                 Coverage Diff                 @@
##           development/9.1    #5886      +/-   ##
===================================================
+ Coverage            83.03%   83.20%   +0.16%     
===================================================
  Files                  188      188              
  Lines                11992    12055      +63     
===================================================
+ Hits                  9958    10030      +72     
+ Misses                2034     2025       -9     
Flag Coverage Δ
ceph-backend-test 64.18% <47.82%> (-0.09%) ⬇️
file-ft-tests 66.01% <47.82%> (-0.10%) ⬇️
kmip-ft-tests 26.95% <9.78%> (-0.09%) ⬇️
mongo-v0-ft-tests 67.86% <47.82%> (-0.14%) ⬇️
mongo-v1-ft-tests 67.86% <47.82%> (-0.11%) ⬇️
multiple-backend 33.96% <17.39%> (-0.07%) ⬇️
sur-tests 34.52% <9.78%> (-1.00%) ⬇️
sur-tests-inflights 36.49% <9.78%> (-0.13%) ⬇️
unit 67.12% <85.86%> (+0.14%) ⬆️
utapi-v2-tests 33.30% <15.21%> (-0.08%) ⬇️

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.

@bert-e bert-e merged commit fa33ce2 into development/9.1 Jul 23, 2025
120 of 127 checks passed
@bert-e bert-e deleted the w/9.1/bugfix/CLDSRV-674 branch July 23, 2025 14:11
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.

2 participants