Remove optional param CtlplaneNetmask

This change removes the CtlplaneNetmask parameter since it is not required since we're using CIDR's. Signed-off-by: Brendan Shephard <bshephar@redhat.com>
openstack-k8s-operators · Jan 10, 2025 · 6cefc79 · 6cefc79
1 parent 0fd6a2a
commit 6cefc79
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 27 deletions.
diff --git a/api/bases/baremetal.openstack.org_openstackbaremetalsets.yaml b/api/bases/baremetal.openstack.org_openstackbaremetalsets.yaml
@@ -143,10 +143,6 @@ spec:
                 description: CtlplaneInterface - Interface on the provisioned nodes
                   to use for ctlplane network
                 type: string
-              ctlplaneNetmask:
-                description: 'CtlplaneNetmask - Netmask to use for ctlplane network
-                  (TODO: acquire this is another manner?)'
-                type: string
               ctlplaneVlan:
                 description: CtlplaneVlan - Vlan for ctlplane network
                 type: integer

diff --git a/api/v1beta1/openstackbaremetalset_types.go b/api/v1beta1/openstackbaremetalset_types.go
@@ -80,9 +80,6 @@ type OpenStackBaremetalSetTemplateSpec struct {
 	DeploymentSSHSecret string `json:"deploymentSSHSecret"`
 	// CtlplaneInterface - Interface on the provisioned nodes to use for ctlplane network
 	CtlplaneInterface string `json:"ctlplaneInterface"`
-	// +kubebuilder:validation:Optional
-	// CtlplaneNetmask - Netmask to use for ctlplane network (TODO: acquire this is another manner?)
-	CtlplaneNetmask string `json:"ctlplaneNetmask,omitempty"`
 	// +kubebuilder:default=openshift-machine-api
 	// +kubebuilder:validation:Optional
 	// BmhNamespace Namespace to look for BaremetalHosts(default: openshift-machine-api)

diff --git a/config/crd/bases/baremetal.openstack.org_openstackbaremetalsets.yaml b/config/crd/bases/baremetal.openstack.org_openstackbaremetalsets.yaml
@@ -143,10 +143,6 @@ spec:
                 description: CtlplaneInterface - Interface on the provisioned nodes
                   to use for ctlplane network
                 type: string
-              ctlplaneNetmask:
-                description: 'CtlplaneNetmask - Netmask to use for ctlplane network
-                  (TODO: acquire this is another manner?)'
-                type: string
               ctlplaneVlan:
                 description: CtlplaneVlan - Vlan for ctlplane network
                 type: integer

diff --git a/pkg/openstackbaremetalset/baremetalhost.go b/pkg/openstackbaremetalset/baremetalhost.go
@@ -99,22 +99,7 @@ func BaremetalHostProvision(
 		// Check IP version and set template variables accordingly
 		ipAddr, ipNet, err := net.ParseCIDR(ctlPlaneIP)
 		if err != nil {
-			// TODO: Remove this conversion once all usage sets ctlPlaneIP in CIDR format.
-			ipAddr = net.ParseIP(ctlPlaneIP)
-			if ipAddr == nil {
-				return err
-			}
-
-			var ipPrefix int
-			if ipAddr.To4() != nil {
-				ipPrefix, _ = net.IPMask(net.ParseIP(instance.Spec.CtlplaneNetmask).To4()).Size()
-			} else {
-				ipPrefix, _ = net.IPMask(net.ParseIP(instance.Spec.CtlplaneNetmask).To16()).Size()
-			}
-			_, ipNet, err = net.ParseCIDR(fmt.Sprintf("%s/%d", ipAddr, ipPrefix))
-			if err != nil {
-				return err
-			}
+			return err
 		}
 
 		CtlplaneIPVersion := "ipv6"