add test

Signed-off-by: He Jie Xu <hejie.xu@intel.com>
envoyproxy · Jan 23, 2024 · 01a90af · 01a90af
1 parent d3b0b67
commit 01a90af
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 3 deletions.
diff --git a/api/v1alpha1/envoypatchpolicy_types.go b/api/v1alpha1/envoypatchpolicy_types.go
@@ -115,7 +115,7 @@ type JSONPatchOperation struct {
 	// From is the location of the source document/field where the operation will be performed
 	// Refer to https://datatracker.ietf.org/doc/html/rfc6901 for more details.
 	// +optional
-	From string `json:"from,omitempty"`
+	From *string `json:"from,omitempty"`
 	// Value is the new value of the path location.
 	Value apiextensionsv1.JSON `json:"value"`
 }

diff --git a/internal/ir/xds.go b/internal/ir/xds.go
@@ -1159,7 +1159,7 @@ type JSONPatchOperation struct {
 	// From is the location of the source document/field where the operation will be performed
 	// Refer to https://datatracker.ietf.org/doc/html/rfc6901 for more details.
 	// +optional
-	From string `json:"from,omitempty" yaml:"from,omitempty"`
+	From *string `json:"from,omitempty" yaml:"from,omitempty"`
 	// Value is the new value of the path location.
 	Value apiextensionsv1.JSON `json:"value" yaml:"value"`
 }

diff --git a/internal/xds/translator/testdata/in/xds-ir/jsonpatch.yaml b/internal/xds/translator/testdata/in/xds-ir/jsonpatch.yaml
@@ -70,6 +70,25 @@ envoyPatchPolicies:
         tls_certificate:
           certificate_chain:
             inline_bytes: Y2VydC1kYXRh
+  - type: "type.googleapis.com/envoy.config.endpoint.v3.ClusterLoadAssignment"
+    name: "first-route-dest"
+    operation:
+      op: add
+      path: "/endpoints/1"
+      value:
+        lbEndpoints:
+          - endpoint:
+              address:
+                socketAddress:
+                  address: 1.2.3.4
+                  portValue: 50000
+            loadBalancingWeight: 1
+  - type: "type.googleapis.com/envoy.config.endpoint.v3.ClusterLoadAssignment"
+    name: "first-route-dest"
+    operation:
+      op: "move"
+      from: "/endpoints/0/load_balancing_weight"
+      path: "/endpoints/1/load_balancing_weight"
 http:
 - name: "first-listener"
   address: "0.0.0.0"

diff --git a/internal/xds/translator/testdata/out/xds-ir/jsonpatch.endpoints.yaml b/internal/xds/translator/testdata/out/xds-ir/jsonpatch.endpoints.yaml
@@ -7,6 +7,13 @@
             address: 1.2.3.4
             portValue: 50000
       loadBalancingWeight: 1
-    loadBalancingWeight: 50
     locality:
       region: first-route-dest/backend/0
+  - lbEndpoints:
+    - endpoint:
+        address:
+          socketAddress:
+            address: 1.2.3.4
+            portValue: 50000
+      loadBalancingWeight: 1
+    loadBalancingWeight: 50