gvisor/kokoro: don't modify tests names in the BUILD file

PiperOrigin-RevId: 253746380

runsc/test/build_defs.bzl

@@ -0,0 +1,19 @@
+"""Defines a rule for runsc test targets."""
+
+load("@io_bazel_rules_go//go:def.bzl", _go_test = "go_test")
+
+# runtime_test is a macro that will create targets to run the given test target
+# with different runtime options.
+def runtime_test(**kwargs):
+    """Runs the given test target with different runtime options."""
+    name = kwargs["name"]
+    _go_test(**kwargs)
+    kwargs["name"] = name + "_hostnet"
+    kwargs["args"] = ["--runtime-type=hostnet"]
+    _go_test(**kwargs)
+    kwargs["name"] = name + "_kvm"
+    kwargs["args"] = ["--runtime-type=kvm"]
+    _go_test(**kwargs)
+    kwargs["name"] = name + "_overlay"
+    kwargs["args"] = ["--runtime-type=overlay"]
+    _go_test(**kwargs)

runsc/test/image/BUILD

@@ -1,8 +1,9 @@
-load("@io_bazel_rules_go//go:def.bzl", "go_library", "go_test")
+load("@io_bazel_rules_go//go:def.bzl", "go_library")
+load("//runsc/test:build_defs.bzl", "runtime_test")
 
 package(licenses = ["notice"])
 
-go_test(
+runtime_test(
     name = "image_test",
     size = "large",
     srcs = [

runsc/test/integration/BUILD

@@ -1,8 +1,9 @@
-load("@io_bazel_rules_go//go:def.bzl", "go_library", "go_test")
+load("@io_bazel_rules_go//go:def.bzl", "go_library")
+load("//runsc/test:build_defs.bzl", "runtime_test")
 
 package(licenses = ["notice"])
 
-go_test(
+runtime_test(
     name = "integration_test",
     size = "large",
     srcs = [

runsc/test/testutil/docker.go

@@ -15,6 +15,7 @@
 package testutil
 
 import (
+	"flag"
 	"fmt"
 	"io/ioutil"
 	"log"
@@ -30,10 +31,15 @@ import (
 	"github.com/kr/pty"
 )
 
+var runtimeType = flag.String("runtime-type", "", "specify which runtime to use: kvm, hostnet, overlay")
+
 func getRuntime() string {
 	r, ok := os.LookupEnv("RUNSC_RUNTIME")
 	if !ok {
-		return "runsc-test"
+		r = "runsc-test"
+	}
+	if *runtimeType != "" {
+		r += "-" + *runtimeType
 	}
 	return r
 }

tools/run_tests.sh

@@ -181,20 +181,24 @@ run_docker_tests() {
 
   # These names are used to exclude tests not supported in certain
   # configuration, e.g. save/restore not supported with hostnet.
-  declare -a variations=("" "-kvm" "-hostnet" "-overlay")
-  for v in "${variations[@]}"; do
-    # Change test names otherwise each run of tests will overwrite logs and
-    # results of the previous run.
-    sed -i "s/name = \"integration_test.*\"/name = \"integration_test${v}\"/" runsc/test/integration/BUILD
-    sed -i "s/name = \"image_test.*\"/name = \"image_test${v}\"/" runsc/test/image/BUILD
-    # Run runsc tests with docker that are tagged manual.
-    bazel test \
-      "${BAZEL_BUILD_FLAGS[@]}" \
-      --test_env=RUNSC_RUNTIME="${RUNTIME}${v}" \
-      --test_output=all \
-      //runsc/test/image:image_test${v} \
-      //runsc/test/integration:integration_test${v}
-  done
+  # Run runsc tests with docker that are tagged manual.
+  #
+  # The --nocache_test_results option is used here to eliminate cached results
+  # from the previous run for the runc runtime.
+  bazel test \
+    "${BAZEL_BUILD_FLAGS[@]}" \
+    --test_env=RUNSC_RUNTIME="${RUNTIME}" \
+    --test_output=all \
+    --nocache_test_results \
+    --test_output=streamed \
+    //runsc/test/integration:integration_test \
+    //runsc/test/integration:integration_test_hostnet \
+    //runsc/test/integration:integration_test_overlay \
+    //runsc/test/integration:integration_test_kvm \
+    //runsc/test/image:image_test \
+    //runsc/test/image:image_test_overlay \
+    //runsc/test/image:image_test_hostnet \
+    //runsc/test/image:image_test_kvm
 }
 
 # Run the tests that require root.