Remove requirement of having deprecated clusterNetworkCIDR/hostSubnetLength in master.networkConfig

Currently in order for openshift to start the first entry in clusterNetworks and the old clusterNetworkCIDR/hostSubnetLength have to match. Change it
so the older clusterNetworkCIDR/hostSubnetLength fields are no longer required.

Bug 1534779

Author: JacobTanenbaum

pkg/cmd/server/apis/config/v1/conversions.go

@@ -392,31 +392,17 @@ func addConversionFuncs(scheme *runtime.Scheme) error {
 			if err := s.DefaultConvert(in, out, conversion.IgnoreMissingFields); err != nil {
 				return err
 			}
-			if len(in.DeprecatedClusterNetworkCIDR) > 0 || in.DeprecatedHostSubnetLength > 0 {
-				if len(out.ClusterNetworks) > 0 {
-					out.ClusterNetworks[0].CIDR = in.DeprecatedClusterNetworkCIDR
-					out.ClusterNetworks[0].HostSubnetLength = in.DeprecatedHostSubnetLength
-				} else {
-					out.ClusterNetworks = []internal.ClusterNetworkEntry{
-						{
-							CIDR:             in.DeprecatedClusterNetworkCIDR,
-							HostSubnetLength: in.DeprecatedHostSubnetLength,
-						},
-					}
+
+			if len(out.ClusterNetworks) == 0 {
+				out.ClusterNetworks = []internal.ClusterNetworkEntry{
+					{
+						CIDR:             in.DeprecatedClusterNetworkCIDR,
+						HostSubnetLength: in.DeprecatedHostSubnetLength,
+					},
 				}
 			}
 			return nil
 		},
-		func(in *internal.MasterNetworkConfig, out *MasterNetworkConfig, s conversion.Scope) error {
-			if err := s.DefaultConvert(in, out, conversion.IgnoreMissingFields); err != nil {
-				return err
-			}
-			if len(in.ClusterNetworks) > 0 {
-				out.DeprecatedHostSubnetLength = in.ClusterNetworks[0].HostSubnetLength
-				out.DeprecatedClusterNetworkCIDR = in.ClusterNetworks[0].CIDR
-			}
-			return nil
-		},
 
 		metav1.Convert_resource_Quantity_To_resource_Quantity,
 		metav1.Convert_bool_To_Pointer_bool,

pkg/cmd/server/apis/config/validation/master.go

@@ -719,11 +719,25 @@ func ValidateIngressIPNetworkCIDR(config *configapi.MasterConfig, fldPath *field
 func ValidateDeprecatedClusterNetworkConfig(config *configapi.MasterConfig, fldPath *field.Path) ValidationResults {
 	validationResults := ValidationResults{}
 
-	if len(config.NetworkConfig.ClusterNetworks) > 0 && config.NetworkConfig.DeprecatedHostSubnetLength != config.NetworkConfig.ClusterNetworks[0].HostSubnetLength {
-		validationResults.AddErrors(field.Invalid(fldPath.Child("hostSubnetLength"), config.NetworkConfig.DeprecatedHostSubnetLength, "cannot set hostSubnetLength and clusterNetworks, please use clusterNetworks"))
-	}
-	if len(config.NetworkConfig.ClusterNetworks) > 0 && config.NetworkConfig.DeprecatedClusterNetworkCIDR != config.NetworkConfig.ClusterNetworks[0].CIDR {
-		validationResults.AddErrors(field.Invalid(fldPath.Child("clusterNetworkCIDR"), config.NetworkConfig.DeprecatedClusterNetworkCIDR, "cannot set clusterNetworkCIDR and clusterNetworks, please use clusterNetworks"))
+	fmt.Printf("KEYWORD: len(config.NetworkConfig.ClusterNetworks)=%d\n", len(config.NetworkConfig.ClusterNetworks))
+	if len(config.NetworkConfig.ClusterNetworks) > 1 {
+		if config.NetworkConfig.DeprecatedHostSubnetLength != 0 {
+			validationResults.AddErrors(field.Invalid(fldPath.Child("hostSubnetLength"), config.NetworkConfig.DeprecatedHostSubnetLength, "cannot set hostSubnetLength and clusterNetworks, please use clusterNetworks"))
+		}
+		if len(config.NetworkConfig.DeprecatedClusterNetworkCIDR) != 0 {
+			validationResults.AddErrors(field.Invalid(fldPath.Child("clusterNetworkCIDR"), config.NetworkConfig.DeprecatedClusterNetworkCIDR, "cannot set clusterNetworkCIDR and clusterNetworks, please use clusterNetworks"))
+		}
+
+	} else if len(config.NetworkConfig.ClusterNetworks) == 1 {
+		fmt.Printf("KEYWORD: I THINK YOU ARE HERE!\n")
+		if config.NetworkConfig.DeprecatedHostSubnetLength != config.NetworkConfig.ClusterNetworks[0].HostSubnetLength && config.NetworkConfig.DeprecatedHostSubnetLength != 0 {
+			fmt.Printf("\tKEYWORD: condif.NetworkConfig.DeprecatedHostsubnetLength: %d\n\tKEYWORD:config.NetworkConfig.ClusterNEtworks[0].HostSubnetLength:%d\n", config.NetworkConfig.DeprecatedHostSubnetLength, config.NetworkConfig.ClusterNetworks[0].HostSubnetLength)
+			validationResults.AddErrors(field.Invalid(fldPath.Child("hostSubnetLength"), config.NetworkConfig.DeprecatedHostSubnetLength, "cannot set hostSubnetLength and clusterNetworks, please use clusterNetworks"))
+		}
+		if config.NetworkConfig.DeprecatedClusterNetworkCIDR != config.NetworkConfig.ClusterNetworks[0].CIDR && len(config.NetworkConfig.DeprecatedClusterNetworkCIDR) != 0 {
+			validationResults.AddErrors(field.Invalid(fldPath.Child("clusterNetworkCIDR"), config.NetworkConfig.DeprecatedClusterNetworkCIDR, "cannot set clusterNetworkCIDR and clusterNetworks, please use clusterNetworks"))
+		}
 	}
+
 	return validationResults
 }