Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test #2604

Draft
wants to merge 6 commits into
base: bugfix/BB-641
Choose a base branch
from
Draft

test #2604

wants to merge 6 commits into from

Conversation

francoisferrand
Copy link
Contributor

  • gha: bump codecov@v5
  • Fix dockerfile lint issues
  • Do not delete Zenko object when not in OOB
  • Use x-amz-meta-scal-version-id when ingesting object
  • debug
  • Properly handle archive
  • fix codecov notification

Make OOB location-aware, and thus do not delete the Zenko object when it
gets removed from S3C by gc at the end of the transition.

Issue: BB-590
This custom (but reserved) user metadata field is set by Cloudserver
when restoring cold object in OOB bucket.

Issue: BB-590
Copy link

codecov bot commented Dec 20, 2024

Codecov Report

Attention: Patch coverage is 69.38776% with 15 lines in your changes missing coverage. Please review.

Project coverage is 55.44%. Comparing base (ca50af2) to head (04886d4).

Files with missing lines Patch % Lines
extensions/mongoProcessor/MongoQueueProcessor.js 69.38% 15 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

Files with missing lines Coverage Δ
extensions/mongoProcessor/MongoQueueProcessor.js 64.68% <69.38%> (+1.96%) ⬆️

... and 2 files with indirect coverage changes

Components Coverage Δ
Bucket Notification 18.51% <ø> (ø)
Core Library 62.05% <ø> (+0.13%) ⬆️
Ingestion 67.86% <69.38%> (+0.33%) ⬆️
Lifecycle 47.09% <ø> (ø)
Oplog Populator 84.20% <ø> (ø)
Replication 51.01% <ø> (ø)
Bucket Scanner 85.60% <ø> (ø)
@@                Coverage Diff                @@
##           bugfix/BB-641    #2604      +/-   ##
=================================================
+ Coverage          55.34%   55.44%   +0.10%     
=================================================
  Files                198      198              
  Lines              12925    12949      +24     
=================================================
+ Hits                7153     7180      +27     
+ Misses              5762     5759       -3     
  Partials              10       10              
Flag Coverage Δ
api:retry 9.60% <0.00%> (-0.02%) ⬇️
api:routes 9.50% <0.00%> (-0.02%) ⬇️
bucket-scanner 85.60% <ø> (ø)
ingestion 12.66% <69.38%> (+0.19%) ⬆️
lib 7.50% <0.00%> (-0.03%) ⬇️
lifecycle 19.38% <0.00%> (-0.06%) ⬇️
notification 0.88% <0.00%> (-0.01%) ⬇️
replication 18.84% <0.00%> (-0.04%) ⬇️
unit 5.12% <0.00%> (-0.01%) ⬇️

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

@francoisferrand francoisferrand force-pushed the improvement/BB-590 branch 4 times, most recently from f8b4d35 to 92350c3 Compare December 20, 2024 15:59
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