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

Rewrite clientv3 health balancer #9106

Closed
gyuho opened this issue Jan 6, 2018 · 1 comment
Closed

Rewrite clientv3 health balancer #9106

gyuho opened this issue Jan 6, 2018 · 1 comment
Assignees
@gyuho
Labels
area/clientv3
Milestone
etcd-v3.4

Comments

@gyuho
Copy link
Contributor

gyuho commented Jan 6, 2018 •
edited
Loading

Current/old balancer works (sort of) like this #8828 (comment).

Basically it keeps track of unhealthy endpoints, and retries healthy ones on transient disconnects.

We can do better.

  1. Only try one endpoint at a time (currently connect to all at once, shut down others once pinned).
  2. Only keep track of healthy endpoints (the ones that we know for sure is healthy).
  3. Use new gRPC balancer interface L9: new balancer and resolver APIs for gRPC-go grpc/proposal#30.

TODO:


I will add a design doc later with more details.
@xiang90 xiang90 added this to the v3.4.0 milestone Jan 6, 2018
@gyuho gyuho mentioned this issue Jan 17, 2018
Closed
@gyuho gyuho self-assigned this Jan 24, 2018
@gyuho gyuho mentioned this issue Jan 24, 2018
Closed
@gyuho gyuho mentioned this issue Mar 19, 2018
Merged
@gyuho gyuho mentioned this issue May 17, 2018
Closed
@gyuho
Copy link
Contributor Author

gyuho commented Jun 15, 2018

Should be all fixed via #9860.

@gyuho gyuho closed this as completed Jun 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gyuho gyuho

Labels
area/clientv3
Milestone
etcd-v3.4
Development

No branches or pull requests
2 participants
@xiang90 @gyuho