-
Notifications
You must be signed in to change notification settings - Fork 38.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
Automated cherry pick of #67288: Immediate close the other half of the connection when #68041
Automated cherry pick of #67288: Immediate close the other half of the connection when #68041
Conversation
- don't wait for the both sides of the connection to close on their own. If one closes there doesn't seem to be any point in continuing the other half. - for port-forward, this prevents the api-server and kubelet both waiting on the other with no result. Normally the connection eventually times out. This cleans up the connection appropriately. - add a log message on closing the connection that corresponds to opening the connection
@tossmilestone: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/assign @deads2k |
/assign @liggitt @lavalamp @smarterclayton |
/assign @foxish |
/ok-to-test |
@tossmilestone, this needs lgtm and approval. Also, can you please write the release note in user-facing terms? i.e. what impact it will have. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lavalamp, tossmilestone The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@foxish is the patch release manager and is already assigned. |
@foxish This isn't a user facing change. There should be nothing different. It changes how server components interact with each other. |
Cherry pick of #67288 on release-1.11.
#67288: Immediate close the other half of the connection when
Release note: