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

Locality aware routing #862

Closed
howardjohn opened this issue Apr 2, 2024 · 0 comments · Fixed by #868
Closed

Locality aware routing #862

howardjohn opened this issue Apr 2, 2024 · 0 comments · Fixed by #868
Assignees
@howardjohn

Comments

@howardjohn
Copy link
Member

We should support some 'locality aware' load balancing in ztunnel.

Kubernetes core Service has this now in 1.30: https://github.com/kubernetes/enhancements/tree/master/keps/sig-network/4444-service-routing-preference#summary. We can utilize this as the API.

This gives a 'PreferClose' to strictly use the close endpoints only, if they are available.

There is also existing "use only same node" mode that we will want to implement.

We will want to make PreferClose the default for waypoints.

API design TBD
@howardjohn howardjohn self-assigned this Apr 2, 2024
howardjohn added a commit to howardjohn/ztunnel that referenced this issue Apr 3, 2024
@howardjohn
Implement locality aware load balancing
e943b23 
Fixes istio#862
@howardjohn howardjohn mentioned this issue Apr 3, 2024
Merged
howardjohn added a commit to howardjohn/ztunnel that referenced this issue Apr 3, 2024
@howardjohn
Implement locality aware load balancing
563a616 
Fixes istio#862
istio-testing pushed a commit that referenced this issue Apr 4, 2024
@howardjohn
Implement locality aware load balancing (#868)
f918a76 
* Implement locality aware load balancing

Fixes #862

* Update docs

* Add tests and pick better names

* fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@howardjohn howardjohn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement locality aware load balancing howardjohn/ztunnel
1 participant
@howardjohn