Skip to content

[1.5] libct/int: don't exclude BPF fds from leak check - #5490

Merged
thaJeztah merged 2 commits into
opencontainers:release-1.5from
kolyshkin:1.5-5487
Sep 24, 2026
Merged

thaJeztah merged 2 commits into
opencontainers:release-1.5from
kolyshkin:1.5-5487

Conversation

@kolyshkin

Copy link
Copy Markdown
Contributor

Backport of #5487 to release-1.5. Original description follows.


This is a carry of #5322 (by @rata).

The cgroups dependency bump from that PR is no longer needed, since
opencontainers/cgroups v0.1.0 (which includes the eBPF fd leak fixes
from opencontainers/cgroups#60) was already vendored in #5403.

What remains is:

  • removing the anon_inode:bpf-prog exclusion from the fd leak test;
  • a CHANGELOG entry.

Closes #5322.

Now the cgroups dependency has been upgraded and it doesn't cause these
leaks anymore. Let's remove the exclude.

Signed-off-by: Rodrigo Campos <rodrigo@amutable.com>
(cherry picked from commit f9b4cf6)
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin kolyshkin added the backport/1.5-pr A backport PR to release-1.5 label Sep 24, 2026
Signed-off-by: Rodrigo Campos <rodrigo@amutable.com>
(cherry picked from commit 884bf2e)
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin

Copy link
Copy Markdown
Contributor Author

I hope this is the last one before #5466 (needed solely for CHANGELOG entry); @opencontainers/runc-maintainers PTAL

@kolyshkin kolyshkin mentioned this pull request Sep 24, 2026

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztah
thaJeztah merged commit 59e1324 into opencontainers:release-1.5 Sep 24, 2026
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci backport/1.5-pr A backport PR to release-1.5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants