functional tests for upstreamSettingsPolicy

Author: salonichf5

Diff for: tests/framework/crossplane.go

@@ -30,6 +30,8 @@ type ExpectedNginxField struct {
 	Location string
 	// Servers are the server names that the directive should exist in.
 	Servers []string
+	// Upstream are the upstream names that the directive should exist in.
+	Upstreams []string
 	// ValueSubstringAllowed allows the expected value to be a substring of the real value.
 	// This makes it easier for cases when real values are complex file names or contain things we
 	// don't care about, and we just want to check if a substring exists.
@@ -64,6 +66,8 @@ func ValidateNginxFieldExists(conf *Payload, expFieldCfg ExpectedNginxField) err
 					}
 				}
 			}
+
+			return validateUpstreamDirectives(expFieldCfg, directive)
 		}
 	}
 
@@ -75,6 +79,19 @@ func ValidateNginxFieldExists(conf *Payload, expFieldCfg ExpectedNginxField) err
 	return fmt.Errorf("field not found; expected: %+v\nNGINX conf: %s", expFieldCfg, string(b))
 }
 
+func validateUpstreamDirectives(expFieldCfg ExpectedNginxField, directive *Directive) error {
+	for _, upstreamName := range expFieldCfg.Upstreams {
+		if directive.Directive == "upstream" && directive.Args[0] == upstreamName {
+			for _, upstreamDirective := range directive.Block {
+				if expFieldCfg.fieldFound(upstreamDirective) {
+					return nil
+				}
+			}
+		}
+	}
+	return nil
+}
+
 func getServerName(serverBlock Directives) string {
 	for _, directive := range serverBlock {
 		if directive.Directive == "server_name" {

Diff for: tests/suite/manifests/upstream-settings-policy/cafe.yaml

@@ -63,4 +63,4 @@ spec:
     name: http
   selector:
     app: tea
----
+---

Diff for: tests/suite/manifests/upstream-settings-policy/invalid-usps.yaml

@@ -0,0 +1,84 @@
+apiVersion: gateway.networking.k8s.io/v1
+kind: Gateway
+metadata:
+  name: gateway-httproute-allowed
+spec:
+  gatewayClassName: nginx
+  listeners:
+  - name: http
+    port: 80
+    protocol: HTTP
+    hostname: "soda.example.com"
+    allowedRoutes:
+      namespaces:
+        from: All
+      kinds:
+        kind: GRPCRoute
+---
+apiVersion: apps/v1
+kind: Deployment
+metadata:
+  name: soda
+spec:
+  replicas: 1
+  selector:
+    matchLabels:
+      app: soda
+  template:
+    metadata:
+      labels:
+        app: soda
+    spec:
+      containers:
+      - name: soda
+        image: nginxdemos/nginx-hello:plain-text
+        ports:
+        - containerPort: 8080
+---
+apiVersion: v1
+kind: Service
+metadata:
+  name: soda
+spec:
+  ports:
+  - port: 80
+    targetPort: 8080
+    protocol: TCP
+    name: http
+  selector:
+    app: soda
+---
+apiVersion: gateway.networking.k8s.io/v1
+kind: HTTPRoute
+metadata:
+  name: soda
+spec:
+  parentRefs:
+  - name: gateway-httproute-allowed
+    sectionName: http
+  hostnames:
+  - "soda.example.com"
+  rules:
+  - matches:
+    - path:
+        type: Exact
+        value: /soda
+    backendRefs:
+    - name: soda
+      port: 80
+---
+apiVersion: gateway.nginx.org/v1alpha1
+kind: UpstreamSettingsPolicy
+metadata:
+  name: soda-svc-usp
+spec:
+  zoneSize: 512k
+  targetRefs:
+  - group: core
+    kind: Service
+    name: soda
+  keepAlive:
+    connections: 10
+    requests: 3
+    time: 10s
+    timeout: 50s

Diff for: tests/suite/manifests/upstream-settings-policy/routes.yaml

@@ -32,4 +32,4 @@ spec:
             method: SayHello
       backendRefs:
         - name: grpc-backend
-          port: 8080
+          port: 8080

Diff for: tests/suite/manifests/upstream-settings-policy/valid-merge-usps.yaml

@@ -0,0 +1,25 @@
+apiVersion: gateway.nginx.org/v1alpha1
+kind: UpstreamSettingsPolicy
+metadata:
+  name: coffee-svc-usp-1
+spec:
+  zoneSize: 64k
+  targetRefs:
+    - group: core
+      kind: Service
+      name: coffee
+---
+apiVersion: gateway.nginx.org/v1alpha1
+kind: UpstreamSettingsPolicy
+metadata:
+  name: coffee-svc-usp-2
+spec:
+  targetRefs:
+    - group: core
+      kind: Service
+      name: coffee
+  keepAlive:
+    connections: 100
+    requests: 55
+    time: 1m
+    timeout: 5h

Diff for: tests/suite/manifests/upstream-settings-policy/valid-usps-first-wins.yaml

@@ -0,0 +1,21 @@
+apiVersion: gateway.nginx.org/v1alpha1
+kind: UpstreamSettingsPolicy
+metadata:
+  name: coffee-svc-usp-1
+spec:
+  zoneSize: 64k
+  targetRefs:
+    - group: core
+      kind: Service
+      name: coffee
+---
+apiVersion: gateway.nginx.org/v1alpha1
+kind: UpstreamSettingsPolicy
+metadata:
+  name: coffee-svc-usp-2
+spec:
+  zoneSize: 128k
+  targetRefs:
+    - group: core
+      kind: Service
+      name: coffee

Diff for: tests/suite/manifests/upstream-settings-policy/valid-usps.yaml

@@ -19,7 +19,7 @@ kind: UpstreamSettingsPolicy
 metadata:
   name: tea-multiple-svc-usp
 spec:
-  zoneSize: 512k
+  zoneSize: 128k
   targetRefs:
     - group: core
       kind: Service
@@ -28,10 +28,10 @@ spec:
       kind: Service
       name: coffee
   keepAlive:
-    connections: 10
-    requests: 3
-    time: 10s
-    timeout: 50s
+    connections: 12
+    requests: 31
+    time: 20s
+    timeout: 40s
 ---
 apiVersion: gateway.nginx.org/v1alpha1
 kind: UpstreamSettingsPolicy