123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280 |
- /*
- Copyright 2016 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
- import (
- "context"
- "fmt"
- "strings"
- "time"
- v1 "k8s.io/api/core/v1"
- metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
- "k8s.io/apimachinery/pkg/util/sets"
- "k8s.io/apimachinery/pkg/util/wait"
- clientset "k8s.io/client-go/kubernetes"
- cloudprovider "k8s.io/cloud-provider"
- "k8s.io/kubernetes/pkg/master/ports"
- "k8s.io/kubernetes/test/e2e/framework"
- e2enetwork "k8s.io/kubernetes/test/e2e/framework/network"
- e2enode "k8s.io/kubernetes/test/e2e/framework/node"
- "k8s.io/kubernetes/test/e2e/framework/providers/gce"
- e2eservice "k8s.io/kubernetes/test/e2e/framework/service"
- e2eskipper "k8s.io/kubernetes/test/e2e/framework/skipper"
- gcecloud "k8s.io/legacy-cloud-providers/gce"
- "github.com/onsi/ginkgo"
- )
- const (
- firewallTestTCPTimeout = time.Duration(1 * time.Second)
- // Set ports outside of 30000-32767, 80 and 8080 to avoid being whitelisted by the e2e cluster
- firewallTestHTTPPort = int32(29999)
- firewallTestUDPPort = int32(29998)
- )
- var _ = SIGDescribe("Firewall rule", func() {
- var firewallTestName = "firewall-test"
- f := framework.NewDefaultFramework(firewallTestName)
- var cs clientset.Interface
- var cloudConfig framework.CloudConfig
- var gceCloud *gcecloud.Cloud
- ginkgo.BeforeEach(func() {
- e2eskipper.SkipUnlessProviderIs("gce")
- var err error
- cs = f.ClientSet
- cloudConfig = framework.TestContext.CloudConfig
- gceCloud, err = gce.GetGCECloud()
- framework.ExpectNoError(err)
- })
- // This test takes around 6 minutes to run
- ginkgo.It("[Slow] [Serial] should create valid firewall rules for LoadBalancer type service", func() {
- ns := f.Namespace.Name
- // This source ranges is just used to examine we have exact same things on LB firewall rules
- firewallTestSourceRanges := []string{"0.0.0.0/1", "128.0.0.0/1"}
- serviceName := "firewall-test-loadbalancer"
- ginkgo.By("Getting cluster ID")
- clusterID, err := gce.GetClusterID(cs)
- framework.ExpectNoError(err)
- framework.Logf("Got cluster ID: %v", clusterID)
- jig := e2eservice.NewTestJig(cs, ns, serviceName)
- nodeList, err := e2enode.GetBoundedReadySchedulableNodes(cs, e2eservice.MaxNodesForEndpointsTests)
- framework.ExpectNoError(err)
- nodesNames := []string{}
- for _, node := range nodeList.Items {
- nodesNames = append(nodesNames, node.Name)
- }
- nodesSet := sets.NewString(nodesNames...)
- ginkgo.By("Creating a LoadBalancer type service with ExternalTrafficPolicy=Global")
- svc, err := jig.CreateLoadBalancerService(e2eservice.GetServiceLoadBalancerCreationTimeout(cs), func(svc *v1.Service) {
- svc.Spec.Ports = []v1.ServicePort{{Protocol: v1.ProtocolTCP, Port: firewallTestHTTPPort}}
- svc.Spec.LoadBalancerSourceRanges = firewallTestSourceRanges
- })
- framework.ExpectNoError(err)
- defer func() {
- _, err = jig.UpdateService(func(svc *v1.Service) {
- svc.Spec.Type = v1.ServiceTypeNodePort
- svc.Spec.LoadBalancerSourceRanges = nil
- })
- framework.ExpectNoError(err)
- err = cs.CoreV1().Services(svc.Namespace).Delete(context.TODO(), svc.Name, nil)
- framework.ExpectNoError(err)
- ginkgo.By("Waiting for the local traffic health check firewall rule to be deleted")
- localHCFwName := gce.MakeHealthCheckFirewallNameForLBService(clusterID, cloudprovider.DefaultLoadBalancerName(svc), false)
- _, err := gce.WaitForFirewallRule(gceCloud, localHCFwName, false, e2eservice.LoadBalancerCleanupTimeout)
- framework.ExpectNoError(err)
- }()
- svcExternalIP := svc.Status.LoadBalancer.Ingress[0].IP
- ginkgo.By("Checking if service's firewall rule is correct")
- lbFw := gce.ConstructFirewallForLBService(svc, cloudConfig.NodeTag)
- fw, err := gceCloud.GetFirewall(lbFw.Name)
- framework.ExpectNoError(err)
- err = gce.VerifyFirewallRule(fw, lbFw, cloudConfig.Network, false)
- framework.ExpectNoError(err)
- ginkgo.By("Checking if service's nodes health check firewall rule is correct")
- nodesHCFw := gce.ConstructHealthCheckFirewallForLBService(clusterID, svc, cloudConfig.NodeTag, true)
- fw, err = gceCloud.GetFirewall(nodesHCFw.Name)
- framework.ExpectNoError(err)
- err = gce.VerifyFirewallRule(fw, nodesHCFw, cloudConfig.Network, false)
- framework.ExpectNoError(err)
- // OnlyLocal service is needed to examine which exact nodes the requests are being forwarded to by the Load Balancer on GCE
- ginkgo.By("Updating LoadBalancer service to ExternalTrafficPolicy=Local")
- svc, err = jig.UpdateService(func(svc *v1.Service) {
- svc.Spec.ExternalTrafficPolicy = v1.ServiceExternalTrafficPolicyTypeLocal
- })
- framework.ExpectNoError(err)
- ginkgo.By("Waiting for the nodes health check firewall rule to be deleted")
- _, err = gce.WaitForFirewallRule(gceCloud, nodesHCFw.Name, false, e2eservice.LoadBalancerCleanupTimeout)
- framework.ExpectNoError(err)
- ginkgo.By("Waiting for the correct local traffic health check firewall rule to be created")
- localHCFw := gce.ConstructHealthCheckFirewallForLBService(clusterID, svc, cloudConfig.NodeTag, false)
- fw, err = gce.WaitForFirewallRule(gceCloud, localHCFw.Name, true, e2eservice.GetServiceLoadBalancerCreationTimeout(cs))
- framework.ExpectNoError(err)
- err = gce.VerifyFirewallRule(fw, localHCFw, cloudConfig.Network, false)
- framework.ExpectNoError(err)
- ginkgo.By(fmt.Sprintf("Creating netexec pods on at most %v nodes", e2eservice.MaxNodesForEndpointsTests))
- for i, nodeName := range nodesNames {
- podName := fmt.Sprintf("netexec%v", i)
- framework.Logf("Creating netexec pod %q on node %v in namespace %q", podName, nodeName, ns)
- pod := f.NewAgnhostPod(podName,
- "netexec",
- fmt.Sprintf("--http-port=%d", firewallTestHTTPPort),
- fmt.Sprintf("--udp-port=%d", firewallTestUDPPort))
- pod.ObjectMeta.Labels = jig.Labels
- pod.Spec.NodeName = nodeName
- pod.Spec.HostNetwork = true
- _, err := cs.CoreV1().Pods(ns).Create(context.TODO(), pod, metav1.CreateOptions{})
- framework.ExpectNoError(err)
- framework.ExpectNoError(f.WaitForPodReady(podName))
- framework.Logf("Netexec pod %q in namespace %q running", podName, ns)
- defer func() {
- framework.Logf("Cleaning up the netexec pod: %v", podName)
- err = cs.CoreV1().Pods(ns).Delete(context.TODO(), podName, nil)
- framework.ExpectNoError(err)
- }()
- }
- // Send requests from outside of the cluster because internal traffic is whitelisted
- ginkgo.By("Accessing the external service ip from outside, all non-master nodes should be reached")
- err = testHitNodesFromOutside(svcExternalIP, firewallTestHTTPPort, e2eservice.LoadBalancerPropagationTimeoutDefault, nodesSet)
- framework.ExpectNoError(err)
- // Check if there are overlapping tags on the firewall that extend beyond just the vms in our cluster
- // by removing the tag on one vm and make sure it doesn't get any traffic. This is an imperfect
- // simulation, we really want to check that traffic doesn't reach a vm outside the GKE cluster, but
- // that's much harder to do in the current e2e framework.
- ginkgo.By(fmt.Sprintf("Removing tags from one of the nodes: %v", nodesNames[0]))
- nodesSet.Delete(nodesNames[0])
- // Instance could run in a different zone in multi-zone test. Figure out which zone
- // it is in before proceeding.
- zone := cloudConfig.Zone
- if zoneInLabel, ok := nodeList.Items[0].Labels[v1.LabelZoneFailureDomain]; ok {
- zone = zoneInLabel
- }
- removedTags := gce.SetInstanceTags(cloudConfig, nodesNames[0], zone, []string{})
- defer func() {
- ginkgo.By("Adding tags back to the node and wait till the traffic is recovered")
- nodesSet.Insert(nodesNames[0])
- gce.SetInstanceTags(cloudConfig, nodesNames[0], zone, removedTags)
- // Make sure traffic is recovered before exit
- err = testHitNodesFromOutside(svcExternalIP, firewallTestHTTPPort, e2eservice.LoadBalancerPropagationTimeoutDefault, nodesSet)
- framework.ExpectNoError(err)
- }()
- ginkgo.By("Accessing serivce through the external ip and examine got no response from the node without tags")
- err = testHitNodesFromOutsideWithCount(svcExternalIP, firewallTestHTTPPort, e2eservice.LoadBalancerPropagationTimeoutDefault, nodesSet, 15)
- framework.ExpectNoError(err)
- })
- ginkgo.It("should have correct firewall rules for e2e cluster", func() {
- nodes, err := e2enode.GetReadySchedulableNodes(cs)
- framework.ExpectNoError(err)
- ginkgo.By("Checking if e2e firewall rules are correct")
- for _, expFw := range gce.GetE2eFirewalls(cloudConfig.MasterName, cloudConfig.MasterTag, cloudConfig.NodeTag, cloudConfig.Network, cloudConfig.ClusterIPRange) {
- fw, err := gceCloud.GetFirewall(expFw.Name)
- framework.ExpectNoError(err)
- err = gce.VerifyFirewallRule(fw, expFw, cloudConfig.Network, false)
- framework.ExpectNoError(err)
- }
- ginkgo.By("Checking well known ports on master and nodes are not exposed externally")
- nodeAddr := e2enode.FirstAddress(nodes, v1.NodeExternalIP)
- if nodeAddr == "" {
- framework.Failf("did not find any node addresses")
- }
- masterAddresses := framework.GetAllMasterAddresses(cs)
- for _, masterAddress := range masterAddresses {
- assertNotReachableHTTPTimeout(masterAddress, ports.InsecureKubeControllerManagerPort, firewallTestTCPTimeout)
- assertNotReachableHTTPTimeout(masterAddress, ports.InsecureSchedulerPort, firewallTestTCPTimeout)
- }
- assertNotReachableHTTPTimeout(nodeAddr, ports.KubeletPort, firewallTestTCPTimeout)
- assertNotReachableHTTPTimeout(nodeAddr, ports.KubeletReadOnlyPort, firewallTestTCPTimeout)
- assertNotReachableHTTPTimeout(nodeAddr, ports.ProxyStatusPort, firewallTestTCPTimeout)
- })
- })
- func assertNotReachableHTTPTimeout(ip string, port int, timeout time.Duration) {
- result := e2enetwork.PokeHTTP(ip, port, "/", &e2enetwork.HTTPPokeParams{Timeout: timeout})
- if result.Status == e2enetwork.HTTPError {
- framework.Failf("Unexpected error checking for reachability of %s:%d: %v", ip, port, result.Error)
- }
- if result.Code != 0 {
- framework.Failf("Was unexpectedly able to reach %s:%d", ip, port)
- }
- }
- // testHitNodesFromOutside checkes HTTP connectivity from outside.
- func testHitNodesFromOutside(externalIP string, httpPort int32, timeout time.Duration, expectedHosts sets.String) error {
- return testHitNodesFromOutsideWithCount(externalIP, httpPort, timeout, expectedHosts, 1)
- }
- // testHitNodesFromOutsideWithCount checkes HTTP connectivity from outside with count.
- func testHitNodesFromOutsideWithCount(externalIP string, httpPort int32, timeout time.Duration, expectedHosts sets.String,
- countToSucceed int) error {
- framework.Logf("Waiting up to %v for satisfying expectedHosts for %v times", timeout, countToSucceed)
- hittedHosts := sets.NewString()
- count := 0
- condition := func() (bool, error) {
- result := e2enetwork.PokeHTTP(externalIP, int(httpPort), "/hostname", &e2enetwork.HTTPPokeParams{Timeout: 1 * time.Second})
- if result.Status != e2enetwork.HTTPSuccess {
- return false, nil
- }
- hittedHost := strings.TrimSpace(string(result.Body))
- if !expectedHosts.Has(hittedHost) {
- framework.Logf("Error hitting unexpected host: %v, reset counter: %v", hittedHost, count)
- count = 0
- return false, nil
- }
- if !hittedHosts.Has(hittedHost) {
- hittedHosts.Insert(hittedHost)
- framework.Logf("Missing %+v, got %+v", expectedHosts.Difference(hittedHosts), hittedHosts)
- }
- if hittedHosts.Equal(expectedHosts) {
- count++
- if count >= countToSucceed {
- return true, nil
- }
- }
- return false, nil
- }
- if err := wait.Poll(time.Second, timeout, condition); err != nil {
- return fmt.Errorf("error waiting for expectedHosts: %v, hittedHosts: %v, count: %v, expected count: %v",
- expectedHosts, hittedHosts, count, countToSucceed)
- }
- return nil
- }
|