]> Cypherpunks repositories - gostls13.git/commitdiff
syscall: remove use of IN_KUBERNETES in test
authorTobias Klauser <tklauser@distanz.ch>
Tue, 14 Sep 2021 06:56:54 +0000 (08:56 +0200)
committerTobias Klauser <tobias.klauser@gmail.com>
Tue, 14 Sep 2021 14:09:54 +0000 (14:09 +0000)
CL 201737 dropped the use of IN_KUBERNETES in tests, but it looks like
it did not catch all occurrences.

For #12815
For #34956

Change-Id: I72b89bfb850ba2890e9e6aa39b87167291ab7e9f
Reviewed-on: https://go-review.googlesource.com/c/go/+/349789
Trust: Tobias Klauser <tobias.klauser@gmail.com>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/syscall/exec_linux_test.go

index 85b59ad00d94650368e0305ba62b507c0b8824b0..1555318edae46715333873e39b61f15e8e02b8f9 100644 (file)
@@ -111,14 +111,6 @@ func checkUserNS(t *testing.T) {
                        t.Skip("kernel doesn't support user namespaces")
                }
        }
-
-       // When running under the Go continuous build, skip tests for
-       // now when under Kubernetes. (where things are root but not quite)
-       // Both of these are our own environment variables.
-       // See Issue 12815.
-       if os.Getenv("GO_BUILDER_NAME") != "" && os.Getenv("IN_KUBERNETES") == "1" {
-               t.Skip("skipping test on Kubernetes-based builders; see Issue 12815")
-       }
 }
 
 func whoamiCmd(t *testing.T, uid, gid int, setgroups bool) *exec.Cmd {
@@ -201,14 +193,6 @@ func TestUnshare(t *testing.T) {
                t.Skip("kernel prohibits unshare in unprivileged process, unless using user namespace")
        }
 
-       // When running under the Go continuous build, skip tests for
-       // now when under Kubernetes. (where things are root but not quite)
-       // Both of these are our own environment variables.
-       // See Issue 12815.
-       if os.Getenv("GO_BUILDER_NAME") != "" && os.Getenv("IN_KUBERNETES") == "1" {
-               t.Skip("skipping test on Kubernetes-based builders; see Issue 12815")
-       }
-
        path := "/proc/net/dev"
        if _, err := os.Stat(path); err != nil {
                if os.IsNotExist(err) {