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.

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

clientv3: move health balancer to "balancer" #9462

Merged
merged 1 commit into from
Mar 19, 2018

Conversation

gyuho
Copy link
Contributor

@gyuho gyuho commented Mar 19, 2018

Preparing balancer rewrite with new gRPC. #9106

@gyuho gyuho force-pushed the balancer-clean-up branch from 50ce273 to c5f39e4 Compare March 19, 2018 21:14
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho gyuho force-pushed the balancer-clean-up branch from c5f39e4 to 3aa5711 Compare March 19, 2018 22:16
@codecov-io
Copy link

Codecov Report

Merging #9462 into master will increase coverage by 0.04%.
The diff coverage is 87.05%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9462      +/-   ##
==========================================
+ Coverage   72.51%   72.56%   +0.04%     
==========================================
  Files         364      364              
  Lines       30874    30890      +16     
==========================================
+ Hits        22389    22416      +27     
+ Misses       6855     6854       -1     
+ Partials     1630     1620      -10
Impacted Files Coverage Δ
clientv3/retry.go 94.86% <100%> (ø) ⬆️
clientv3/balancer/grpc1.7-health.go 86.33% <85.5%> (ø)
clientv3/client.go 84.74% <91.66%> (+0.97%) ⬆️
pkg/adt/interval_tree.go 81.68% <0%> (-6.91%) ⬇️
auth/simple_token.go 87.03% <0%> (-6.49%) ⬇️
clientv3/namespace/watch.go 72.72% <0%> (-6.07%) ⬇️
clientv3/txn.go 95.55% <0%> (-4.45%) ⬇️
etcdserver/cluster_util.go 76.64% <0%> (-1.46%) ⬇️
etcdserver/api/v2http/client.go 90.33% <0%> (-0.45%) ⬇️
proxy/grpcproxy/watcher.go 89.79% <0%> (ø) ⬆️
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dea71f3...3aa5711. Read the comment docs.

@gyuho gyuho merged commit 56d1a45 into etcd-io:master Mar 19, 2018
@gyuho gyuho deleted the balancer-clean-up branch March 19, 2018 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants