-
Notifications
You must be signed in to change notification settings - Fork 489
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
WebUI hammers my daemon #882
Comments
Yes. We feel the same feels. There is a runaway cpu load issue caused by using the ipfs-geoip db ipfs/kubo#5613 We've dialled it down, but it could do less. In the short term I will have move the geolocation logic to just the peers page. It will still hammer the daemon, but only when you go looking at your peers list / map. There is #848 to track that. Longer term we need to calm ipfs down. It should be possible to look up all the ipfs-geoip blocks for 800 ipv4 addresses, without causing the fans to spin. |
@hsanjuan web ui no longer tries to do the geolocation unless you are looking at the peers tab. Please take another look; https://webui.ipfs.io has the latest, or when go-ipfs v0.4.19 is released. |
Alas, we've updated deps and broken geolocation completely. #918 |
How many I really like the idea of making B-trees storing the geo-ip information on ipfs though. It's very nice. |
I've dialled it right down, so we queue the requests and excute them one by one rather than in parallel. I plan to redesign the peers page this quarter so that it focuses on highlighting peers near you, and lets you connect to peers manually, rather than just "geolocate them all", to reduce the issue further. |
When I open the webUI (maybe after clicking
Files
) it starts triggering hundredshttp://127.0.0.1:5001/api/v0/object/get
requests to the IPFS daemon and everything stops working well (my browser, my daemon, my computer). I feel things shouldn't be like this?The text was updated successfully, but these errors were encountered: