Moves networking e2e tests to test/e2e/network

This also adds sig-network-{approvers,reviewers} alias

OWNERS_ALIASES

@@ -58,14 +58,26 @@ aliases:
     - vishh
     - yifan-gu
     - yujuhong
-  sig-network-reviewers:
+  sig-network-approvers:
-    - thockin
-    - freehan
     - bowei
-    - mrhohn
-    - dcbw
-    - danwinship
     - caseydavenport
+    - danwinship
+    - dcbw
+    - dnardo
+    - freehan
+    - mrhohn
+    - nicksardo
+    - thockin
+  sig-network-reviewers:
+    - bowei
+    - caseydavenport
+    - danwinship
+    - dcbw
+    - dnardo
+    - freehan
+    - mrhohn
+    - nicksardo
+    - thockin
   sig-apps-reviewers:
     - enisoc
     - erictune

hack/.golint_failures

@@ -813,6 +813,7 @@ test/e2e/kubectl
 test/e2e/lifecycle
 test/e2e/lifecycle/bootstrap
 test/e2e/metrics
+test/e2e/network
 test/e2e/scalability
 test/e2e/scheduling
 test/e2e/storage

test/e2e/BUILD

@@ -28,6 +28,7 @@ go_test(
         "//test/e2e/lifecycle:go_default_library",
         "//test/e2e/lifecycle/bootstrap:go_default_library",
         "//test/e2e/metrics:go_default_library",
+        "//test/e2e/network:go_default_library",
         "//test/e2e/scalability:go_default_library",
         "//test/e2e/scheduling:go_default_library",
         "//test/e2e/storage:go_default_library",
@@ -52,26 +53,18 @@ go_library(
         "audit.go",
         "certificates.go",
         "dashboard.go",
-        "dns.go",
-        "dns_common.go",
-        "dns_configmap.go",
         "e2e.go",
         "events.go",
         "example_cluster_dns.go",
         "examples.go",
-        "firewall.go",
         "generated_clientset.go",
         "gke_local_ssd.go",
         "gke_node_pools.go",
         "ingress.go",
-        "kube_proxy.go",
         "kubelet.go",
         "kubelet_perf.go",
         "limit_range.go",
         "network_partition.go",
-        "network_policy.go",
-        "networking.go",
-        "networking_perf.go",
         "no-snat.go",
         "nodeoutofdisk.go",
         "pod_gc.go",
@@ -80,30 +73,22 @@ go_library(
         "pre_stop.go",
         "resource_quota.go",
         "security_context.go",
-        "service.go",
         "service_accounts.go",
         "service_latency.go",
         "serviceloadbalancers.go",
         "ssh.go",
         "ubernetes_lite.go",
-        "util_iperf.go",
     ],
     tags = ["automanaged"],
     deps = [
         "//pkg/api:go_default_library",
         "//pkg/api/testapi:go_default_library",
         "//pkg/api/v1/pod:go_default_library",
-        "//pkg/api/v1/service:go_default_library",
-        "//pkg/apis/networking:go_default_library",
-        "//pkg/client/clientset_generated/internalclientset:go_default_library",
-        "//pkg/cloudprovider:go_default_library",
         "//pkg/cloudprovider/providers/azure:go_default_library",
         "//pkg/cloudprovider/providers/gce:go_default_library",
-        "//pkg/controller/endpoint:go_default_library",
         "//pkg/controller/node:go_default_library",
         "//pkg/kubelet/apis:go_default_library",
         "//pkg/kubelet/apis/stats/v1alpha1:go_default_library",
-        "//pkg/master/ports:go_default_library",
         "//pkg/quota/evaluator/core:go_default_library",
         "//pkg/util/logs:go_default_library",
         "//pkg/util/version:go_default_library",
@@ -115,7 +100,6 @@ go_library(
         "//test/e2e/manifest:go_default_library",
         "//test/e2e/metrics:go_default_library",
         "//test/e2e_federation:go_default_library",
-        "//test/images/net/nat:go_default_library",
         "//test/utils:go_default_library",
         "//vendor/github.com/golang/glog:go_default_library",
         "//vendor/github.com/google/cadvisor/info/v1:go_default_library",
@@ -152,6 +136,8 @@ go_library(
     ],
 )
 
+# This is a handwritten rule. Do not delete, it will not be regenerated by
+# update-bazel.sh.
 genrule(
     name = "gen_e2e.test",
     testonly = 1,
@@ -184,6 +170,7 @@ filegroup(
         "//test/e2e/lifecycle:all-srcs",
         "//test/e2e/manifest:all-srcs",
         "//test/e2e/metrics:all-srcs",
+        "//test/e2e/network:all-srcs",
         "//test/e2e/perftype:all-srcs",
         "//test/e2e/scalability:all-srcs",
         "//test/e2e/scheduling:all-srcs",

test/e2e/e2e_test.go

@@ -28,6 +28,7 @@ import (
 	_ "k8s.io/kubernetes/test/e2e/kubectl"
 	_ "k8s.io/kubernetes/test/e2e/lifecycle"
 	_ "k8s.io/kubernetes/test/e2e/lifecycle/bootstrap"
+	_ "k8s.io/kubernetes/test/e2e/network"
 	_ "k8s.io/kubernetes/test/e2e/scalability"
 	_ "k8s.io/kubernetes/test/e2e/scheduling"
 	_ "k8s.io/kubernetes/test/e2e/storage"

test/e2e/network/BUILD

@@ -0,0 +1,63 @@
+package(default_visibility = ["//visibility:public"])
+
+licenses(["notice"])
+
+load(
+    "@io_bazel_rules_go//go:def.bzl",
+    "go_library",
+)
+
+go_library(
+    name = "go_default_library",
+    srcs = [
+        "dns.go",
+        "dns_common.go",
+        "dns_configmap.go",
+        "doc.go",
+        "firewall.go",
+        "kube_proxy.go",
+        "network_policy.go",
+        "networking.go",
+        "networking_perf.go",
+        "service.go",
+        "util_iperf.go",
+    ],
+    tags = ["automanaged"],
+    deps = [
+        "//pkg/api/testapi:go_default_library",
+        "//pkg/api/v1/service:go_default_library",
+        "//pkg/apis/networking:go_default_library",
+        "//pkg/client/clientset_generated/internalclientset:go_default_library",
+        "//pkg/cloudprovider:go_default_library",
+        "//pkg/cloudprovider/providers/gce:go_default_library",
+        "//pkg/controller/endpoint:go_default_library",
+        "//pkg/kubelet/apis:go_default_library",
+        "//pkg/master/ports:go_default_library",
+        "//test/e2e/framework:go_default_library",
+        "//test/images/net/nat:go_default_library",
+        "//vendor/github.com/onsi/ginkgo:go_default_library",
+        "//vendor/github.com/onsi/gomega:go_default_library",
+        "//vendor/k8s.io/api/core/v1:go_default_library",
+        "//vendor/k8s.io/apimachinery/pkg/apis/meta/v1:go_default_library",
+        "//vendor/k8s.io/apimachinery/pkg/fields:go_default_library",
+        "//vendor/k8s.io/apimachinery/pkg/labels:go_default_library",
+        "//vendor/k8s.io/apimachinery/pkg/util/intstr:go_default_library",
+        "//vendor/k8s.io/apimachinery/pkg/util/sets:go_default_library",
+        "//vendor/k8s.io/apimachinery/pkg/util/uuid:go_default_library",
+        "//vendor/k8s.io/apimachinery/pkg/util/wait:go_default_library",
+        "//vendor/k8s.io/client-go/kubernetes:go_default_library",
+    ],
+)
+
+filegroup(
+    name = "package-srcs",
+    srcs = glob(["**"]),
+    tags = ["automanaged"],
+    visibility = ["//visibility:private"],
+)
+
+filegroup(
+    name = "all-srcs",
+    srcs = [":package-srcs"],
+    tags = ["automanaged"],
+)

test/e2e/network/OWNERS

@@ -0,0 +1,4 @@
+reviewers:
+  - sig-network-reviewers
+approvers:
+  - sig-network-approvers

test/e2e/network/dns.go

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

test/e2e/network/dns_common.go

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

test/e2e/network/dns_configmap.go

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

test/e2e/network/doc.go

@@ -0,0 +1,18 @@
+/*
+Copyright 2014 The Kubernetes Authors.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+// Package network are the end-to-end tests for Kubernetes networking.
+package network

test/e2e/network/firewall.go

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

test/e2e/network/kube_proxy.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package e2e
+package network
 
 import (
 	"encoding/json"

test/e2e/network/network_policy.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package e2e
+package network
 
 import (
 	"k8s.io/api/core/v1"

test/e2e/network/networking.go

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

test/e2e/network/networking_perf.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package e2e
+package network
 
 // Tests network performance using iperf or other containers.
 import (

test/e2e/network/service.go

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

test/e2e/network/util_iperf.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package e2e
+package network
 
 // Tests network performance using iperf or other containers.
 import (