diff options
author | Brad Fitzpatrick <bradfitz@golang.org> | 2017-07-14 19:02:05 +0000 |
---|---|---|
committer | Brad Fitzpatrick <bradfitz@golang.org> | 2017-07-14 21:05:21 +0000 |
commit | 121d076679a376ef5549c858195c5a8357cba36e (patch) | |
tree | 0a6ebd8d9ffe9c67cb84fa6a0f45476f3816e741 | |
parent | 7914369e64190ce22e6fd6bba685e3f82338875a (diff) | |
download | go-121d076679a376ef5549c858195c5a8357cba36e.tar.gz go-121d076679a376ef5549c858195c5a8357cba36e.zip |
syscall: skip TestUnshare on Linux upon permission error
Fixes #17206
Change-Id: Id0ebc3a55ea1c5f52608decffee04c8398a8774b
Reviewed-on: https://go-review.googlesource.com/48571
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
-rw-r--r-- | src/syscall/exec_linux_test.go | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/syscall/exec_linux_test.go b/src/syscall/exec_linux_test.go index f5e97bec1f..5c7d8a29c1 100644 --- a/src/syscall/exec_linux_test.go +++ b/src/syscall/exec_linux_test.go @@ -182,6 +182,12 @@ func TestUnshare(t *testing.T) { } out, err := cmd.CombinedOutput() if err != nil { + if strings.Contains(err.Error(), "operation not permitted") { + // Issue 17206: despite all the checks above, + // this still reportedly fails for some users. + // (older kernels?). Just skip. + t.Skip("skipping due to permission error") + } t.Fatalf("Cmd failed with err %v, output: %s", err, out) } |