-
Notifications
You must be signed in to change notification settings - Fork 746
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
tokio-runtime-worker panicked #2099
Comments
Seems related to this issue: #1067
I'd guess it's got something to do with the very high peer count |
Yes. The previous issue was in the delay queues managing the ping and meta-data intervals. There must still be an issue in tokio when these are pushed with very high peer counts. It looks to me like it is currently being addressed in tokio by this PR: tokio-rs/tokio#3270 Hopefully we can update tokio once that gets merged to resolve this issue. It seems to be me its currently probabilistic in hitting specific delay timeouts and that probability is exasperated by large peer counts. An interim solution would be to lower the peer count to reduce the chances of hitting this. In the meantime, I think we need to wait for the upstream fix. |
The proposed fix at tokio-rs/tokio#3270 has been merged into tokio |
It seems |
## Issue Addressed resolves #2129 resolves #2099 addresses some of #1712 unblocks #2076 unblocks #2153 ## Proposed Changes - Updates all the dependencies mentioned in #2129, except for web3. They haven't merged their tokio 1.0 update because they are waiting on some dependencies of their own. Since we only use web3 in tests, I think updating it in a separate issue is fine. If they are able to merge soon though, I can update in this PR. - Updates `tokio_util` to 0.6.2 and `bytes` to 1.0.1. - We haven't made a discv5 release since merging tokio 1.0 updates so I'm using a commit rather than release atm. **Edit:** I think we should merge an update of `tokio_util` to 0.6.2 into discv5 before this release because it has panic fixes in `DelayQueue` --> PR in discv5: sigp/discv5#58 ## Additional Info tokio 1.0 changes that required some changes in lighthouse: - `interval.next().await.is_some()` -> `interval.tick().await` - `sleep` future is now `!Unpin` -> tokio-rs/tokio#3028 - `try_recv` has been temporarily removed from `mpsc` -> tokio-rs/tokio#3350 - stream features have moved to `tokio-stream` and `broadcast::Receiver::into_stream()` has been temporarily removed -> `tokio-rs/tokio#2870 - I've copied over the `BroadcastStream` wrapper from this PR, but can update to use `tokio-stream` once it's merged tokio-rs/tokio#3384 Co-authored-by: realbigsean <seananderson33@gmail.com>
Hopefully resolved in #2172, which includes |
Description
Hello,
We see this error after which Lighthouse paralyzes. No backtrace, unfortunately.
Version
Present Behaviour
Lighthouse Beacon Node stops any network activity and doesn't submit attestations.
Expected Behaviour
Lighthouse Beacon Node performs network activity and submits attestations.
Steps to resolve
We simply restarted the node.
The text was updated successfully, but these errors were encountered: