diff options
| author | Valentin Rothberg <rothberg@redhat.com> | 2021-01-21 15:41:14 +0100 |
|---|---|---|
| committer | Matthew Heon <matthew.heon@pm.me> | 2021-01-29 15:24:04 -0500 |
| commit | c48753bfa0d56b855f71dc6ad62e8eff62cf98e1 (patch) | |
| tree | 39e2dc6733a6485e71f60eb725ce5b85f613d3f6 /test/system/070-build.bats | |
| parent | 0b500515d8b8900851763a17b61609556f58d99c (diff) | |
| download | podman-c48753bfa0d56b855f71dc6ad62e8eff62cf98e1.tar.gz podman-c48753bfa0d56b855f71dc6ad62e8eff62cf98e1.tar.bz2 podman-c48753bfa0d56b855f71dc6ad62e8eff62cf98e1.zip | |
workdir presence checks
A container's workdir can be specified via the CLI via `--workdir` and
via an image config with the CLI having precedence.
Since images have a tendency to specify workdirs without necessarily
shipping the paths with the root FS, make sure that Podman creates the
workdir. When specified via the CLI, do not create the path, but check
for its existence and return a human-friendly error.
NOTE: `crun` is performing a similar check that would yield exit code
127. With this change, however, Podman performs the check and yields
exit code 126. Since this is specific to `crun`, I do not consider it
to be a breaking change of Podman.
Fixes: #9040
Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
Diffstat (limited to 'test/system/070-build.bats')
| -rw-r--r-- | test/system/070-build.bats | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/test/system/070-build.bats b/test/system/070-build.bats index 0e83a184b..bf9fa789c 100644 --- a/test/system/070-build.bats +++ b/test/system/070-build.bats @@ -145,10 +145,12 @@ EOF https_proxy=https-proxy-in-env-file EOF + # NOTE: it's important to not create the workdir. + # Podman will make sure to create a missing workdir + # if needed. See #9040. cat >$tmpdir/Containerfile <<EOF FROM $IMAGE LABEL $label_name=$label_value -RUN mkdir $workdir WORKDIR $workdir # Test for #7094 - chowning of invalid symlinks |