diff options
| author | Matthew Heon <matthew.heon@pm.me> | 2020-07-15 15:25:12 -0400 |
|---|---|---|
| committer | Valentin Rothberg <rothberg@redhat.com> | 2020-08-11 15:06:37 +0200 |
| commit | ca00601b14f2253e5b9f89280d39c5d498efc9f3 (patch) | |
| tree | c056c435a15d9a8d261d75c76253384dfc33c75f /contrib/cirrus/logformatter.t | |
| parent | 3dfd8630a51a37734ad8c51162c4d004b8ffffb2 (diff) | |
| download | podman-ca00601b14f2253e5b9f89280d39c5d498efc9f3.tar.gz podman-ca00601b14f2253e5b9f89280d39c5d498efc9f3.tar.bz2 podman-ca00601b14f2253e5b9f89280d39c5d498efc9f3.zip | |
Make changes to /etc/passwd on disk for non-read only
Bind-mounting /etc/passwd into the container is problematic
becuase of how system utilities like `useradd` work. They want
to make a copy and then rename to try to prevent breakage; this
is, unfortunately, impossible when the file they want to rename
is a bind mount. The current behavior is fine for read-only
containers, though, because we expect useradd to fail in those
cases.
Instead of bind-mounting, we can edit /etc/passwd in the
container's rootfs. This is kind of gross, because the change
will show up in `podman diff` and similar tools, and will be
included in images made by `podman commit`. However, it's a lot
better than breaking important system tools.
Fixes #6953
Signed-off-by: Matthew Heon <matthew.heon@pm.me>
Diffstat (limited to 'contrib/cirrus/logformatter.t')
0 files changed, 0 insertions, 0 deletions