diff options
| author | Daniel J Walsh <dwalsh@redhat.com> | 2021-02-15 16:22:44 -0500 |
|---|---|---|
| committer | Matthew Heon <mheon@redhat.com> | 2021-02-18 10:48:14 -0500 |
| commit | 538ced1258b6fcf698e9fee18f98bd4f534c674d (patch) | |
| tree | 07d0df72fbf24e6ea36f40111c62fb362060152d /test | |
| parent | af37b6762f70b7dd31272d403c4c89753a54b281 (diff) | |
| download | podman-538ced1258b6fcf698e9fee18f98bd4f534c674d.tar.gz podman-538ced1258b6fcf698e9fee18f98bd4f534c674d.tar.bz2 podman-538ced1258b6fcf698e9fee18f98bd4f534c674d.zip | |
Don't chown workdir if it already exists
Currently podman is always chowning the WORKDIR to root:root
This PR will return if the WORKDIR already exists.
Fixes: https://github.com/containers/podman/issues/9387
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/e2e/run_working_dir_test.go | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/test/e2e/run_working_dir_test.go b/test/e2e/run_working_dir_test.go index 59538448e..948ed05e7 100644 --- a/test/e2e/run_working_dir_test.go +++ b/test/e2e/run_working_dir_test.go @@ -47,7 +47,7 @@ var _ = Describe("Podman run", func() { It("podman run a container on an image with a workdir", func() { dockerfile := `FROM alpine -RUN mkdir -p /home/foobar +RUN mkdir -p /home/foobar /etc/foobar; chown bin:bin /etc/foobar WORKDIR /etc/foobar` podmanTest.BuildImage(dockerfile, "test", "false") @@ -56,6 +56,10 @@ WORKDIR /etc/foobar` Expect(session.ExitCode()).To(Equal(0)) Expect(session.OutputToString()).To(Equal("/etc/foobar")) + session = podmanTest.Podman([]string{"run", "test", "ls", "-ld", "."}) + session.WaitWithDefaultTimeout() + Expect(session.LineInOutputContains("bin")).To(BeTrue()) + session = podmanTest.Podman([]string{"run", "--workdir", "/home/foobar", "test", "pwd"}) session.WaitWithDefaultTimeout() Expect(session.ExitCode()).To(Equal(0)) |