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

dht stream hang clogs up queries #3657

Closed
whyrusleeping opened this issue Feb 6, 2017 · 3 comments
Closed

dht stream hang clogs up queries #3657

whyrusleeping opened this issue Feb 6, 2017 · 3 comments
Labels
kind/bug A bug in existing code (including security flaws) topic/dht Topic dht

Comments

@whyrusleeping
Copy link
Member

Occasionally we're seeing a stream hang (i think its related with #3651 ) in the dht code that causes a locking issue and leads to a bunch of queries getting stuck. The goroutine in question that is stuck looks like goroutine 400009791 in https://ipfs.io/ipfs/Qmax6PQz7n1zxd6RxbFPdqK3MC89q93JXtc5r6e7Nuswuw/ipfs.stacks

The root of the problem is these random yamux stream hangs, but we should be able to alleviate the issue by adding timeouts to the dht, and not blocking queries on a single peer.

@whyrusleeping whyrusleeping added kind/bug A bug in existing code (including security flaws) topic/dht Topic dht labels Feb 6, 2017
@ghost
Copy link

ghost commented Feb 6, 2017

One more from pluto.i.ipfs.io: https://ipfs.io/ipfs/QmeTtVLvKb97QCwhzmJG6YKhyiSSybwCeWBhH2CmENy68y

@whyrusleeping
Copy link
Member Author

I think these have been resolved by a recent fix in stream handling timeouts. Maybe keep it open until the Reset changes have been merged through?

cc @Stebalien

@Stebalien
Copy link
Member

Which recent fix? Unfortunately, I can't download those stack traces (grumble permanent web...). I don't recall making or seeing any timeout changes related to yamux.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug A bug in existing code (including security flaws) topic/dht Topic dht
Projects
None yet
Development

No branches or pull requests

2 participants