-
Notifications
You must be signed in to change notification settings - Fork 91
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
Add e2e for locality aware lb #1146
Comments
@hzxuzhonghu I would like to solve this issue. Please assign the issue to me 🙌 |
Thank you. First you can try to learn about this feature, and then should learn how to write e2e case /assign @Murdock9803 |
@hzxuzhonghu Thank you for this... I will do as you said, and will try to come up with a good PR soon 🙌 |
Hey @Murdock9803, are you working on this? If this is stale, I’d love to pick it up. Let me know. Thanks! |
@7h3-3mp7y-m4n Yes you can work on this 🙌 |
Thank you, @Murdock9803 and @hzxuzhonghu! I’ll go through the docs and raise the PR as soon as possible. In the meantime, @hzxuzhonghu, would it be possible to connect with you on Slack? That way, I can reach out for help if I run into any issues. |
@7h3-3mp7y-m4n you can post your doubts on the group chat on Slack, so that everyone can get benefit fron this. You can find the slack invite link from kmesh.net site 🙌 |
thanks @Murdock9803! |
Kmesh has supported locality lb FYI https://kmesh.net/en/docs/userguide/locality_loadbalance/
However it lacks e2e tests now
The text was updated successfully, but these errors were encountered: