Skip to content

[1.5] tests/int: fix flake in "resources.unified override" (backport #5269)#5286

Merged
rata merged 1 commit into
opencontainers:release-1.5from
lifubang:backport-5269-1.5
May 22, 2026
Merged

[1.5] tests/int: fix flake in "resources.unified override" (backport #5269)#5286
rata merged 1 commit into
opencontainers:release-1.5from
lifubang:backport-5269-1.5

Conversation

@lifubang
Copy link
Copy Markdown
Member

Summary

(cherry picked from commit 3fabb4d070a8a0e583e4c639949e9306610cba)

@lifubang lifubang force-pushed the backport-5269-1.5 branch from d6d6ae5 to 3229d47 Compare May 18, 2026 13:51
@lifubang lifubang changed the title tests/int: fix flake in "resources.unified override" (backport #5269) [1.5] tests/int: fix flake in "resources.unified override" (backport #5269) May 18, 2026
@lifubang lifubang added the backport/1.5-pr A backport PR to release-1.5 label May 18, 2026
@kolyshkin kolyshkin modified the milestones: 1.3.6, 1.5.0 May 18, 2026
Copy link
Copy Markdown
Member

@rata rata left a comment

Choose a reason for hiding this comment

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

LGTM

@rata rata enabled auto-merge May 22, 2026 08:28
@rata
Copy link
Copy Markdown
Member

rata commented May 22, 2026

I can't understand why this PR can't be merged, all checks pass and it has 2 approvals. It doesn't have conflicts, but the branch is out of date with the base branch. I enabled auto-merge, but that didn't do the trick. I'll click "rebase branch", as I think this is just a github bug and the rebase shouldn't cause any merge commit (the other option is to merge latest 1.5 with this). Let's see if I can merge after that

As runc binary grows in size over time (new features, more
dependencies) some tests start to flake because of low memory limits.

One such test is "runc run (cgroup v2 resources.unified override)";
it obviously fails because of 1M memory limit:

> runc run failed: unable to start container process: container init was OOM-killed (memory limit too low?)

Increase the limits 4x. Do the same for the "unified only" test.

Fixes issue 5264.

Reported-by: Kevin Berry <kpberry11@gmail.com>
Reported-by: Ricardo Branco <rbranco@suse.de>
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
(cherry picked from commit 3fabb4d)
Signed-off-by: lifubang <lifubang@acmcoder.com>
Co-authored-by: Claude <noreply@anthropic.com>
@rata rata force-pushed the backport-5269-1.5 branch from 3229d47 to 7522b50 Compare May 22, 2026 08:32
@rata rata merged commit 2c0d0d7 into opencontainers:release-1.5 May 22, 2026
52 of 54 checks passed
@rata
Copy link
Copy Markdown
Member

rata commented May 22, 2026

Okay, the rebase did the trick and github realized it could be merged

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.

3 participants