Merge pull request #48730 from shyamjvs/reposition-scale-tests

Automatic merge from submit-queue

Move performance tests to test/e2e/scalability subdirectory

Following the move to make e2e tests more organized and accountable to the respective SIGs.

cc @kubernetes/sig-scalability-pr-reviews @wojtek-t @gmarek @grodrigues3

test/e2e/BUILD

@@ -25,7 +25,7 @@ go_test(
         "//test/e2e/extension:go_default_library",
         "//test/e2e/framework:go_default_library",
         "//test/e2e/metrics:go_default_library",
-        "//test/e2e/perf:go_default_library",
+        "//test/e2e/scalability:go_default_library",
         "//test/e2e/scheduling:go_default_library",
         "//test/e2e/storage:go_default_library",
         "//test/e2e/workload:go_default_library",
@@ -58,7 +58,6 @@ go_library(
         "dns_common.go",
         "dns_configmap.go",
         "e2e.go",
-        "empty.go",
         "empty_dir_wrapper.go",
         "etcd_failure.go",
         "events.go",
@@ -140,7 +139,6 @@ go_library(
         "//test/e2e/generated:go_default_library",
         "//test/e2e/manifest:go_default_library",
         "//test/e2e/metrics:go_default_library",
-        "//test/e2e/perf:go_default_library",
         "//test/e2e/scheduling:go_default_library",
         "//test/e2e/upgrades:go_default_library",
         "//test/e2e/workload:go_default_library",
@@ -229,8 +227,8 @@ filegroup(
         "//test/e2e/generated:all-srcs",
         "//test/e2e/manifest:all-srcs",
         "//test/e2e/metrics:all-srcs",
-        "//test/e2e/perf:all-srcs",
         "//test/e2e/perftype:all-srcs",
+        "//test/e2e/scalability:all-srcs",
         "//test/e2e/scheduling:all-srcs",
         "//test/e2e/storage:all-srcs",
         "//test/e2e/testing-manifests:all-srcs",

test/e2e/e2e_test.go

@@ -24,7 +24,7 @@ import (
 	_ "k8s.io/kubernetes/test/e2e/cluster-logging"
 	_ "k8s.io/kubernetes/test/e2e/extension"
 	"k8s.io/kubernetes/test/e2e/framework"
-	_ "k8s.io/kubernetes/test/e2e/perf"
+	_ "k8s.io/kubernetes/test/e2e/scalability"
 	_ "k8s.io/kubernetes/test/e2e/scheduling"
 	_ "k8s.io/kubernetes/test/e2e/storage"
 	_ "k8s.io/kubernetes/test/e2e/workload"

test/e2e/perf/BUILD → test/e2e/scalability/BUILD

@@ -11,6 +11,7 @@ go_library(
     name = "go_default_library",
     srcs = [
         "density.go",
+        "empty.go",
         "load.go",
     ],
     tags = ["automanaged"],

test/e2e/perf/OWNERS → test/e2e/scalability/OWNERS

@@ -1,7 +1,9 @@
 approvers:
 - gmarek
+- shyamjvs
 - wojtek-t
 reviewers:
 - gmarek
-- wojtek-t
+- shyamjvs
 - timothysc
+- wojtek-t

test/e2e/perf/density.go → test/e2e/scalability/density.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package perf
+package scalability
 
 import (
 	"fmt"

test/e2e/empty.go → test/e2e/scalability/empty.go

@@ -14,15 +14,14 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package e2e
+package scalability
 
 import (
 	"time"
 
 	"k8s.io/api/core/v1"
 	"k8s.io/kubernetes/pkg/api"
 	"k8s.io/kubernetes/test/e2e/framework"
-	"k8s.io/kubernetes/test/e2e/perf"
 
 	. "github.com/onsi/ginkgo"
 )
@@ -43,7 +42,7 @@ var _ = framework.KubeDescribe("Empty [Feature:Empty]", func() {
 	})
 
 	It("starts a pod", func() {
-		configs, _, _ := perf.GenerateConfigsForGroup([]*v1.Namespace{f.Namespace}, "empty-pod", 1, 1, framework.GetPauseImageName(f.ClientSet), []string{}, api.Kind("ReplicationController"), 0, 0)
+		configs, _, _ := GenerateConfigsForGroup([]*v1.Namespace{f.Namespace}, "empty-pod", 1, 1, framework.GetPauseImageName(f.ClientSet), []string{}, api.Kind("ReplicationController"), 0, 0)
 		if len(configs) != 1 {
 			framework.Failf("generateConfigs should have generated single config")
 		}

test/e2e/perf/load.go → test/e2e/scalability/load.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package perf
+package scalability
 
 import (
 	"fmt"