chore: renames IngressGatewaySpec type to GatewaySpec

Original name is confusing as it does not actually define anything related to the ingress. It is configuring Istio Gateway resource which is, in case of Model Serving, attached to ingress service. This change makes the change of the golang type, but not publicly exposed `ingressGateway` config defined for KServe component.
opendatahub-io · Jul 29, 2024 · 3d3e6c9 · 3d3e6c9
1 parent df92280
commit 3d3e6c9
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 14 deletions.
diff --git a/apis/infrastructure/v1/serverless_types.go b/apis/infrastructure/v1/serverless_types.go
@@ -17,5 +17,5 @@ type ServingSpec struct {
 	Name string `json:"name,omitempty"`
 	// IngressGateway allows to customize some parameters for the Istio Ingress Gateway
 	// that is bound to KNative-Serving.
-	IngressGateway IngressGatewaySpec `json:"ingressGateway,omitempty"`
+	IngressGateway GatewaySpec `json:"ingressGateway,omitempty"`
 }
diff --git a/apis/infrastructure/v1/servicemesh_types.go b/apis/infrastructure/v1/servicemesh_types.go
@@ -30,15 +30,15 @@ type ControlPlaneSpec struct {
 	MetricsCollection string `json:"metricsCollection,omitempty"`
 }
 
-// IngressGatewaySpec represents the configuration of the Ingress Gateways.
-type IngressGatewaySpec struct {
+// GatewaySpec represents the configuration of the Ingress Gateways.
+type GatewaySpec struct {
 	// Domain specifies the DNS name for intercepting ingress requests coming from
 	// outside the cluster. Most likely, you will want to use a wildcard name,
 	// like *.example.com. If not set, the domain of the OpenShift Ingress is used.
 	// If you choose to generate a certificate, this is the domain used for the certificate request.
 	Domain string `json:"domain,omitempty"`
 	// Certificate specifies configuration of the TLS certificate securing communications of
-	// the for Ingress Gateway.
+	// the for gateway.
 	Certificate CertificateSpec `json:"certificate,omitempty"`
 }
 

diff --git a/apis/infrastructure/v1/zz_generated.deepcopy.go b/apis/infrastructure/v1/zz_generated.deepcopy.go
diff --git a/config/crd/bases/datasciencecluster.opendatahub.io_datascienceclusters.yaml b/config/crd/bases/datasciencecluster.opendatahub.io_datascienceclusters.yaml
@@ -246,7 +246,7 @@ spec:
                               certificate:
                                 description: Certificate specifies configuration of
                                   the TLS certificate securing communications of the
-                                  for Ingress Gateway.
+                                  for gateway.
                                 properties:
                                   secretName:
                                     description: SecretName specifies the name of

diff --git a/tests/e2e/dsc_creation_test.go b/tests/e2e/dsc_creation_test.go
@@ -348,7 +348,7 @@ func (tc *testContext) validateDSC() error {
 	expServingSpec := infrav1.ServingSpec{
 		ManagementState: operatorv1.Managed,
 		Name:            "knative-serving",
-		IngressGateway: infrav1.IngressGatewaySpec{
+		IngressGateway: infrav1.GatewaySpec{
 			Certificate: infrav1.CertificateSpec{
 				Type: infrav1.OpenshiftDefaultIngress,
 			},

diff --git a/tests/integration/features/serverless_feature_test.go b/tests/integration/features/serverless_feature_test.go
@@ -181,7 +181,7 @@ var _ = Describe("Serverless feature", func() {
 			It("should set default value when value is empty in the DSCI", func(ctx context.Context) {
 				// given
 				serving := infrav1.ServingSpec{
-					IngressGateway: infrav1.IngressGatewaySpec{
+					IngressGateway: infrav1.GatewaySpec{
 						Certificate: infrav1.CertificateSpec{
 							SecretName: "",
 						},
@@ -199,7 +199,7 @@ var _ = Describe("Serverless feature", func() {
 			It("should use user value when set in the DSCI", func(ctx context.Context) {
 				// given
 				serving := infrav1.ServingSpec{
-					IngressGateway: infrav1.IngressGatewaySpec{
+					IngressGateway: infrav1.GatewaySpec{
 						Certificate: infrav1.CertificateSpec{
 							SecretName: "top-secret-service",
 						},
@@ -224,7 +224,7 @@ var _ = Describe("Serverless feature", func() {
 				Expect(envTestClient.Create(ctx, osIngressResource)).To(Succeed())
 
 				serving := infrav1.ServingSpec{
-					IngressGateway: infrav1.IngressGatewaySpec{
+					IngressGateway: infrav1.GatewaySpec{
 						Domain: "",
 					},
 				}
@@ -240,7 +240,7 @@ var _ = Describe("Serverless feature", func() {
 			It("should use user value when set in the DSCI", func(ctx context.Context) {
 				// given
 				serving := infrav1.ServingSpec{
-					IngressGateway: infrav1.IngressGatewaySpec{
+					IngressGateway: infrav1.GatewaySpec{
 						Domain: fixtures.TestDomainFooCom,
 					},
 				}