Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI fails on almalinux 9 #4454

Open
kolyshkin opened this issue Oct 18, 2024 · 2 comments · May be fixed by #4455
Open

CI fails on almalinux 9 #4454

kolyshkin opened this issue Oct 18, 2024 · 2 comments · May be fixed by #4455
Labels

Comments

@kolyshkin
Copy link
Contributor

kolyshkin commented Oct 18, 2024

Almalinux-9 CI job (which runs on Cirrus CI) is consistently timing out for the last week or so.

I took a look, and it seems it's caused by a recent kernel update:

  • was working file with 5.14.0-427.33.1.el9_4.x86_64;
  • started failing (timing out) with 5.14.0-427.37.1.el9_4.x86_64.

The CI job times out while executing the "update paused container" integration test, here it is:

https://github.com/opencontainers/runc/blob/d82235c9d0e249c89f136811b97c9277eae80f8c/tests/integration/update.bats#L830C1-L853C1

I have seen it being timed out on both systemd and fs driver, and on both root and rootless tests.

@kolyshkin
Copy link
Contributor Author

Unable to reproduce locally, but disabling the test for almalinux-9 apparently fixes the issue.

@kolyshkin
Copy link
Contributor Author

The CI job times out while executing the "update paused container" integration test

I was wrong, this is probably the next test, i.e. "update memory vs CheckBeforeUpdate". Here it is:

@test "update memory vs CheckBeforeUpdate" {
requires cgroups_v2
[ $EUID -ne 0 ] && requires rootless_cgroup
runc run -d --console-socket "$CONSOLE_SOCKET" test_update
[ "$status" -eq 0 ]
# Setting memory to low value with checkBeforeUpdate=true should fail.
runc update -r - test_update <<EOF
{
"memory": {
"limit": 1024,
"checkBeforeUpdate": true
}
}
EOF
[ "$status" -ne 0 ]
[[ "$output" == *"rejecting memory limit"* ]]
testcontainer test_update running
# Setting memory+swap to low value with checkBeforeUpdate=true should fail.
runc update -r - test_update <<EOF
{
"memory": {
"limit": 1024,
"swap": 2048,
"checkBeforeUpdate": true
}
}
EOF
[ "$status" -ne 0 ]
[[ "$output" == *"rejecting memory+swap limit"* ]]
testcontainer test_update running
# The container will be OOM killed, and runc might either succeed
# or fail depending on the timing, so we don't check its exit code.
runc update test_update --memory 1024
wait_for_container 10 1 test_update stopped
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant