Skip to content

[DO NOT MERGE] vendor: test opencontainers/cgroups#64#5340

Draft
kolyshkin wants to merge 2 commits into
opencontainers:mainfrom
kolyshkin:test-cgroups-drop-ebpf
Draft

[DO NOT MERGE] vendor: test opencontainers/cgroups#64#5340
kolyshkin wants to merge 2 commits into
opencontainers:mainfrom
kolyshkin:test-cgroups-drop-ebpf

Conversation

@kolyshkin

Copy link
Copy Markdown
Contributor

Let's drop the second biggest (by size) runc dependency and reduce the binary size by another ~1MB.

Draft/DNM until opencontainers/cgroups#64 is merged/released.

rata and others added 2 commits June 23, 2026 00:30
This new version includes fixes for EPBF fd leaks:
	opencontainers/cgroups#60

Let's update to it.

However, the Stats() method was added in the new release to the
interface, so tests were failing with:
	# github.com/opencontainers/runc/libcontainer [github.com/opencontainers/runc/libcontainer.test]
./container_linux_test.go:125:18: cannot use &mockCgroupManager{…} (value of type *mockCgroupManager) as cgroups.Manager value in struct literal: *mockCgroupManager does not implement cgroups.Manager (missing method Stats)

Let's add the method to the mock too.

In the future, probably the mock should come from the cgroups module.

Signed-off-by: Rodrigo Campos <rodrigo@amutable.com>
Let's drop the second biggest (by size) runc dependency and reduce the
binary size by another ~1MB.

Draft/DNM until opencontainers/cgroups#64 is merged/released.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin kolyshkin force-pushed the test-cgroups-drop-ebpf branch from 3345e95 to b376143 Compare June 23, 2026 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants