From a5fe397cf806cbc488757dbd73a3d83eb5da11cd Mon Sep 17 00:00:00 2001 From: Nicolas Lacasse Date: Wed, 24 Oct 2018 23:06:51 -0700 Subject: [PATCH] Re-enable TestPythonHello now that ptrace seccomp issue is resolved. PiperOrigin-RevId: 218636184 Change-Id: I44deac3f32276d06955c5fb1e28c5970bb08f5fd --- runsc/test/image/image_test.go | 6 ------ runsc/test/testutil/docker.go | 9 --------- 2 files changed, 15 deletions(-) diff --git a/runsc/test/image/image_test.go b/runsc/test/image/image_test.go index 763152b47..f7e750d71 100644 --- a/runsc/test/image/image_test.go +++ b/runsc/test/image/image_test.go @@ -192,12 +192,6 @@ func TestMysql(t *testing.T) { } func TestPythonHello(t *testing.T) { - // TODO: This test occasionally hangs on the ptrace - // platform. Re-enable it once we have this issue fixed. - if testutil.IsPtracePlatform() { - t.Skipf("Skipping PythonHello test on ptrace platform") - } - if err := testutil.Pull("google/python-hello"); err != nil { t.Fatalf("docker pull failed: %v", err) } diff --git a/runsc/test/testutil/docker.go b/runsc/test/testutil/docker.go index 3f74e0770..9a76397be 100644 --- a/runsc/test/testutil/docker.go +++ b/runsc/test/testutil/docker.go @@ -49,15 +49,6 @@ func IsPauseResumeSupported() bool { return !strings.Contains(getRuntime(), "hostnet") } -// IsPtracePlatform returns true if the runtime is using ptrace platform. -// -// TODO: Tests should not depend on the platform, but -// TestPythonHello sometimes hangs on ptrace. Once that is debugged, this -// method should go away. -func IsPtracePlatform() bool { - return !strings.Contains(getRuntime(), "kvm") -} - // EnsureSupportedDockerVersion checks if correct docker is installed. func EnsureSupportedDockerVersion() { cmd := exec.Command("docker", "version")