Skip to content

[release/2.0] Fix issue preventing some v2 shims from shutting down properly#11741

Merged
samuelkarp merged 1 commit intocontainerd:release/2.0from
samuelkarp:issue-11708/2.0
Apr 22, 2025
Merged

[release/2.0] Fix issue preventing some v2 shims from shutting down properly#11741
samuelkarp merged 1 commit intocontainerd:release/2.0from
samuelkarp:issue-11708/2.0

Conversation

@samuelkarp
Copy link
Copy Markdown
Member

(cherry picked from commit 4b4e6f7)

This is a manual cherry-pick of #11735 since there were merge conflicts. Opening as a draft until #11735 is merged.

Signed-off-by: yang yang <[email protected]>
(cherry picked from commit 4b4e6f7)
Signed-off-by: Samuel Karp <[email protected]>
@k8s-ci-robot
Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@dmcgowan
Copy link
Copy Markdown
Member

Can you title this based on the fix?

@github-project-automation github-project-automation Bot moved this from Needs Reviewers to Review In Progress in Pull Request Review Apr 22, 2025
@samuelkarp samuelkarp changed the title [release/2.0] do not set sandbox id when using podsandbox type [release/2.0] Fix issue preventing some v2 shims from shutting down properly Apr 22, 2025
@samuelkarp samuelkarp merged commit 9b633a9 into containerd:release/2.0 Apr 22, 2025
100 of 103 checks passed
@github-project-automation github-project-automation Bot moved this from Review In Progress to Done in Pull Request Review Apr 22, 2025
hugomoran159 added a commit to hugomoran159/containerd that referenced this pull request Mar 11, 2026
Don't set sandbox ID on containers when using the internal podsandbox
sandboxer. gVisor's shim doesn't implement the Sandbox Manager API,
so linking containers to it causes kubelet to detect SandboxChanged
and restart pods every ~2 minutes.

Cherry-picked from containerd PR containerd#11741 / Talos siderolabs/pkgs#1235.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cri Container Runtime Interface (CRI) impact/changelog size/S

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants