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

Disabled IndexProvider causes "panic: runtime error: invalid memory address or nil pointer dereference" - v1.17.0.rc1 #8916

Closed
8 of 18 tasks
donkabat opened this issue Jun 25, 2022 · 4 comments
Labels
area/index-provider kind/bug Kind: Bug P1 P1: Must be resolved

Comments

@donkabat
Copy link

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-miner version 1.17.0-rc1+calibnet+git.d2fe153e3

Describe the Bug

I'm not able to run lotus miner with disabled IndexProvider.

New lotus-miner's installation & calibnet.
Before first run I set config.toml:
[IndexProvider]
Enable = false

It worked for a day and next it didnt want to run after restart. Probably it was the first restart after first storage deal.

Logging Information

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x2013e37]

goroutine 15347 [running]:
github.com/filecoin-project/index-provider/engine.(*Engine).Start(0xc00ef98770, {0x43aa8c0, 0xc000f432f0})
        /home/filecoin/go/pkg/mod/github.com/filecoin-project/index-provider@v0.6.1/engine/engine.go:123 +0x437
github.com/filecoin-project/lotus/node/modules.IndexProvider.func1.1({0x43aa8c0, 0xc000f432f0})
        /home/filecoin/networks/calibration/build/lotus/node/modules/storageminer_idxprov.go:97 +0x34
go.uber.org/fx/internal/lifecycle.(*Lifecycle).runStartHook(0xc00034c8c0, {0x43aa8c0, 0xc000f432f0}, {0xc00ef873a0, 0xc00ef873b0, {{0xc00dde4370, 0x42}, {0x7e2cb8f, 0x54}, 0x5c}})
        /home/filecoin/go/pkg/mod/go.uber.org/fx@v1.15.0/internal/lifecycle/lifecycle.go:118 +0x1fd
go.uber.org/fx/internal/lifecycle.(*Lifecycle).Start(0xc00034c8c0, {0x43aa8c0, 0xc000f432f0})
        /home/filecoin/go/pkg/mod/go.uber.org/fx@v1.15.0/internal/lifecycle/lifecycle.go:83 +0x2a5
go.uber.org/fx.(*App).start(0xc001086c30, {0x43aa8c0, 0xc000f432f0})
        /home/filecoin/go/pkg/mod/go.uber.org/fx@v1.15.0/app.go:745 +0x36
go.uber.org/fx.withTimeout.func1()
        /home/filecoin/go/pkg/mod/go.uber.org/fx@v1.15.0/app.go:977 +0x32
created by go.uber.org/fx.withTimeout
        /home/filecoin/go/pkg/mod/go.uber.org/fx@v1.15.0/app.go:977 +0xf1

Repo Steps

No response

@donkabat donkabat changed the title Disabled index-provider causes "panic: runtime error: invalid memory address or nil pointer dereference" - v1.17.0.rc1 Disabled IndexProvider causes "panic: runtime error: invalid memory address or nil pointer dereference" - v1.17.0.rc1 Jun 25, 2022
@donkabat
Copy link
Author

After change to v1.16.0 the error is gone.
On v1.17.0_rc1 still the same.

@rjan90
Copy link
Contributor

rjan90 commented Jun 25, 2022

Hey @donkabat Thank you so much for the issue-report! This should and must be fixed before the final release is cut. I have raised the issue to the devs, and will come back to you later when we have found a solution.

I will repro tomorrow to confirm that I see this on my side as well!

@rjan90 rjan90 added P1 P1: Must be resolved area/index-provider and removed need/triage labels Jun 25, 2022
@masih
Copy link
Member

masih commented Jun 27, 2022

Fixed already in index-provider version 0.7.0 and later. The fix in lotus requires upgrading index-provider dependency. This is currently in progress pending transient dependency upgrades for go-cid 0.2.0 which is unrelated to index-provider.

@rjan90
Copy link
Contributor

rjan90 commented Jul 8, 2022

Closing this since the index-provider version has been upgraded in the lotus v1.17.1-rc2 version which should resolve this issue. If you are experiencing this problem on the newer release candidate / versions - please open a new ticket!

Thanks again for the ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/index-provider kind/bug Kind: Bug P1 P1: Must be resolved
Projects
None yet
Development

No branches or pull requests

3 participants