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

[gomobile-lib] better control over the max number of connected peers #20

Open
aeddi opened this issue Jan 21, 2020 · 2 comments
Open

[gomobile-lib] better control over the max number of connected peers #20

aeddi opened this issue Jan 21, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@aeddi
Copy link
Member

aeddi commented Jan 21, 2020

For the moment, the limits that we set in the config concerning the number of peers (high/low watermarks, grace period) does not seem to regulate correctly the maximum number of connected peers (probably due to a misuse / misunderstanding on our part).

We need to find a way to precisely control the maximum number of peers a device is connected to because:

  • On iOS and Android, the number of connected peers is strongly related to the resource consumption
  • On iOS a problem occurs very quickly: the OS throws "too many open files" errors when it reaches about 200 connected peers (this limit is not configurable on iOS)

I will do precise tests with different high/low watermarks and grace period settings and post reports on this issue so that we have a better view of the problem before thinking about solutions.

@aeddi aeddi self-assigned this Jan 21, 2020
@jefft0 jefft0 added the bug Something isn't working label Jul 13, 2022
@jefft0
Copy link
Collaborator

jefft0 commented Apr 24, 2023

Hi @aeddi . If we upgrade kubo to get the Resource Manager (like we did in weshnet), do you think it would resolve this issue?

@jefft0 jefft0 unassigned aeddi May 4, 2023
@aeddi
Copy link
Member Author

aeddi commented May 4, 2023

Hi :)
Honestly I haven't tested the new Kubo Resource Manager yet, it could resolve the issue, would be worth a try

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants