smoldot icon indicating copy to clipboard operation
smoldot copied to clipboard

Smoldot Crash during statemine/statemint tests

Open wirednkod opened this issue 2 years ago • 3 comments

While talking with @PierreBesson , he mentioned that during various tests that he was performing, for connecting light client with Statemine (using Substrate connect) the following errors appeared:

smoldot crash

wirednkod avatar Jun 20 '22 14:06 wirednkod

What I think happens is:

  • We queue a parahead fetching operation, but the operation doesn't start immediately.
  • The block whose parahead we are fetching gets pruned. The code currently unpins the block.
  • The fetching operation gets executed. The panics happens because the block is unpinned.

tomaka avatar Jun 20 '22 14:06 tomaka

What I think happens is:

  • We queue a parahead fetching operation, but the operation doesn't start immediately.
  • The block whose parahead we are fetching gets pruned. The code currently unpins the block.
  • The fetching operation gets executed. The panics happens because the block is unpinned.

Fwiw this is an error thrown during multiple local tests, Pierre was trying to connect to statemint/statemine

wirednkod avatar Jun 20 '22 16:06 wirednkod

My diagnostic is wrong. The pruning is done after the parahead fetch operation.

tomaka avatar Jun 21 '22 08:06 tomaka

I have no idea what causes this, but https://github.com/paritytech/smoldot/pull/2631 improves the error message. It might also have been accidentally fixed by a couple of PRs since then. I'm going to close this issue, please open a new one if you have a similar error.

tomaka avatar Sep 07 '22 11:09 tomaka