From 3e4afe2307c3db0208bc33b06829ec00f8dd6f37 Mon Sep 17 00:00:00 2001 From: Brad Fitzpatrick Date: Thu, 30 Mar 2017 18:00:10 -0700 Subject: [PATCH] syscall: skip test on TestUnshareMountNameSpace permission error TestUnshareMountNameSpace fails on arm64 due to permission problems. Skip that test for now when permission problems are encountered, so we don't regress elsewhere in the meantime. Updates #19698 Change-Id: I9058928afa474b813652c9489f343b8957160a6c Reviewed-on: https://go-review.googlesource.com/39052 Run-TryBot: Brad Fitzpatrick TryBot-Result: Gobot Gobot Reviewed-by: Ian Lance Taylor Reviewed-by: Josh Bleecher Snyder --- src/syscall/exec_linux_test.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/syscall/exec_linux_test.go b/src/syscall/exec_linux_test.go index ed44ddf7f3..9551e7ec71 100644 --- a/src/syscall/exec_linux_test.go +++ b/src/syscall/exec_linux_test.go @@ -297,7 +297,10 @@ func TestUnshareMountNameSpace(t *testing.T) { o, err := cmd.CombinedOutput() if err != nil { - t.Fatalf("unshare failed: %v, %v", o, err) + if strings.Contains(err.Error(), ": permission denied") { + t.Skipf("Skipping test (golang.org/issue/19698); unshare failed due to permissions: %s, %v", o, err) + } + t.Fatalf("unshare failed: %s, %v", o, err) } // How do we tell if the namespace was really unshared? It turns out -- 2.48.1