Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Crash when using --force-sealing #5803

Closed
SeanBannister opened this issue Jun 9, 2017 · 5 comments · Fixed by #5965
Closed

Crash when using --force-sealing #5803

SeanBannister opened this issue Jun 9, 2017 · 5 comments · Fixed by #5965
Labels
F2-bug 🐞 The client fails to follow expected behavior. M4-core ⛓ Core client code / Rust.

Comments

@SeanBannister
Copy link

SeanBannister commented Jun 9, 2017

I'm new to parity so I don't have any debugging knowledge.

When I run Parity with --force-sealing parity crashes as soon as it tries to process any transaction. The following error is shown:
thread '<unknown>' has overflowed its stack

I'm running Windows 8.1.

The exact command I'm running is:
parity.exe --geth --force-ui --chain dev --force-sealing
Removing --force-sealing the command runs fine.

I'm not 100% sure I need this command, just trying to get a testing environment going.

@keorn
Copy link

keorn commented Jun 9, 2017

There is no need for --force-sealing for --chain dev. What Parity version are you using?

@keorn keorn added F2-bug 🐞 The client fails to follow expected behavior. M4-core ⛓ Core client code / Rust. labels Jun 9, 2017
@SeanBannister
Copy link
Author

Thanks keorn, I'm running: v1.6.8-beta-c396229-20170608/x86_64-windows-msvc/rustc1.17.0

The reason I was trying to use --force-sealing is when I submitted a transaction the UI said it still required more confirmations, if I submitted another transaction it would cause the confirmation count to increase. I know there must be a way to get parity to continue to perform confirmations and I presumed this was it?

@keorn
Copy link

keorn commented Jun 10, 2017

There is no need to wait for additional confirmations, this only applies to the public network where you can get chain reorganisations.

@guanqun
Copy link
Contributor

guanqun commented Jun 30, 2017

Should we simply add a check to reject the combination of these two options? If you think it's a good idea, I can help add this. :)

@keorn
Copy link

keorn commented Jun 30, 2017

Would be great, it's a rather simple fix. Dev chain has reseal_min_period set to 0 here. --force-sealing should not be allowed whenever it's 0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
F2-bug 🐞 The client fails to follow expected behavior. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants