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

sd/etcdv3 keepalive channel not served #799

Closed
fangdingjun opened this issue Nov 21, 2018 · 0 comments · Fixed by #803
Closed

sd/etcdv3 keepalive channel not served #799

fangdingjun opened this issue Nov 21, 2018 · 0 comments · Fixed by #803

Comments

@fangdingjun
Copy link
Contributor

c.hbch, err = c.leaser.KeepAlive(c.ctx, c.leaseID)

the channel c.hbch not served, etcd library print a lots of log like this, every 3 seconds

{"level":"warn","ts":1542791960.4143248,"caller":"clientv3/lease.go:524","msg":"lease keepalive response queue is full; dropping response send","queue-size":16,"queue-capacity":16}
{"level":"warn","ts":1542791963.92008,"caller":"clientv3/lease.go:524","msg":"lease keepalive response queue is full; dropping response send","queue-size":16,"queue-capacity":16}
{"level":"warn","ts":1542791967.4572115,"caller":"clientv3/lease.go:524","msg":"lease keepalive response queue is full; dropping response send","queue-size":16,"queue-capacity":16}

fangdingjun added a commit to fangdingjun/kit that referenced this issue Nov 22, 2018
read and discard keepalive response,
make etcd library not to complain
fix go-kit#799
peterbourgon pushed a commit that referenced this issue Mar 11, 2019
read and discard keepalive response,
make etcd library not to complain
fix #799
peterbourgon pushed a commit that referenced this issue Mar 12, 2019
* fix bug #799

I have a test. Just close etcd, after 10 seconds, it will dead loop crazy, cpu100%.
Need add a little guard code.

* Update client.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants