Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: dagstore: add dagstore register-shard command #8645

Merged
merged 4 commits into from
May 20, 2022
Merged

feat: dagstore: add dagstore register-shard command #8645

merged 4 commits into from
May 20, 2022

Conversation

LexLuthr
Copy link
Contributor

@LexLuthr LexLuthr commented May 12, 2022

Related Issues

[]
#8644

Proposed Changes

$ lotus-miner dagstore register-shard --help
NAME:
lotus-miner dagstore register-shard - Register a shard

USAGE:
lotus-miner dagstore register-shard [command options] [key]

OPTIONS:
--color use color in display output (default: depends on output being a TTY)
--help, -h show help (default: false)

Additional Info

Relates issue filecoin-project/boost#509

Checklist

Before you mark the PR ready for review, please make sure that:

  • All commits have a clear commit message.
  • The PR title is in the form of of <PR type>: <area>: <change being made>
    • example: fix: mempool: Introduce a cache for valid signatures
    • PR type: fix, feat, INTERFACE BREAKING CHANGE, CONSENSUS BREAKING, build, chore, ci, docs,perf, refactor, revert, style, test
    • area: api, chain, state, vm, data transfer, market, mempool, message, block production, multisig, networking, paychan, proving, sealing, wallet, deps
  • This PR has tests for new functionality or change in behaviour
  • If new user-facing features are introduced, clear usage guidelines and / or documentation updates should be included in https://lotus.filecoin.io or Discussion Tutorials.
  • CI is green

@LexLuthr LexLuthr marked this pull request as ready for review May 12, 2022 17:02
@LexLuthr LexLuthr requested a review from a team as a code owner May 12, 2022 17:02
@LexLuthr LexLuthr marked this pull request as draft May 12, 2022 18:44
@LexLuthr LexLuthr marked this pull request as ready for review May 12, 2022 19:15
@codecov
Copy link

codecov bot commented May 12, 2022

Codecov Report

Merging #8645 (8f71927) into master (33a05d3) will decrease coverage by 0.13%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8645      +/-   ##
==========================================
- Coverage   40.94%   40.81%   -0.14%     
==========================================
  Files         686      686              
  Lines       75750    75781      +31     
==========================================
- Hits        31017    30929      -88     
- Misses      39394    39513     +119     
  Partials     5339     5339              
Impacted Files Coverage Δ
api/api_storage.go 0.00% <ø> (ø)
cmd/lotus-miner/dagstore.go 19.58% <0.00%> (-2.47%) ⬇️
node/impl/storminer.go 23.78% <0.00%> (-0.56%) ⬇️
itests/kit/blockminer.go 70.65% <0.00%> (-19.03%) ⬇️
chain/actors/builtin/miner/diff.go 48.52% <0.00%> (-10.30%) ⬇️
cli/util.go 70.83% <0.00%> (-8.34%) ⬇️
extern/sector-storage/worker_tracked.go 80.50% <0.00%> (-6.78%) ⬇️
miner/miner.go 56.72% <0.00%> (-3.94%) ⬇️
storage/wdpost_sched.go 81.37% <0.00%> (-3.93%) ⬇️
node/hello/hello.go 63.63% <0.00%> (-3.41%) ⬇️
... and 17 more

@magik6k magik6k merged commit 8552cdb into filecoin-project:master May 20, 2022
@LexLuthr LexLuthr deleted the feat/dag-reg-shard branch May 31, 2022 14:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Backport https://github.com/filecoin-project/boost/pull/517 to lotus-miner
3 participants