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

fix: post workers: check proving params on startup #8736

Merged
merged 1 commit into from
May 27, 2022

Conversation

magik6k
Copy link
Contributor

@magik6k magik6k commented May 27, 2022

PoSt workers are doing snarks, so they need proof params

@magik6k magik6k requested a review from a team as a code owner May 27, 2022 11:13
@codecov
Copy link

codecov bot commented May 27, 2022

Codecov Report

Merging #8736 (7d7ecb5) into master (7836e20) will increase coverage by 0.04%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8736      +/-   ##
==========================================
+ Coverage   40.84%   40.88%   +0.04%     
==========================================
  Files         692      692              
  Lines       76177    76177              
==========================================
+ Hits        31112    31144      +32     
+ Misses      39688    39666      -22     
+ Partials     5377     5367      -10     
Impacted Files Coverage Δ
cmd/lotus-worker/main.go 0.00% <0.00%> (ø)
chain/events/message_cache.go 87.50% <0.00%> (-12.50%) ⬇️
chain/actors/builtin/miner/diff.go 48.52% <0.00%> (-10.30%) ⬇️
chain/events/events_called.go 83.90% <0.00%> (-1.96%) ⬇️
miner/miner.go 55.73% <0.00%> (-0.99%) ⬇️
storage/wdpost_changehandler.go 98.11% <0.00%> (-0.48%) ⬇️
chain/sync_manager.go 74.22% <0.00%> (-0.32%) ⬇️
chain/stmgr/call.go 69.18% <0.00%> (ø)
node/impl/storminer.go 23.78% <0.00%> (+0.30%) ⬆️
chain/messagepool/selection.go 83.84% <0.00%> (+0.38%) ⬆️
... and 9 more

@arajasek arajasek merged commit 69d3542 into master May 27, 2022
@arajasek arajasek deleted the fix/postworker-check-params branch May 27, 2022 15:00
# 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.

2 participants