-
Notifications
You must be signed in to change notification settings - Fork 99
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
Load Balancing Algorithm doesn't consider weight of nodes #98
Comments
Also, thanks a lot! Love the project! More than happy to submit a PR if this is interesting |
Hi @prostko Sounds like an interesting idea. If you would like to work on this you can take a look at
List of test files for a given node index from dynamic tests splitA bit similar idea to Least Connections Algorithm you can find in Knapsack Pro Queue Mode that will dynamically split tests between parallel nodes. This way you solve problems when nodes start at a different times or test files have random execution time. You can check this article https://docs.knapsackpro.com/2020/how-to-speed-up-ruby-and-javascript-tests-with-ci-parallelisation |
I'm closing this issue because #99 was merged. |
It seems that this is the current algorithm used to assign specs to nodes:
Round Robin
This can be optimized by:
Least Connections Algorithm
This way the nodes will be more balanced.
The text was updated successfully, but these errors were encountered: