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

Potential Issue With the AutoPrune SpiltStore Feature in v1.17.1-rc3 #9236

Closed
8 of 18 tasks
TippyFlitsUK opened this issue Aug 31, 2022 · 7 comments
Closed
8 of 18 tasks
Assignees
Labels
kind/bug Kind: Bug need/team-input Hint: Needs Team Input P2 P2: Should be resolved splitstore

Comments

@TippyFlitsUK
Copy link
Contributor

TippyFlitsUK commented Aug 31, 2022

Checklist

  • This is not a security-related bug/issue. If it is, please follow please follow the security policy.
  • This is not a question or a support request. If you have any lotus related questions, please ask in the lotus 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 lotus 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 lotus.

Lotus component

  • lotus daemon - chain sync
  • lotus miner - mining and block production
  • lotus miner/worker - sealing
  • lotus miner - proving(WindowPoSt)
  • lotus miner/market - storage deal
  • lotus miner/market - retrieval deal
  • lotus miner/market - data transfer
  • lotus client
  • lotus JSON-RPC API
  • lotus message management (mpool)
  • Other

Lotus Version

lotus version 1.17.1-rc3+mainnet+git.079dab0d7

Describe the Bug

TSEs are tracking a potential issue with the new EnableColdStoreAutoPrune feature.

Lotus is configured with the following settings:

[Chainstore]
 EnableSplitstore = true
 ColdStoreType = "universal"
 EnableColdStoreAutoPrune = true

We are currently not observing any noticeable automatic pruning of the chain taking place on Lotus nodes with EnableColdStoreAutoPrune enabled and ColdStoreType set to universal. This behaviour has been confirmed by multiple SPs currently taking part in SplitStore testing.

I have attached an hourly log of folder sizes for both /.lotus/datastore/splitstore and /.lotus/datastore/chain directories below dating back to 10th August 2022.

https://github.com/filecoin-project/lotus/files/9461826/splitstore.log

Logging Information

https://github.com/filecoin-project/lotus/files/9461826/splitstore.log

Repo Steps

Set the following in /.lotus/config.toml:

[Chainstore]
 EnableSplitstore = true
 ColdStoreType = "universal"
 EnableColdStoreAutoPrune = true

Run Lotus daemon and monitor /.lotus/datastore/chain folder size.

@TippyFlitsUK
Copy link
Contributor Author

@ZenGround0
Copy link
Contributor

Unfortunately it looks like I miscommunicated the configuration

It should be

[Chainstore]
 EnableSplitstore = true
[Chainstore.Splitstore]
 ColdStoreType = "universal"
 EnableColdStoreAutoPrune = true

I'm not sure if lotus config is smart enough to match fields in the top level to matching fields in sublevels but I'm suspicious that it is not. I think this would explain the behavior you are seeing because universal is the default coldstore type and false is the default for enabling auto prune of the cold store.

@jennijuju
Copy link
Member

should we close this?

@TippyFlitsUK
Copy link
Contributor Author

This is still happening even with the updated config.toml @jennijuju.

@TippyFlitsUK TippyFlitsUK added P2 P2: Should be resolved and removed P1 P1: Must be resolved labels Sep 17, 2022
@TippyFlitsUK
Copy link
Contributor Author

Updated logs taken after the config.toml update @ZenGround0 👍.

https://drive.google.com/drive/folders/1rxjPOt4otb4dSuZHBWen_rlvBPHnyqCM?usp=sharing

@mur-me
Copy link

mur-me commented Oct 31, 2022

Thanks for this feature guys, looking forward for this to use it! ❤️

@rjan90
Copy link
Contributor

rjan90 commented Sep 19, 2023

I think its fair to close this issue now @TippyFlitsUK given that EnableColdStoreAutoPrune has been deprecated?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Kind: Bug need/team-input Hint: Needs Team Input P2 P2: Should be resolved splitstore
Projects
None yet
Development

No branches or pull requests

5 participants