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

notify_on_oom_v2 is being used in cgroup v2 #136

Open
Eason0729 opened this issue May 16, 2024 · 0 comments
Open

notify_on_oom_v2 is being used in cgroup v2 #136

Eason0729 opened this issue May 16, 2024 · 0 comments
Labels
bug Something isn't working needs-review

Comments

@Eason0729
Copy link

Describe the bug
memory.oom_control is used in notify_on_oom_v2, if I recall correctly, memory.oom_control only exist in cgroup v1 when memory subsystem is mounted.

Expected behavior
Return ErrorKind::Other, or document the behavior somewhere.

Additional context

called `Result::unwrap()` on an `Err` value: Error { kind: ReadFailed("/sys/fs/cgroup/abc.0/memory.oom_control"), cause: Some(Os { code: 2, kind: NotFound, message: "No such file or directory" }) }
@Eason0729 Eason0729 added bug Something isn't working needs-review labels May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-review
Projects
None yet
Development

No branches or pull requests

1 participant