Skip to content

Conversation

@nellh
Copy link
Contributor

@nellh nellh commented Nov 3, 2025

The regular upload API would repopulate dropped annex objects. This will skip objects unless the size has changed. Ideally we would only skip if the checksum was different but this requires moving to the git based uploader for the web app. This quick fix will prevent repopulating these objects and the web uploader is already skipping files of the same size otherwise.

@codecov
Copy link

codecov bot commented Nov 3, 2025

Codecov Report

❌ Patch coverage is 27.27273% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.09%. Comparing base (6e49fd4) to head (662b5a3).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
...ervices/datalad/datalad_service/handlers/upload.py 27.27% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3617      +/-   ##
==========================================
- Coverage   43.09%   43.09%   -0.01%     
==========================================
  Files         635      635              
  Lines       33604    33615      +11     
  Branches     1516     1516              
==========================================
+ Hits        14483    14486       +3     
- Misses      18982    18990       +8     
  Partials      139      139              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@nellh nellh merged commit 62f86de into master Nov 4, 2025
13 of 16 checks passed
@nellh nellh deleted the web-uploader-dropped-object-fix branch November 4, 2025 22:35
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