Skip to content

Conversation

lifupan
Copy link
Member

@lifupan lifupan commented May 22, 2025

When try to delete a cgroup, it's needed to move all of the tasks/procs in the cgroup into root cgroup and then delete it.

Since for cgroup v2, it doesn't support to move thread into root cgroup, thus move the processes instead of moving tasks can fix this issue.

@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label May 22, 2025
When try to delete a cgroup, it's needed to move all of the
tasks/procs in the cgroup into root cgroup and then delete it.

Since for cgroup v2, it doesn't  support to move thread into
root cgroup, thus move the processes instead of moving tasks
can fix this issue.

Signed-off-by: Fupan Li <fupan.lfp@antgroup.com>
Copy link
Contributor

@Apokleos Apokleos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM @lifupan

@Apokleos Apokleos requested a review from justxuewei May 27, 2025 01:36
Copy link
Member

@RuoqingHe RuoqingHe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me, thanks @lifupan

@RuoqingHe RuoqingHe merged commit 139dc13 into kata-containers:main May 27, 2025
804 of 842 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/tiny Smallest and simplest task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants