diff options
author | Alexander Morozov <lk4d4math@gmail.com> | 2017-08-23 11:49:22 -0700 |
---|---|---|
committer | Ian Lance Taylor <iant@golang.org> | 2017-08-23 21:46:55 +0000 |
commit | 75fbc8a3d00cf310a5f67aae946fbc346dbb725d (patch) | |
tree | 98744a1c0f7bc589fec32e65dd9077f48cb6a5d6 /src/syscall/exec_linux_test.go | |
parent | 8f1e2a2610765528068107e33ab0d1d2ff224ce3 (diff) | |
download | go-git-75fbc8a3d00cf310a5f67aae946fbc346dbb725d.tar.gz |
syscall: skip some exec tests in container
For those tests there won't be enough permissions in containers.
I decided to go this way instead of just skipping os.IsPermission errors because
many of those tests were specifically written to check false positive permission
errors.
Fixes #21379
Change-Id: Ie25e1d6d47f85bb6b570352638440f3ac1e18e03
Reviewed-on: https://go-review.googlesource.com/58170
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src/syscall/exec_linux_test.go')
-rw-r--r-- | src/syscall/exec_linux_test.go | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/src/syscall/exec_linux_test.go b/src/syscall/exec_linux_test.go index 114deec5bb..79a7916dde 100644 --- a/src/syscall/exec_linux_test.go +++ b/src/syscall/exec_linux_test.go @@ -23,6 +23,24 @@ import ( "unsafe" ) +func isDocker() bool { + _, err := os.Stat("/.dockerenv") + return err == nil +} + +func isLXC() bool { + return os.Getenv("container") == "lxc" +} + +func skipInContainer(t *testing.T) { + if isDocker() { + t.Skip("skip this test in Docker container") + } + if isLXC() { + t.Skip("skip this test in LXC container") + } +} + // Check if we are in a chroot by checking if the inode of / is // different from 2 (there is no better test available to non-root on // linux). @@ -35,6 +53,7 @@ func isChrooted(t *testing.T) bool { } func checkUserNS(t *testing.T) { + skipInContainer(t) if _, err := os.Stat("/proc/self/ns/user"); err != nil { if os.IsNotExist(err) { t.Skip("kernel doesn't support user namespaces") @@ -147,6 +166,7 @@ func TestEmptyCredGroupsDisableSetgroups(t *testing.T) { } func TestUnshare(t *testing.T) { + skipInContainer(t) // Make sure we are running as root so we have permissions to use unshare // and create a network namespace. if os.Getuid() != 0 { @@ -293,6 +313,7 @@ func TestUnshareMountNameSpaceHelper(*testing.T) { // Test for Issue 38471: unshare fails because systemd has forced / to be shared func TestUnshareMountNameSpace(t *testing.T) { + skipInContainer(t) // Make sure we are running as root so we have permissions to use unshare // and create a network namespace. if os.Getuid() != 0 { @@ -342,6 +363,7 @@ func TestUnshareMountNameSpace(t *testing.T) { // Test for Issue 20103: unshare fails when chroot is used func TestUnshareMountNameSpaceChroot(t *testing.T) { + skipInContainer(t) // Make sure we are running as root so we have permissions to use unshare // and create a network namespace. if os.Getuid() != 0 { @@ -477,6 +499,7 @@ func TestAmbientCapsHelper(*testing.T) { } func TestAmbientCaps(t *testing.T) { + skipInContainer(t) // Make sure we are running as root so we have permissions to use unshare // and create a network namespace. if os.Getuid() != 0 { |