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

cgroup: Stumble on if we can't set up oom handling #385

Merged
merged 1 commit into from Feb 27, 2023

Conversation

cgwalters
Copy link
Contributor

OpenShift 4.13 tentatively plans to use cgroupv1 on a RHEL9 era kernel, which breaks only in PREEMPT_RT aka kernel-rt due to
torvalds/linux@2343e88

For now let's stumble on without oom notifications.

Copy link
Member

@giuseppe giuseppe left a comment

Choose a reason for hiding this comment

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

LGTM

@haircommander
Copy link
Collaborator

seems to be a fmt error

OpenShift 4.13 tentatively plans to use cgroupv1 on a RHEL9
era kernel, which breaks only in `PREEMPT_RT` aka `kernel-rt`
due to
torvalds/linux@2343e88

For now let's stumble on without oom notifications.

Signed-off-by: Colin Walters <walters@verbum.org>
@cgwalters
Copy link
Contributor Author

seems to be a fmt error

Fixed

@haircommander haircommander merged commit 5cff0ce into containers:main Feb 27, 2023
@rhatdan
Copy link
Member

rhatdan commented Feb 28, 2023

Why cgroups V1 on RHEL 9?

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.

None yet

4 participants