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

allow storage miners to manage a blocklist of piece CIDs #2069

Merged
merged 7 commits into from
Jun 19, 2020
Merged

allow storage miners to manage a blocklist of piece CIDs #2069

merged 7 commits into from
Jun 19, 2020

Conversation

laser
Copy link
Contributor

@laser laser commented Jun 18, 2020

Why does this PR exist?

Addresses the storage provider side of this request from @whyrusleeping.

What's in this PR?

This PR adds three new commands lotus-storage-miner deals get-blocklist (writes the piece CID blocklist to stdout, one per line, using the specified encoding), lotus-storage-miner deals set-blocklist (replaces the existing storage miner piece CID-blocklist with the contents of the provided file or the CIDs read from stdin), and lotus-storage-miner deals reset-blocklist, which wipes the storage miner's blocklist.

@laser laser changed the base branch from master to next June 18, 2020 22:50
@laser laser requested review from magik6k and arajasek June 18, 2020 22:50
Copy link
Contributor

@magik6k magik6k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So I guess this is better than nothing at all, but we'll need to expand it into its own module, with some more features

  • Using datastore instead of the config
  • Use double-hashed CIDs
  • Don't pipe entire blocklists over RPC
  • (future) Potentially allow calling external services handling blocklists

@magik6k magik6k changed the base branch from next to master June 19, 2020 19:17
@magik6k magik6k merged commit 596ed33 into filecoin-project:master Jun 19, 2020
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