update: grpcstatus type

kkk777-7 · kkk777-7 · commit f661cda5a4e4 · 2026-04-21T15:50:07.000+09:00
Signed-off-by: kkk777-7 &lt;kota.kimura0725@gmail.com&gt;
diff --git a/api/v1alpha1/fault_injection.go b/api/v1alpha1/fault_injection.go
@@ -53,7 +53,9 @@ type FaultInjectionAbort struct {
 	// GrpcStatus specifies the GRPC status code to be returned
 	//
 	// +optional
-	GrpcStatus *int32 `json:"grpcStatus,omitempty"`
+	// +kubebuilder:validation:Minimum=0
+	// +kubebuilder:validation:Maximum=16
+	GrpcStatus *uint32 `json:"grpcStatus,omitempty"`
 
 	// Percentage specifies the percentage of requests to be aborted. Default 100%, if set 0, no requests will be aborted. Accuracy to 0.0001%.
 	// +optional
diff --git a/api/v1alpha1/zz_generated.deepcopy.go b/api/v1alpha1/zz_generated.deepcopy.go
diff --git a/charts/gateway-crds-helm/templates/generated/gateway.envoyproxy.io_backendtrafficpolicies.yaml b/charts/gateway-crds-helm/templates/generated/gateway.envoyproxy.io_backendtrafficpolicies.yaml
@@ -349,6 +349,8 @@ spec:
                         description: GrpcStatus specifies the GRPC status code to
                           be returned
                         format: int32
+                        maximum: 16
+                        minimum: 0
                         type: integer
                       httpStatus:
                         description: StatusCode specifies the HTTP status code to
diff --git a/charts/gateway-helm/charts/crds/crds/generated/gateway.envoyproxy.io_backendtrafficpolicies.yaml b/charts/gateway-helm/charts/crds/crds/generated/gateway.envoyproxy.io_backendtrafficpolicies.yaml
@@ -348,6 +348,8 @@ spec:
                         description: GrpcStatus specifies the GRPC status code to
                           be returned
                         format: int32
+                        maximum: 16
+                        minimum: 0
                         type: integer
                       httpStatus:
                         description: StatusCode specifies the HTTP status code to
diff --git a/internal/ir/xds.go b/internal/ir/xds.go
@@ -1655,7 +1655,7 @@ type FaultInjectionAbort struct {
 	// HTTPStatus defines the HTTP status code to be returned.
 	HTTPStatus *int32 `json:"httpStatus,omitempty" yaml:"httpStatus,omitempty"`
 	// GrpcStatus defines the gRPC status code to be returned.
-	GrpcStatus *int32 `json:"grpcStatus,omitempty" yaml:"grpcStatus,omitempty"`
+	GrpcStatus *uint32 `json:"grpcStatus,omitempty" yaml:"grpcStatus,omitempty"`
 	// Percentage defines the percentage of requests to be aborted.
 	Percentage *float32 `json:"percentage,omitempty" yaml:"percentage,omitempty"`
 }
diff --git a/internal/ir/zz_generated.deepcopy.go b/internal/ir/zz_generated.deepcopy.go
diff --git a/internal/xds/translator/fault.go b/internal/xds/translator/fault.go
@@ -152,7 +152,7 @@ func (*fault) patchRoute(route *routev3.Route, irRoute *ir.HTTPRoute, _ *ir.HTTP
 		}
 		if abort.GrpcStatus != nil {
 			routeCfgProto.Abort.ErrorType = &xdshttpfaultv3.FaultAbort_GrpcStatus{
-				GrpcStatus: uint32(*abort.GrpcStatus),
+				GrpcStatus: *abort.GrpcStatus,
 			}
 		}
 	}
diff --git a/test/cel-validation/backendtrafficpolicy_test.go b/test/cel-validation/backendtrafficpolicy_test.go
@@ -996,7 +996,7 @@ func TestBackendTrafficPolicyTarget(t *testing.T) {
 					FaultInjection: &egv1a1.FaultInjection{
 						Abort: &egv1a1.FaultInjectionAbort{
 							HTTPStatus: new(int32(200)),
-							GrpcStatus: new(int32(80)),
+							GrpcStatus: new(uint32(80)),
 						},
 					},
 				}
@@ -1044,7 +1044,7 @@ func TestBackendTrafficPolicyTarget(t *testing.T) {
 					},
 					FaultInjection: &egv1a1.FaultInjection{
 						Abort: &egv1a1.FaultInjectionAbort{
-							GrpcStatus: new(int32(10)),
+							GrpcStatus: new(uint32(10)),
 							Percentage: new(float32(20)),
 						},
 					},
diff --git a/test/helm/gateway-crds-helm/all.out.yaml b/test/helm/gateway-crds-helm/all.out.yaml
@@ -22876,6 +22876,8 @@ spec:
                         description: GrpcStatus specifies the GRPC status code to
                           be returned
                         format: int32
+                        maximum: 16
+                        minimum: 0
                         type: integer
                       httpStatus:
                         description: StatusCode specifies the HTTP status code to
diff --git a/test/helm/gateway-crds-helm/e2e.out.yaml b/test/helm/gateway-crds-helm/e2e.out.yaml
@@ -849,6 +849,8 @@ spec:
                         description: GrpcStatus specifies the GRPC status code to
                           be returned
                         format: int32
+                        maximum: 16
+                        minimum: 0
                         type: integer
                       httpStatus:
                         description: StatusCode specifies the HTTP status code to
diff --git a/test/helm/gateway-crds-helm/envoy-gateway-crds.out.yaml b/test/helm/gateway-crds-helm/envoy-gateway-crds.out.yaml
@@ -849,6 +849,8 @@ spec:
                         description: GrpcStatus specifies the GRPC status code to
                           be returned
                         format: int32
+                        maximum: 16
+                        minimum: 0
                         type: integer
                       httpStatus:
                         description: StatusCode specifies the HTTP status code to

Original file line number	Diff line number	Diff line change
`@@ -1655,7 +1655,7 @@ type FaultInjectionAbort struct {`
`1655`	`1655`	`// HTTPStatus defines the HTTP status code to be returned.`
`1656`	`1656`	HTTPStatus *int32 `json:"httpStatus,omitempty" yaml:"httpStatus,omitempty"`
`1657`	`1657`	`// GrpcStatus defines the gRPC status code to be returned.`
`1658`		- GrpcStatus *int32 `json:"grpcStatus,omitempty" yaml:"grpcStatus,omitempty"`
	`1658`	+ GrpcStatus *uint32 `json:"grpcStatus,omitempty" yaml:"grpcStatus,omitempty"`
`1659`	`1659`	`// Percentage defines the percentage of requests to be aborted.`
`1660`	`1660`	Percentage *float32 `json:"percentage,omitempty" yaml:"percentage,omitempty"`
`1661`	`1661`	`}`
Original file line number	Diff line number	Diff line change
`@@ -152,7 +152,7 @@ func (fault) patchRoute(route routev3.Route, irRoute ir.HTTPRoute, _ ir.HTTP`
`152`	`152`	`}`
`153`	`153`	`if abort.GrpcStatus != nil {`
`154`	`154`	`routeCfgProto.Abort.ErrorType = &xdshttpfaultv3.FaultAbort_GrpcStatus{`
`155`		`- GrpcStatus: uint32(*abort.GrpcStatus),`
	`155`	`+ GrpcStatus: *abort.GrpcStatus,`
`156`	`156`	`}`
`157`	`157`	`}`
`158`	`158`	`}`