Skip to content

Conversation

@bert-e
Copy link
Contributor

@bert-e bert-e commented Mar 20, 2025

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

Do not edit this pull request directly.
If you need to amend/cancel the changeset on branch
w/9.0/improvement/CLDSRV-623-arsn-bump-perf, please follow this
procedure:

 $ git fetch
 $ git checkout w/9.0/improvement/CLDSRV-623-arsn-bump-perf
 $ # <amend or cancel the changeset by _adding_ new commits>
 $ git push origin w/9.0/improvement/CLDSRV-623-arsn-bump-perf

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

…-perf' into w/8.8/improvement/CLDSRV-623-arsn-bump-perf

 Conflicts:
	lib/api/apiUtils/bucket/getNotificationConfiguration.js
	lib/api/apiUtils/object/locationConstraintCheck.js
	lib/api/apiUtils/object/versioning.js
	lib/api/bucketGet.js
	lib/api/bucketPut.js
	lib/api/objectPut.js
	package.json
	tests/multipleBackend/multipartUpload.js
	tests/unit/api/objectHead.js
	tests/unit/api/objectPut.js
	yarn.lock
To fix duplicated error instanciation for perf
@BourgoisMickael BourgoisMickael force-pushed the w/9.0/improvement/CLDSRV-623-arsn-bump-perf branch from 588ca80 to 8aefbbf Compare March 21, 2025 00:38
@BourgoisMickael BourgoisMickael force-pushed the w/9.0/improvement/CLDSRV-623-arsn-bump-perf branch from 8aefbbf to cf821ae Compare March 21, 2025 14:56
@codecov
Copy link

codecov bot commented Mar 21, 2025

Codecov Report

Attention: Patch coverage is 48.79518% with 85 lines in your changes missing coverage. Please review.

Project coverage is 73.69%. Comparing base (089611f) to head (cf821ae).
Report is 209 commits behind head on development/9.0.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
lib/api/apiUtils/bucket/bucketWebsite.js 4.00% 24 Missing ⚠️
lib/routes/routeBackbeat.js 12.50% 14 Missing ⚠️
lib/api/apiUtils/bucket/bucketCors.js 10.00% 9 Missing ⚠️
lib/api/objectCopy.js 16.66% 5 Missing ⚠️
lib/api/bucketPutVersioning.js 50.00% 3 Missing ⚠️
lib/api/bucketUpdateQuota.js 25.00% 3 Missing ⚠️
lib/api/metadataSearch.js 25.00% 3 Missing ⚠️
lib/api/objectGet.js 25.00% 3 Missing ⚠️
lib/api/apiUtils/bucket/checkPreferredLocations.js 0.00% 2 Missing ⚠️
lib/api/corsPreflight.js 33.33% 2 Missing ⚠️
... and 17 more
Additional details and impacted files

Impacted file tree graph

Files with missing lines Coverage Δ
lib/api/apiUtils/authorization/permissionChecks.js 87.68% <100.00%> (ø)
lib/api/apiUtils/bucket/bucketEncryption.js 71.42% <100.00%> (ø)
lib/api/apiUtils/bucket/validateSearch.js 96.42% <100.00%> (ø)
...piUtils/object/getReplicationBackendDataLocator.js 100.00% <100.00%> (ø)
lib/api/apiUtils/object/locationConstraintCheck.js 100.00% <100.00%> (ø)
lib/api/apiUtils/object/locationHeaderCheck.js 100.00% <100.00%> (ø)
lib/api/apiUtils/object/objectLockHelpers.js 81.73% <100.00%> (ø)
lib/api/apiUtils/object/validateChecksumHeaders.js 100.00% <100.00%> (ø)
lib/api/backbeat/listLifecycleCurrents.js 94.11% <100.00%> (ø)
lib/api/backbeat/listLifecycleNonCurrents.js 94.64% <100.00%> (ø)
... and 35 more
@@               Coverage Diff                @@
##           development/9.0    #5772   +/-   ##
================================================
  Coverage            73.69%   73.69%           
================================================
  Files                  186      186           
  Lines                11718    11719    +1     
================================================
+ Hits                  8635     8636    +1     
  Misses                3083     3083           
Flag Coverage Δ
ceph-backend-test 44.23% <27.71%> (+<0.01%) ⬆️
file-ft-tests 47.51% <31.92%> (+<0.01%) ⬆️
kmip-ft-tests 26.45% <27.10%> (+<0.01%) ⬆️
mongo-v0-ft-tests 47.83% <31.92%> (+<0.01%) ⬆️
mongo-v1-ft-tests 47.85% <31.92%> (+<0.01%) ⬆️
multiple-backend 31.98% <27.71%> (+<0.01%) ⬆️
quota-tests 32.10% <27.71%> (+<0.01%) ⬆️
quota-tests-inflights 34.06% <27.71%> (-0.01%) ⬇️
unit 65.51% <45.18%> (+<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.

@bert-e bert-e merged commit cf821ae into development/9.0 Mar 21, 2025
19 of 20 checks passed
@bert-e bert-e deleted the w/9.0/improvement/CLDSRV-623-arsn-bump-perf branch March 21, 2025 15:27
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.

3 participants