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.

Sign up for GitHub

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

Jump to bottom

client: ensure minimal cgroup controllers enabled #15027

Merged
merged 3 commits into from
Oct 24, 2022
Merged

client: ensure minimal cgroup controllers enabled #15027

merged 3 commits into from
Oct 24, 2022

Commits on Oct 24, 2022

  1. client: ensure minimal cgroup controllers enabled 

    This PR fixes a bug where Nomad could not operate properly on operating
systems that set the root cgroup.subtree_control to a set of controllers that
do not include the minimal set of controllers needed by Nomad.

Nomad needs these controllers enabled to operate:
- cpuset
- cpu
- io
- memory
- pids

Now, Nomad will ensure these controllers are enabled during Client initialization,
adding them to cgroup.subtree_control as necessary. This should be particularly
helpful on the RHEL/CentOS/Fedora family of system. Ubuntu systems should be
unaffected as they enable all controllers by default.

Fixes: #14494
    @shoenig
    shoenig committed Oct 24, 2022
    Configuration menu
    Copy the full SHA
    fb67296 View commit details
    Browse the repository at this point in the history

  2. docs: cleanup doc string

    @shoenig
    shoenig committed Oct 24, 2022
    Configuration menu
    Copy the full SHA
    ba8acad View commit details
    Browse the repository at this point in the history

  3. client: cleanup controller writes, enhance log messages

    @shoenig
    shoenig committed Oct 24, 2022
    Configuration menu
    Copy the full SHA
    d274e95 View commit details
    Browse the repository at this point in the history