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

16 EiB free storage space after decreasing MaxStagingDealsBytes #1297

Closed
7 of 12 tasks
web3-providers opened this issue Mar 17, 2023 · 2 comments · Fixed by #1315
Closed
7 of 12 tasks

16 EiB free storage space after decreasing MaxStagingDealsBytes #1297

web3-providers opened this issue Mar 17, 2023 · 2 comments · Fixed by #1315
Assignees
Labels

Comments

@web3-providers
Copy link

Checklist

  • This is not a question or a support request. If you have any boost related questions, please ask in the discussion forum.
  • This is not a new feature request. If it is, please file a feature request instead.
  • This is not an enhancement request. If it is, please file a improvement suggestion instead.
  • I have searched on the issue tracker and the discussion forum, and there is no existing related issue or discussion.
  • I am running the Latest release, or the most recent RC(release canadiate) for the upcoming release or the dev branch(master), or have an issue updating to any of these.
  • I did not make any code changes to boost.

Boost component

  • boost daemon - storage providers
  • boost client
  • boost UI
  • boost data-transfer
  • boost index-provider
  • Other

Boost Version

boostd version 1.6.0+git.aced70f

Describe the Bug

reducing the MaxStagingDealsBytes to from 1500000000000 to 750000000000 after boost did already accept incomming files up to 1.5TB lead to this strange effect (Free Space 16 EiB ) :

image

Logging Information

Non that I find related

Repo Steps

  1. Run '...'
  2. Do '...'
  3. See error '...'
    ...
@LexLuthr
Copy link
Collaborator

@jacobheun Can you please update your explanation of the bug here?

@jacobheun
Copy link
Contributor

I believe the bug is in the calculation at https://github.com/filecoin-project/boost/blob/v1.6.1-rc1/storagemanager/storagemanager.go#L65. The tagged storage is over the new value set in the config, which is likely causing the uint overflow to come back as 16EiB.

We should add a unit test around that and then just return 0 if the tagged amount is greater. This should only occur if the MaxStagingDealsBytes is set lower than currently tagged space, but you'll just keep accepting deals if space is the only potential limited in the deal filters.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants