-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Flood of connection timed out message and accordingly side effect #3192
Comments
Signed-off-by: Fenggang <ginobiliwang@gmail.com>
Signed-off-by: Fenggang <ginobiliwang@gmail.com>
Signed-off-by: Fenggang <ginobiliwang@gmail.com>
Signed-off-by: Fenggang <ginobiliwang@gmail.com>
Rolling back to v1.7.0 resolves issue for us in all clusters. |
@pssa Rolling back to v1.7.0 is just a temporary walk-around. @edsiper has pushed this commit for "lib: monkey: sync event epoll changes on closing timeout fd". With the above commit, connection fd would be recycled right after the retry tasks ends. As a result, libevent library would trigger connection timeout after "net.connect_timeout", and that is where "[upstream] connection xxxxx to xxxxxxx timed out after xxxxx seconds" comes from. Accordingly PR #3202 could just to resolve the issue. @edsiper @nokute78 correct me if there is any misunderstanding. |
@ginobiliwang ... i'm facing the same problem ... any chance this will make it to 1.7.3? |
Hit by this one as well, rolled back to 1.7.0 as per the comments, and confirm we don't see tons of timeouts errors anymore after that. |
The patch 1938ec8 will be merged from v1.7.3 (not released yet.) |
close this issue, fix is by 1938ec8 |
Bug Report
Describe the bug
Similar things reported in the following issue entry. In my test, the failure of recycling these connections would have side effect on reconnecting to remote side.
#2950
#2800
Behavior is like, we could spot large amount of log information as below.
To Reproduce
server side console output
Expected behavior
The text was updated successfully, but these errors were encountered: