Skip to content

do not consider pending blobs on disk space query#3619

Merged
eukreign merged 1 commit intomasterfrom
fix_blob_accounting
Jun 8, 2022
Merged

do not consider pending blobs on disk space query#3619
eukreign merged 1 commit intomasterfrom
fix_blob_accounting

Conversation

@shyba
Copy link
Copy Markdown
Member

@shyba shyba commented Jun 8, 2022

Add a check for status being finished, so we do not count pending/abandoned downloads.
closes #3617

@shyba shyba added type: bug Existing functionality is wrong or broken area: blobs/files labels Jun 8, 2022
@shyba shyba requested a review from eukreign June 8, 2022 14:28
@lbry-bot lbry-bot assigned eukreign and unassigned shyba Jun 8, 2022
@lbry-bot lbry-bot assigned shyba and unassigned eukreign Jun 8, 2022
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.02%) to 57.361% when pulling ec4f9cc on fix_blob_accounting into c04fbb2 on master.

@eukreign eukreign merged commit 582f79b into master Jun 8, 2022
@eukreign eukreign deleted the fix_blob_accounting branch June 8, 2022 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: blobs/files type: bug Existing functionality is wrong or broken

Projects

None yet

Development

Successfully merging this pull request may close these issues.

content_blobs_storage_used_mb seems to include "pending" + "finished" blobs. (Making it not to match the actual disk-usage)

3 participants