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

Wavelink always using the node from NodePool which is connected first to Nodepool (Ignoring the specs and load of the lavalink server ) #235

Closed
munishkhatri720 opened this issue Jul 18, 2023 · 3 comments · Fixed by #254

Comments

@munishkhatri720
Copy link

image

node : wavelink.Node = wavelink.Node(uri='https://eu-lavalink.lexnet.cc:443',password="lexn3tl@val!nk",secure=True,retries=3,id="LEXNET 1")

is connected first to Nodepool but it is a public node and worst quality (too much load on it )
but it still playing with this as u can see it is reconnecting again and again

image

@munishkhatri720
Copy link
Author

wavelink Nodepool has to rearrange nodes by their loads

@chillymosh
Copy link
Contributor

chillymosh commented Jul 18, 2023

Hi,
The NodePool gets connected to lowest-use Node in the NodePool already, which is determined by how many Players your current Bot has running on it and not what the node has running on it. We do generally recommend running your own node.

Please make sure you are on the latest version, preferably the git version (2.6.1) as it has some minor fixes.

This may be subject to change in the future.

@munishkhatri720
Copy link
Author

Hi, The NodePool gets connected to lowest-use Node in the NodePool already. Please make sure you are on the latest version, preferably the git version (2.6.1) as it has some minor fixes.

Do note that the load is determined by how many Players your current Bot has running on it and not what the public node has running on it.

but other bots are playing on a public node , it has to consider all, like javascript lavalink wrapper "erela.js" have

@EvieePy EvieePy linked a pull request Nov 27, 2023 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants