Skip to content

CLDSRV-902: calculate final checksum from parts#6169

Open
leif-scality wants to merge 2 commits into
improvement/CLDSRV-903-prettier-lint-filesfrom
improvement/CLDSRV-902-calculate-final-checksum-from-parts
Open

CLDSRV-902: calculate final checksum from parts#6169
leif-scality wants to merge 2 commits into
improvement/CLDSRV-903-prettier-lint-filesfrom
improvement/CLDSRV-902-calculate-final-checksum-from-parts

Conversation

@leif-scality
Copy link
Copy Markdown
Contributor

@leif-scality leif-scality commented May 15, 2026

  • calculate final object checksum from parts

@claude
Copy link
Copy Markdown

claude Bot commented May 15, 2026

LGTM

Review by Claude Code

@codecov
Copy link
Copy Markdown

codecov Bot commented May 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.26%. Comparing base (93efed1) to head (8754b9a).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

Files with missing lines Coverage Δ
lib/api/apiUtils/integrity/crcCombine.js 100.00% <100.00%> (ø)
lib/api/apiUtils/integrity/validateChecksums.js 99.04% <100.00%> (+0.05%) ⬆️

... and 1 file with indirect coverage changes

@@                              Coverage Diff                               @@
##           improvement/CLDSRV-903-prettier-lint-files    #6169      +/-   ##
==============================================================================
+ Coverage                                       85.17%   85.26%   +0.09%     
==============================================================================
  Files                                             207      208       +1     
  Lines                                           13796    13889      +93     
==============================================================================
+ Hits                                            11751    11843      +92     
- Misses                                           2045     2046       +1     
Flag Coverage Δ
file-ft-tests 68.91% <5.37%> (-0.43%) ⬇️
kmip-ft-tests 28.18% <5.37%> (-0.16%) ⬇️
mongo-v0-ft-tests 70.08% <5.37%> (-0.44%) ⬇️
mongo-v1-ft-tests 70.07% <5.37%> (-0.46%) ⬇️
multiple-backend 36.52% <5.37%> (-0.21%) ⬇️
sur-tests 35.52% <5.37%> (-0.21%) ⬇️
sur-tests-inflights 37.41% <5.37%> (-0.25%) ⬇️
unit 72.02% <100.00%> (+0.18%) ⬆️
utapi-v2-tests 34.65% <5.37%> (-0.33%) ⬇️

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.

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.

1 participant