-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
outputs/http: add option to control idle connection timeout #8055
outputs/http: add option to control idle connection timeout #8055
Conversation
Please rebase on master to fix the AppVeyor problem. Also to clarify, this is for the |
Done with rebase. And yes this is for the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Could you please rebase again to get rid of the CI error!?
Hey @YuriGrigorov why did you close the PR??? |
Sorry for this. I screwed up with the rebase. |
Yes! Finally! "All checks have passed" @srebhan сould you please check if everything is ok? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything ok in my view.
@YuriGrigorov did you already sign the CLA? https://www.influxdata.com/legal/cla/ |
@YuriGrigorov please sign the CLA as soon as possible so we don't have to revert back the merged changes. Thanks! |
Done. CLA signed. |
…ta#8055) Co-authored-by: Yuri Grigorov <jurijs.grigorovs@corp.mail.ru>
Co-authored-by: Yuri Grigorov <jurijs.grigorovs@corp.mail.ru> (cherry picked from commit c319e63)
…ta#8055) Co-authored-by: Yuri Grigorov <jurijs.grigorovs@corp.mail.ru>
This PR resolves the issue when Telegraf stays in front of Load Balancer.
Some servers has hardcoded IdleTimeut which is higher then 15 seconds (default Telegrafs keep-alive timeout). In this case Telegraf creates keep-alive connection and it is never closed because of reuse.
As a result, balancing does not work.