From 60a51ce020c1ef0c846c1d08d8880fd5a48d01b1 Mon Sep 17 00:00:00 2001 From: John Belamaric Date: Fri, 26 Jul 2019 11:00:24 -0700 Subject: [PATCH] Promote NodePort service creation e2e test to conformance --- test/conformance/testdata/conformance.txt | 1 + test/e2e/network/service.go | 12 ++++++++---- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/test/conformance/testdata/conformance.txt b/test/conformance/testdata/conformance.txt index 0ee6780faf1..f28f4a2043f 100644 --- a/test/conformance/testdata/conformance.txt +++ b/test/conformance/testdata/conformance.txt @@ -208,6 +208,7 @@ test/e2e/network/proxy.go: "should proxy through a service and a pod" test/e2e/network/service.go: "should provide secure master service" test/e2e/network/service.go: "should serve a basic endpoint from pods" test/e2e/network/service.go: "should serve multiport endpoints from pods" +test/e2e/network/service.go: "should be able to create a functioning NodePort service" test/e2e/network/service_latency.go: "should not be very high" test/e2e/node/events.go: "should be sent by kubelets and the scheduler about pods scheduling and running" test/e2e/node/pods.go: "should be submitted and removed" diff --git a/test/e2e/network/service.go b/test/e2e/network/service.go index 06c17ce863e..71606977cbc 100644 --- a/test/e2e/network/service.go +++ b/test/e2e/network/service.go @@ -507,10 +507,14 @@ var _ = SIGDescribe("Services", func() { framework.ExpectNoError(e2eservice.VerifyServeHostnameServiceUp(cs, ns, host, podNames2, svc2IP, servicePort)) }) - // TODO: Run this test against the userspace proxy and nodes - // configured with a default deny firewall to validate that the - // proxy whitelists NodePort traffic. - ginkgo.It("should be able to create a functioning NodePort service", func() { + /* + Release : v1.16 + Testname: Service, NodePort Service + Description: Create a TCP NodePort service, and test reachability from a client Pod. + The client Pod MUST be able to access the NodePort service by service name and cluster + IP on the service port, and on nodes' internal and external IPs on the NodePort. + */ + framework.ConformanceIt("should be able to create a functioning NodePort service", func() { serviceName := "nodeport-test" ns := f.Namespace.Name