-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
skip WaitForAttachAndMount for terminated pods in syncPod #49724
skip WaitForAttachAndMount for terminated pods in syncPod #49724
Conversation
this is the same gating logic that we use for pod level cgroup creation and destruction, so it appears safe to me. will wait for bots to report green. |
not for this pr, but a good future gating test would be to see if we get events about a pod from the node after its death. i am not sure how long it should wait, but maybe a node e2e could let us experiement on it in future. |
/release-note-none |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: derekwaynecarr, sjenning Associated issue: 49663 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Can I get a cherry-pick-candidate on this? |
/retest |
3 similar comments
/retest |
/retest |
/retest |
/retest |
/retest Review the full test history for this PR. |
8 similar comments
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
4 similar comments
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
/retest Review the full test history for this PR. |
Automatic merge from submit-queue (batch tested with PRs 49284, 49555, 47639, 49526, 49724) |
Automatic merge from submit-queue UPSTREAM: 49724: skip WaitForAttachAndMount for terminated pods in syncPod Fixes #14383 xref kubernetes/kubernetes#49724 @derekwaynecarr @eparis @smarterclayton @jwforres
Fixes #49663
I tried to tread lightly with a small localized change because this needs to be picked to 1.7 and 1.6 as well.
I suspect this has been as issue since we started unmounting volumes on pod termination #37228
xref openshift/origin#14383
@derekwaynecarr @eparis @smarterclayton @saad-ali @jwforres
/release-note-none