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

Skip subgid check #51

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Skip subgid check #51

wants to merge 3 commits into from

Commits on Dec 16, 2021

  1. Add variable skip_subgid_change with default value true 

    Adding subgids and subguids is not needed anymore since useradd
adds them automatically.

The addition implemented here only works for the first user (id 1000 with
subids starting from 100000).
For other users it leads to overlapping subuid/subgid ranges
which might lead to very confusing issues when running containers.

The addition is left here in case someone is using it or wants
to override default ids with some specific values.

Fixes ikke-t#34
    Johannes Aalto committed Dec 16, 2021
    Configuration menu
    Copy the full SHA
    0d6e216 View commit details
    Browse the repository at this point in the history

  2. docs: add alternative to include-role 

    It feels appropriate to mention this alternative since
the author of the anti-pattern article seems to propose this approach too
(in the more recent comments of the article).
    Johannes Aalto committed Dec 16, 2021
    Configuration menu
    Copy the full SHA
    1db8536 View commit details
    Browse the repository at this point in the history

  3. fix subuid/subgid checks 

    "find" module only checks that files exist in a directory.
In this case it did not fail since the path was a file but there was
a warning:

  TASK [ikke_t.podman_container_systemd : check if user is in subuid file] ***********************
  [WARNING]: Skipped '/etc/subuid' path due to this access issue: '/etc/subuid' is not a directory
  ok: [hostname]
    Johannes Aalto committed Dec 16, 2021
    Configuration menu
    Copy the full SHA
    05e11de View commit details
    Browse the repository at this point in the history