Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

cgroups: fix failed to remove sandbox cgroup #1264

Merged
merged 1 commit into from
Feb 21, 2019
Merged

cgroups: fix failed to remove sandbox cgroup #1264

merged 1 commit into from
Feb 21, 2019

Conversation

Ace-Tang
Copy link
Contributor

sandbox cgroup use V1NoConstraints, this only create memory subsystem,
but when delete, load parent cgroup always use cgroups.V1, so other
subsystem path can not be find, sandbox cgroup can not be deleted.

Fixes: #1263

Signed-off-by: Ace-Tang [email protected]

sandbox cgroup use V1NoConstraints, this only create memory subsystem,
but when delete, load parent cgroup always use `cgroups.V1`, so other
subsystem path can not be find, sandbox cgroup can not be deleted.

Fixes: #1263

Signed-off-by: Ace-Tang <[email protected]>
@Ace-Tang
Copy link
Contributor Author

Hey, @devimc , could you please to check if this is correct

@devimc
Copy link

devimc commented Feb 21, 2019

@Ace-Tang sure I'll review it, changing state to do-not-merge and assigning it to me

@devimc devimc self-requested a review February 21, 2019 13:44
@devimc devimc self-assigned this Feb 21, 2019
@devimc
Copy link

devimc commented Feb 21, 2019

@Ace-Tang you're right, good catch!

@devimc
Copy link

devimc commented Feb 21, 2019

/test

@devimc
Copy link

devimc commented Feb 21, 2019

@kata-containers/runtime one more review

@jcvenegas
Copy link
Member

good finding! lgtm

@devimc devimc merged commit 16bd983 into kata-containers:master Feb 21, 2019
@Ace-Tang
Copy link
Contributor Author

Thank for review @devimc @jcvenegas

@egernst egernst mentioned this pull request Feb 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants