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

lotus-miner proving check broken #8560

Closed
8 of 18 tasks
RobQuistNL opened this issue Apr 27, 2022 · 6 comments · Fixed by #9643
Closed
8 of 18 tasks

lotus-miner proving check broken #8560

RobQuistNL opened this issue Apr 27, 2022 · 6 comments · Fixed by #9643
Assignees
Labels
area/proving Area: Proving kind/bug Kind: Bug P2 P2: Should be resolved

Comments

@RobQuistNL
Copy link
Contributor

RobQuistNL commented Apr 27, 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

v1.15.2-rc1

Describe the Bug

Command issued: lotus-miner proving check 0

Output:

ERROR: rg is nil

Logging Information

$ lotus-miner proving check 0
ERROR: rg is nil

Repo Steps

  1. Run wdpost / winningpost workers remote
  2. Run lotus-miner proving check 0
  3. See error ERROR: rg is nil
@Reiers Reiers added area/proving Area: Proving need/analysis Hint: Needs Analysis and removed need/triage labels Apr 27, 2022
@Reiers Reiers self-assigned this Apr 27, 2022
@Reiers
Copy link

Reiers commented Apr 27, 2022

Hi @RobQuistNL

Thanks for the report. I have added labels.
I will repro and add more logs.

@Reiers
Copy link

Reiers commented Apr 28, 2022

Hi @RobQuistNL

I move the other response to the correct issue.

So from my repro session Im seeing the same thing as you:

Miner log:

2022-04-28T14:02:56.307+0200	WARN	rpc	[email protected]/handler.go:279	error in RPC call to 'Filecoin.CheckProvable': rg is nil:
    [github.com/filecoin-project/lotus/extern/sector-storage.(*Manager).CheckProvable](http://github.com/filecoin-project/lotus/extern/sector-storage.(*Manager).CheckProvable)
        /home/reiers/lotus/extern/sector-storage/faults.go:32
reiers@techhedge:~$ lotus-miner proving check 0
ERROR: rg is nil

lotus-miner proving check --slow 0 - still works, at least here.

reiers@techhedge:~$ lotus-miner proving check --slow 0
deadline  partition  sector  status
0         0          1202    good
0         0          1233    good
0         0          1130    good
0         0          1757    good
0         0          1923    good
0         0          1304    good
+++

Lotus version:

Daemon:  1.15.2-rc1+mainnet+git.dcf6f6414+api1.5.0
Local: lotus version 1.15.2-rc1+mainnet+git.dcf6f6414

Thank you !

@Reiers Reiers added the need/author-input Hint: Needs Author Input label Apr 28, 2022
@RobQuistNL
Copy link
Contributor Author

The compute window post command you posted works :)

Can you try lotus-miner proving check 0 ? ;)

@Reiers Reiers added P2 P2: Should be resolved and removed need/author-input Hint: Needs Author Input need/analysis Hint: Needs Analysis labels Apr 28, 2022
@RobQuistNL
Copy link
Contributor Author

lotus-miner proving check --faulty 0

gives the same error

@Pegasus-starry
Copy link

I have met the same issue. It related to the code change of CheckProvable in extern/sector-storage/faults.go

@rjan90 rjan90 mentioned this issue Jun 30, 2022
18 tasks
@Aloxaf
Copy link
Contributor

Aloxaf commented Jul 11, 2022

The bug seems to be introduced by 4a874ef

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/proving Area: Proving kind/bug Kind: Bug P2 P2: Should be resolved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants