-
Notifications
You must be signed in to change notification settings - Fork 39.4k
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 #92614: Don't create a new sandbox for pod with #95508
Automated cherry pick of #92614: Don't create a new sandbox for pod with #95508
Conversation
… containers succeeded The kubelet would attempt to create a new sandbox for a pod whose RestartPolicy is OnFailure even after all container succeeded. It caused unnecessary CRI and CNI calls, confusing logs and conflicts between the routine that creates the new sandbox and the routine that kills the Pod. This patch checks the containers to start and stops creating sandbox if no container is supposed to start.
3439ea8
to
085d9ed
Compare
/kind bug |
from parent PR |
@dchen1107 can you review this PR as well since you approved the other 2 prs for another release branches? |
@SergeyKanzhelev @dashpole @dims Can you please approve/merge this for 1.17.14? |
/approve |
/assign @derekwaynecarr |
@puerco @gianarb @jimangel @markyjackson-taulia @mkorbi @onlydole @saschagrunert Could we please get a cherry-pick review and merge this into 1.17.14 now that it is approved? P.S. This is already merged into 1.19.4 as per #94725. |
@alex-mohr dont need to ping everybody |
@dashpole / @dchen1107 can you please review this PR as well since you both reviewed #94959 ? thanks! |
It's not approved yet. Somebody from this list need to approve: kubernetes/test/e2e/node/OWNERS Lines 4 to 7 in 54f3c09
|
@SergeyKanzhelev Sorry, you are absolutely correct. The joys of trying to approve 3 identical pull requests for 3 different k8s branches; easy to get confused. |
I hear you. @derekwaynecarr is typically bulk approve things on Monday or Tuesday before the SIG node meeting. |
@dchen1107 can you review this cherry pick since you reviewed #94959 |
1 similar comment
@dchen1107 can you review this cherry pick since you reviewed #94959 |
1.18 pick has been approved but not yet merged. /approve |
/priority important-soon |
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cpanato, dashpole, derekwaynecarr, tnqn 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 |
Cherry pick of #92614 on release-1.17.
#92614: Don't create a new sandbox for pod with
For details on the cherry pick process, see the cherry pick requests page.