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

Retain procedure appears to crash or get canceled when it fails to delete a piece #6482

Closed
thepaul opened this issue Nov 6, 2023 · 3 comments
Assignees
Labels
Bug Something isn't working

Comments

@thepaul
Copy link
Contributor

thepaul commented Nov 6, 2023

When we are running a Retain operation, and we fail to delete a piece, we should continue with deleting the rest of the pieces, rather than crashing or quitting.

See, in particular, https://forum.storj.io/t/us1-storj-io-disk-average-30-lower-than-actual-disk-usage/24197/16

@thepaul thepaul added the Bug Something isn't working label Nov 6, 2023
@thepaul thepaul self-assigned this Nov 6, 2023
@storj-gerrit
Copy link

storj-gerrit bot commented Nov 6, 2023

Change storagenode/retain: don't quit on error mentions this issue.

@storjrobot
Copy link

This issue has been mentioned on Storj Community Forum (official). There might be relevant details there:

https://forum.storj.io/t/us1-storj-io-disk-average-30-lower-than-actual-disk-usage/24197/18

storjBuildBot pushed a commit that referenced this issue Nov 6, 2023
It has been noted in the forum that, during a Retain operation, when a
piece can't be deleted, the process never completes. The error is
written to the log, but the completion line "Moved pieces to trash
during retain" never is.

This `return` line is the reason. We should instead continue the loop.

Change-Id: I0f51d34aba0e81ad60a75802069b42dc135ad907
Refs: #6482
@iglesiasbrandon
Copy link
Contributor

closing this issue because the fix has been deployed to production. @thepaul @shaupt131

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
Status: Done/Deployed
Development

No branches or pull requests

3 participants