OSASINFRA-3153: move loadBalancer API to GA for OpenStack

This API has not moved and there is no plan to make any change that
would be backward incompatible in the future.

The feature was well tested (and automated) by our QE on this platform,
as well documented on OCP 4.13.

We think this API is ready to be GA'ed.

Author: EmilienM

data/data/install.openshift.io_installconfigs.yaml

@@ -3610,7 +3610,7 @@ spec:
                     type: string
                   loadBalancer:
                     description: LoadBalancer defines how the load balancer used by
-                      the cluster is configured. LoadBalancer is available in TechPreview.
+                      the cluster is configured.
                     properties:
                       type:
                         default: OpenShiftManagedDefault

pkg/types/openstack/platform.go

@@ -120,7 +120,6 @@ type Platform struct {
 	MachinesSubnet string `json:"machinesSubnet,omitempty"`
 
 	// LoadBalancer defines how the load balancer used by the cluster is configured.
-	// LoadBalancer is available in TechPreview.
 	// +optional
 	LoadBalancer *configv1.OpenStackPlatformLoadBalancer `json:"loadBalancer,omitempty"`
 }

pkg/types/openstack/validation/techpreview.go

@@ -14,10 +14,6 @@ func FilledInTechPreviewFields(installConfig *types.InstallConfig) (fields []*fi
 		return nil
 	}
 
-	if installConfig.OpenStack.LoadBalancer != nil {
-		fields = append(fields, field.NewPath("platform", "openstack", "loadBalancer"))
-	}
-
 	if installConfig.ControlPlane != nil && installConfig.ControlPlane.Platform.OpenStack != nil && len(installConfig.ControlPlane.Platform.OpenStack.FailureDomains) > 0 {
 		fields = append(fields, field.NewPath("controlPlane", "platform", "openstack", "failureDomains"))
 	}

pkg/types/openstack/validation/techpreview_test.go

@@ -2028,20 +2028,6 @@ func TestValidateInstallConfig(t *testing.T) {
 			}(),
 			expectedError: "platform.openstack.apiVIPs: Invalid value: \"foobar\": \"foobar\" is not a valid IP",
 		},
-		{
-			name: "should reject load balancer on OpenStack if not TechPreviewNoUpgrade",
-			installConfig: func() *types.InstallConfig {
-				c := validInstallConfig()
-				c.Platform = types.Platform{
-					OpenStack: validOpenStackPlatform(),
-				}
-				c.Platform.OpenStack.LoadBalancer = &configv1.OpenStackPlatformLoadBalancer{
-					Type: configv1.LoadBalancerTypeOpenShiftManagedDefault,
-				}
-				return c
-			}(),
-			expectedError: `platform.openstack.loadBalancer: Forbidden: the TechPreviewNoUpgrade feature set must be enabled to use this field`,
-		},
 		{
 			name: "should not validate vips on VSphere if not set (vips are not required on VSphere)",
 			installConfig: func() *types.InstallConfig {

pkg/types/validation/installconfig_test.go

@@ -30,5 +30,4 @@ platform:
     cloud: ${OS_CLOUD}
     loadBalancer:
       type: OpenShiftManagedDefault
-featureSet: TechPreviewNoUpgrade
 pullSecret: ${PULL_SECRET}

scripts/openstack/manifest-tests/lb-default-techpreview/install-config.yaml

@@ -30,5 +30,4 @@ platform:
     cloud: ${OS_CLOUD}
     loadBalancer:
       type: UserManaged
-featureSet: TechPreviewNoUpgrade
 pullSecret: ${PULL_SECRET}