-
Notifications
You must be signed in to change notification settings - Fork 982
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
overflow when subtracting duration from instant #1805
Comments
Except for the |
I'm using macOS 10.15.5 and rustc 1.45.0 Possibly related issue: rust-lang/rust#48980 |
Oh found something interesting: rust-lang/rust#48980 (comment) Maybe it fails because I rebooted my computer less than a day ago? |
Yep, changing |
Oh wow .. thanks for doing such a thorough investigation! Using time since boot is just mean ;) I'll change it so it doesn't fail no matter when the computer was booted. |
On Mac OSX, Instant::now() is the time since the last boot, which might have been less than a day ago, in which case KillState::new would panic. We now use 'now - 60s' as a time in the past for initialization, and if that fails, we'll use just 'now'. There's not much danger in using 'now', it might just lead to spurious logging and a too-soon update of the kill_rate if the node becomes immediately overloaded after starting. Thanks to kevholder for root-causing the issue and demonstrating a fix. Fixes #1805
On Mac OSX, Instant::now() is the time since the last boot, which might have been less than a day ago, in which case KillState::new would panic. We now use 'now - 60s' as a time in the past for initialization, and if that fails, we'll use just 'now'. There's not much danger in using 'now', it might just lead to spurious logging and a too-soon update of the kill_rate if the node becomes immediately overloaded after starting. Thanks to kevholder for root-causing the issue and demonstrating a fix. Fixes #1805
On Mac OSX, Instant::now() is the time since the last boot, which might have been less than a day ago, in which case KillState::new would panic. We now use 'now - 60s' as a time in the past for initialization, and if that fails, we'll use just 'now'. There's not much danger in using 'now', it might just lead to spurious logging and a too-soon update of the kill_rate if the node becomes immediately overloaded after starting. Thanks to kevholder for root-causing the issue and demonstrating a fix. Fixes #1805
Do you want to request a feature or report a bug?
bug
What is the current behavior?
panic
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
What is the expected behavior?
No panic.
Any idea what is causing this panic? Strangely, I am fairly sure this error was not happening yesterday with the same command-line arguments. I tried resetting the database and IPFS to no avail. I am testing with a heavily modified version of Ethereum API so that might be the culprit but still it would help to have a pointer in the right direction.
The text was updated successfully, but these errors were encountered: