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

etcdclient: leases keep alive halted: etcdserver: request timed out #8618

Closed
zimulala opened this issue Sep 27, 2017 · 5 comments
Closed

etcdclient: leases keep alive halted: etcdserver: request timed out #8618

zimulala opened this issue Sep 27, 2017 · 5 comments

Comments

@zimulala
Copy link

I want to use the interface of NewSession in clientv3 to new a session. But I encountered a problem in the certain case.
The result of NewSession always failed and reported "etcdclient: leases keep alive halted: etcdserver: request timed out".
I think it's because the etcd server occurs an ErrTimeout, the log content is "[warning] etcdserver: [timed out waiting for read index response]". Then it makes the client's channel of donec closed in recvKeepaliveLoop function. And makes the execution result fail.
I think it's unreasonable. In my opinion, handle it in one way that closes the client when the keepalive is halted, or another way that provides an interface which can restart the recvKeepaliveLoop.

@gyuho
Copy link
Contributor

gyuho commented Sep 27, 2017

etcd server occurs an ErrTimeout, the log content is "[warning] etcdserver: [timed out waiting for read index response]"

Means etcdserver is not responding?

And why not just retry in for-loop?

@zimulala
Copy link
Author

No. I mean the etcdserver returns the ErrTimeout to the client, and I retry NewSession.
But I think the recvKeepaliveLoop is returned and donec is closed when the etcdserver returns ErrTimeout as a response. So it is useless to try NewSession again.

@gyuho
Copy link
Contributor

gyuho commented Sep 28, 2017

What is recvKeepaliveLoop? Please provide reproducible code snippet.

@zimulala
Copy link
Author

It is fixed in the master branch.

@zimulala
Copy link
Author

@gyuho Thanks!
RP #7890

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants