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.

Sign up for GitHub

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

Jump to bottom

Least connection mode is not working #810

Closed
nik-netlox opened this issue Sep 25, 2024 · 2 comments
Closed

Least connection mode is not working #810

nik-netlox opened this issue Sep 25, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@nik-netlox
Copy link
Collaborator

Describe the bug
When LB rule is created with --select=lc then client receives connection refused on subsequent connections.

To Reproduce
Steps to reproduce the behavior:
loxilb create rule: loxicmd create lb 20.20.20.1 --tcp=2020:8080 --endpoints=31.31.31.1:1,32.32.32.1:1,33.33.33.1:1 --select=lc

Run server app from all endpoints and open multiple connections from client.

Expected behavior
Client connection request must be sent to the endpoint having least number of active connections.

Screenshots
NA

Environment (please complete the following information):

  • OS: Ubuntu 20.04
  • Kernel Version: 5.15
  • LoxiLB Version: latest
  • Cloud Environment: On-prem
  • K8s Version: NA

Additional context
NA
@nik-netlox nik-netlox added the bug Something isn't working label Sep 25, 2024
TrekkieCoder added a commit to loxilb-io/loxilb-ebpf that referenced this issue Oct 1, 2024
@TrekkieCoder
loxilb-io/loxilb#810 Do not consider inactive EP when finding Least-C…
eb1c3c0 
…onnections
TrekkieCoder added a commit to loxilb-io/loxilb-ebpf that referenced this issue Oct 1, 2024
@TrekkieCoder
loxilb-io/loxilb#810 Do not consider inactive EP when finding Least-C…
52d52aa 
…onnections
UltraInstinct14 added a commit that referenced this issue Oct 2, 2024
@UltraInstinct14
Merge pull request #820 from TrekkieCoder/main
49c623f 
PR: gh-810 fixes for LC selection and added more tests to LC selftests
@TrekkieCoder
Copy link
Collaborator

Some issues were found and fixed. @nik-netlox Can you please reconfirm if issue is resolved ?

TrekkieCoder added a commit that referenced this issue Oct 2, 2024
@TrekkieCoder
Merge pull request #822 from TrekkieCoder/main
5f858a7 
gh-810 Merge lc-fix branch to main
@nik-netlox
Copy link
Collaborator Author

This test has been validated. Closing it.

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

No branches or pull requests
2 participants
@nik-netlox @TrekkieCoder