Skip to content

Commit

Permalink
Use specific image for user namespaces tests
Browse files Browse the repository at this point in the history
Due to the lack of an existing /proc dir in the rootfs of busybox, there
seems to be a race between the mount of /proc and whether the root dir
is already readonly? May need investigation, but for now use an image
that has a /proc dir existing.

Signed-off-by: Phil Estes <estesp@amazon.com>
  • Loading branch information
estesp committed Mar 20, 2021
1 parent a47feaf commit 33776ad
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion integration/client/container_linux_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,8 @@ import (
"golang.org/x/sys/unix"
)

const testUserNSImage = "mirror.gcr.io/library/alpine:latest"

// TestRegressionIssue4769 verifies the number of task exit events.
//
// Issue: https://github.com/containerd/containerd/issues/4769.
Expand Down Expand Up @@ -1596,7 +1598,7 @@ func testUserNamespaces(t *testing.T, readonlyRootFS bool) {
)
defer cancel()

image, err = client.GetImage(ctx, testImage)
image, err = client.Pull(ctx, testUserNSImage, WithPullUnpack)
if err != nil {
t.Fatal(err)
}
Expand Down

0 comments on commit 33776ad

Please sign in to comment.