Skip to content

Commit bc0cd0a

Browse files
Merge pull request openshift#8092 from 2uasimojo/OCPBUGS-30135/OpenShiftSDN-not-supported-message
OCPBUGS-30135: fix "OpenShiftSDN deprecated" error message
2 parents a0b6ba9 + 7f642f3 commit bc0cd0a

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

pkg/types/validation/installconfig.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -379,7 +379,7 @@ func validateNetworking(n *types.Networking, singleNodeOpenShift bool, fldPath *
379379
}
380380

381381
if n.NetworkType == string(operv1.NetworkTypeOpenShiftSDN) {
382-
allErrs = append(allErrs, field.Invalid(fldPath.Child("networkType"), n.NetworkType, "networkType OpenShiftSDN is deprecated, please use OVNKubernetes"))
382+
allErrs = append(allErrs, field.Invalid(fldPath.Child("networkType"), n.NetworkType, "networkType OpenShiftSDN is not supported, please use OVNKubernetes"))
383383
}
384384

385385
if len(n.MachineNetwork) > 0 {

pkg/types/validation/installconfig_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -564,7 +564,7 @@ func TestValidateInstallConfig(t *testing.T) {
564564
c.Networking.ClusterNetworkMTU = 8000
565565
return c
566566
}(),
567-
expectedError: `networking.networkType: Invalid value: "OpenShiftSDN": networkType OpenShiftSDN is deprecated, please use OVNKubernetes, networking.clusterNetworkMTU: Invalid value: 8000: cluster network MTU is not valid with network plugin OpenShiftSDN`,
567+
expectedError: `networking.networkType: Invalid value: "OpenShiftSDN": networkType OpenShiftSDN is not supported, please use OVNKubernetes, networking.clusterNetworkMTU: Invalid value: 8000: cluster network MTU is not valid with network plugin OpenShiftSDN`,
568568
},
569569
{
570570
name: "missing control plane",
@@ -1700,7 +1700,7 @@ func TestValidateInstallConfig(t *testing.T) {
17001700

17011701
return c
17021702
}(),
1703-
expectedError: "[networking.networkType: Invalid value: \"OpenShiftSDN\": networkType OpenShiftSDN is deprecated, please use OVNKubernetes, platform.baremetal.ingressVIPs: Invalid value: \"10.0.0.4\": IP expected to be in one of the machine networks: ffd0::/48]",
1703+
expectedError: "[networking.networkType: Invalid value: \"OpenShiftSDN\": networkType OpenShiftSDN is not supported, please use OVNKubernetes, platform.baremetal.ingressVIPs: Invalid value: \"10.0.0.4\": IP expected to be in one of the machine networks: ffd0::/48]",
17041704
},
17051705
{
17061706
name: "ingressvips_v6_on_openshiftsdn",
@@ -1716,7 +1716,7 @@ func TestValidateInstallConfig(t *testing.T) {
17161716

17171717
return c
17181718
}(),
1719-
expectedError: "[networking.networkType: Invalid value: \"OpenShiftSDN\": networkType OpenShiftSDN is deprecated, please use OVNKubernetes, platform.baremetal.apiVIPs: Invalid value: \"10.0.0.5\": IP expected to be in one of the machine networks: ffd0::/48]",
1719+
expectedError: "[networking.networkType: Invalid value: \"OpenShiftSDN\": networkType OpenShiftSDN is not supported, please use OVNKubernetes, platform.baremetal.apiVIPs: Invalid value: \"10.0.0.5\": IP expected to be in one of the machine networks: ffd0::/48]",
17201720
},
17211721
{
17221722
name: "too_many_apivips",

0 commit comments

Comments
 (0)