fix: r

Signed-off-by: ashing <[email protected]>

Author: ronething

test/e2e/gatewayapi/httproute.go

@@ -211,6 +211,177 @@ spec:
 		})
 	})
 
+	Context("HTTPRoute with Multiple Gateway", func() {
+		var additionalGatewayGroupID string
+		var additionalNamespace string
+
+		var additionalGatewayProxyYaml = `
+apiVersion: gateway.apisix.io/v1alpha1
+kind: GatewayProxy
+metadata:
+  name: additional-proxy-config
+spec:
+  provider:
+    type: ControlPlane
+    controlPlane:
+      endpoints:
+      - %s
+      auth:
+        type: AdminKey
+        adminKey:
+          value: "%s"
+`
+
+		var additionalGateway = `
+apiVersion: gateway.networking.k8s.io/v1
+kind: Gateway
+metadata:
+  name: additional-gateway
+spec:
+  gatewayClassName: %s
+  listeners:
+    - name: http-additional
+      protocol: HTTP
+      port: 80
+      allowedRoutes:
+        namespaces:
+          from: All
+  infrastructure:
+    parametersRef:
+      group: gateway.apisix.io
+      kind: GatewayProxy
+      name: additional-proxy-config
+`
+
+		// HTTPRoute that references both gateways
+		var multiGatewayHTTPRoute = `
+apiVersion: gateway.networking.k8s.io/v1
+kind: HTTPRoute
+metadata:
+  name: multi-gateway-route
+spec:
+  parentRefs:
+  - name: api7ee
+    namespace: %s
+  - name: additional-gateway
+    namespace: %s
+  hostnames:
+  - httpbin.example
+  - httpbin-additional.example
+  rules:
+  - matches: 
+    - path:
+        type: Exact
+        value: /get
+    backendRefs:
+    - name: httpbin-service-e2e-test
+      port: 80
+`
+
+		var gatewayClassName string
+
+		BeforeEach(func() {
+			By("Create GatewayProxy")
+			gatewayProxy := fmt.Sprintf(gatewayProxyYaml, framework.DashboardTLSEndpoint, s.AdminKey())
+			err := s.CreateResourceFromString(gatewayProxy)
+			Expect(err).NotTo(HaveOccurred(), "creating GatewayProxy")
+			time.Sleep(5 * time.Second)
+
+			By("create GatewayClass")
+			gatewayClassName = fmt.Sprintf("api7-%d", time.Now().Unix())
+			err = s.CreateResourceFromStringWithNamespace(fmt.Sprintf(defautlGatewayClass, gatewayClassName, s.GetControllerName()), "")
+			Expect(err).NotTo(HaveOccurred(), "creating GatewayClass")
+			time.Sleep(5 * time.Second)
+
+			By("check GatewayClass condition")
+			gcyaml, err := s.GetResourceYaml("GatewayClass", gatewayClassName)
+			Expect(err).NotTo(HaveOccurred(), "getting GatewayClass yaml")
+			Expect(gcyaml).To(ContainSubstring(`status: "True"`), "checking GatewayClass condition status")
+			Expect(gcyaml).To(ContainSubstring("message: the gatewayclass has been accepted by the api7-ingress-controller"), "checking GatewayClass condition message")
+
+			By("create Gateway")
+			err = s.CreateResourceFromString(fmt.Sprintf(defautlGateway, gatewayClassName))
+			Expect(err).NotTo(HaveOccurred(), "creating Gateway")
+			time.Sleep(5 * time.Second)
+
+			By("check Gateway condition")
+			gwyaml, err := s.GetResourceYaml("Gateway", "api7ee")
+			Expect(err).NotTo(HaveOccurred(), "getting Gateway yaml")
+			Expect(gwyaml).To(ContainSubstring(`status: "True"`), "checking Gateway condition status")
+			Expect(gwyaml).To(ContainSubstring("message: the gateway has been accepted by the api7-ingress-controller"), "checking Gateway condition message")
+
+			By("Create additional gateway group")
+			additionalGatewayGroupID, additionalNamespace, err = s.CreateAdditionalGatewayGroup("multi-gw")
+			Expect(err).NotTo(HaveOccurred(), "creating additional gateway group")
+
+			By("Initialize dataplane client for additional gateway group")
+			err = s.InitializeDataplaneClientForGatewayGroup(additionalGatewayGroupID)
+			Expect(err).NotTo(HaveOccurred(), "initializing dataplane client")
+
+			By("Create additional GatewayProxy")
+			// Get admin key for the additional gateway group
+			resources, exists := s.GetAdditionalGatewayGroup(additionalGatewayGroupID)
+			Expect(exists).To(BeTrue(), "additional gateway group should exist")
+
+			additionalGatewayProxy := fmt.Sprintf(additionalGatewayProxyYaml, framework.DashboardTLSEndpoint, resources.AdminAPIKey)
+			err = s.CreateResourceFromStringWithNamespace(additionalGatewayProxy, additionalNamespace)
+			Expect(err).NotTo(HaveOccurred(), "creating additional GatewayProxy")
+
+			By("Create additional Gateway")
+			err = s.CreateResourceFromStringWithNamespace(
+				fmt.Sprintf(additionalGateway, gatewayClassName),
+				additionalNamespace,
+			)
+			Expect(err).NotTo(HaveOccurred(), "creating additional Gateway")
+			time.Sleep(5 * time.Second)
+		})
+
+		It("HTTPRoute should be accessible through both gateways", func() {
+			By("Create HTTPRoute referencing both gateways")
+			multiGatewayRoute := fmt.Sprintf(multiGatewayHTTPRoute, s.Namespace(), additionalNamespace)
+			ResourceApplied("HTTPRoute", "multi-gateway-route", multiGatewayRoute, 1)
+
+			By("Access through default gateway")
+			s.NewAPISIXClient().
+				GET("/get").
+				WithHost("httpbin.example").
+				Expect().
+				Status(http.StatusOK)
+
+			By("Access through additional gateway")
+			client, err := s.NewAPISIXClientForGatewayGroup(additionalGatewayGroupID)
+			Expect(err).NotTo(HaveOccurred(), "creating client for additional gateway")
+
+			client.
+				GET("/get").
+				WithHost("httpbin-additional.example").
+				Expect().
+				Status(http.StatusOK)
+
+			By("Delete Additional Gateway")
+			err = s.DeleteResourceFromStringWithNamespace(fmt.Sprintf(additionalGateway, gatewayClassName), additionalNamespace)
+			Expect(err).NotTo(HaveOccurred(), "deleting additional Gateway")
+			time.Sleep(5 * time.Second)
+
+			By("HTTPRoute should still be accessible through default gateway")
+			s.NewAPISIXClient().
+				GET("/get").
+				WithHost("httpbin.example").
+				Expect().
+				Status(http.StatusOK)
+
+			By("HTTPRoute should not be accessible through additional gateway")
+			client, err = s.NewAPISIXClientForGatewayGroup(additionalGatewayGroupID)
+			Expect(err).NotTo(HaveOccurred(), "creating client for additional gateway")
+
+			client.
+				GET("/get").
+				WithHost("httpbin-additional.example").
+				Expect().
+				Status(http.StatusNotFound)
+		})
+	})
+
 	Context("HTTPRoute Base", func() {
 		var exactRouteByGet = `
 apiVersion: gateway.networking.k8s.io/v1

test/e2e/scaffold/dp.go

@@ -15,7 +15,6 @@
 package scaffold
 
 import (
-	"github.com/gruntwork-io/terratest/modules/k8s"
 	. "github.com/onsi/gomega"
 
 	"github.com/api7/api7-ingress-controller/test/e2e/framework"
@@ -43,36 +42,13 @@ func (s *Scaffold) deployDataplane() {
 }
 
 func (s *Scaffold) newAPISIXTunnels() error {
-	var (
-		httpNodePort  int
-		httpsNodePort int
-		httpPort      int
-		httpsPort     int
-		serviceName   = "api7ee3-apisix-gateway-mtls"
-	)
-
-	svc := s.dataplaneService
-	for _, port := range svc.Spec.Ports {
-		if port.Name == "http" {
-			httpNodePort = int(port.NodePort)
-			httpPort = int(port.Port)
-		} else if port.Name == "https" {
-			httpsNodePort = int(port.NodePort)
-			httpsPort = int(port.Port)
-		}
-	}
-	s.apisixHttpTunnel = k8s.NewTunnel(s.kubectlOptions, k8s.ResourceTypeService, serviceName,
-		httpNodePort, httpPort)
-	s.apisixHttpsTunnel = k8s.NewTunnel(s.kubectlOptions, k8s.ResourceTypeService, serviceName,
-		httpsNodePort, httpsPort)
-
-	if err := s.apisixHttpTunnel.ForwardPortE(s.t); err != nil {
+	serviceName := "api7ee3-apisix-gateway-mtls"
+	httpTunnel, httpsTunnel, err := s.createDataplaneTunnels(s.dataplaneService, s.kubectlOptions, serviceName)
+	if err != nil {
 		return err
 	}
-	s.addFinalizers(s.apisixHttpTunnel.Close)
-	if err := s.apisixHttpsTunnel.ForwardPortE(s.t); err != nil {
-		return err
-	}
-	s.addFinalizers(s.apisixHttpsTunnel.Close)
+
+	s.apisixHttpTunnel = httpTunnel
+	s.apisixHttpsTunnel = httpsTunnel
 	return nil
 }