Skip to content

Commit 12ef69d

Browse files
committed
OPNET-476: GA Load balancer feature for Nutanix
As part of https://issues.redhat.com/browse/OPNET-305 we are promoting External Load Balancer to GA. This requires installer to soften validations so that the feature is not any more limited only to TechPreview clusters. Contributes-to: OPNET-476
1 parent e527352 commit 12ef69d

File tree

2 files changed

+0
-23
lines changed

2 files changed

+0
-23
lines changed

pkg/types/nutanix/validation/platform.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -73,13 +73,6 @@ func ValidatePlatform(p *nutanix.Platform, fldPath *field.Path, c *types.Install
7373
allErrs = append(allErrs, field.Required(fldPath.Child("subnet"), "must specify the subnet"))
7474
}
7575

76-
// Platform fields only allowed in TechPreviewNoUpgrade
77-
if c.FeatureSet != configv1.TechPreviewNoUpgrade {
78-
if c.Nutanix.LoadBalancer != nil {
79-
allErrs = append(allErrs, field.Forbidden(fldPath.Child("loadBalancer"), "load balancer is not supported in this feature set"))
80-
}
81-
}
82-
8376
if c.Nutanix.LoadBalancer != nil {
8477
if !validateLoadBalancer(c.Nutanix.LoadBalancer.Type) {
8578
allErrs = append(allErrs, field.Invalid(fldPath.Child("loadBalancer", "type"), c.Nutanix.LoadBalancer.Type, "invalid load balancer type"))

pkg/types/nutanix/validation/platform_test.go

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -46,22 +46,6 @@ func TestValidatePlatform(t *testing.T) {
4646
}(),
4747
expectedError: `^test-path\.prismCentral\.endpoint\.address: Required value: must specify the Prism Central endpoint address$`,
4848
},
49-
{
50-
name: "forbidden load balancer field",
51-
platform: validPlatform(),
52-
config: &types.InstallConfig{
53-
Platform: types.Platform{
54-
Nutanix: func() *nutanix.Platform {
55-
p := validPlatform()
56-
p.LoadBalancer = &configv1.NutanixPlatformLoadBalancer{
57-
Type: configv1.LoadBalancerTypeOpenShiftManagedDefault,
58-
}
59-
return p
60-
}(),
61-
},
62-
},
63-
expectedError: `^test-path\.loadBalancer: Forbidden: load balancer is not supported in this feature set`,
64-
},
6549
{
6650
name: "allowed load balancer field with OpenShift managed default",
6751
platform: validPlatform(),

0 commit comments

Comments
 (0)