Add feature to create ports with custom options

This commit adds a new field to the v1alpha4 API:
OpenStackMachineTemplateSpec.ports.

The list of ports are added per instance. Each port may be customized
with options. These ports are created in addition to any ports created
in the `networks:` field.

If `networks:` is not specified, only the ports specified in `ports:`
will be created and attached to the instance. If neither `networks:` nor
`ports:` are specified, the instance will be connected to the default
cluster network and subnet.

This feature is very much based on the work on jsen-, and a lot of
credit goes there for the implementation:

kubernetes-sigs#778

Author: macaptain

api/v1alpha3/conversion.go

@@ -118,3 +118,39 @@ func Convert_v1alpha3_OpenStackClusterSpec_To_v1alpha4_OpenStackClusterSpec(in *
 func Convert_v1alpha3_OpenStackMachineSpec_To_v1alpha4_OpenStackMachineSpec(in *OpenStackMachineSpec, out *v1alpha4.OpenStackMachineSpec, s conversion.Scope) error {
 	return autoConvert_v1alpha3_OpenStackMachineSpec_To_v1alpha4_OpenStackMachineSpec(in, out, s)
 }
+
+// Convert_v1alpha3_OpenStackClusterSpec_To_v1alpha4_OpenStackClusterSpec has to be added by us for the new portOpts
+// parameter in v1alpha4. There is no intention to support this parameter in v1alpha3, so the field is just dropped.
+func Convert_v1alpha4_Network_To_v1alpha3_Network(in *v1alpha4.Network, out *Network, s conversion.Scope) error {
+	return autoConvert_v1alpha4_Network_To_v1alpha3_Network(in, out, s)
+}
+
+// Convert_v1alpha3_OpenStackClusterSpec_To_v1alpha4_OpenStackClusterSpec has to be added by us for the new ports
+// parameter in v1alpha4. There is no intention to support this parameter in v1alpha3, so the field is just dropped.
+func Convert_v1alpha4_OpenStackMachineSpec_To_v1alpha3_OpenStackMachineSpec(in *v1alpha4.OpenStackMachineSpec, out *OpenStackMachineSpec, s conversion.Scope) error {
+	return autoConvert_v1alpha4_OpenStackMachineSpec_To_v1alpha3_OpenStackMachineSpec(in, out, s)
+}
+
+func Convert_Slice_v1alpha4_Network_To_Slice_v1alpha3_Network(in *[]v1alpha4.Network, out *[]Network, s conversion.Scope) error {
+	for i := range *in {
+		inNet := &(*in)[i]
+		outNet := new(Network)
+		if err := autoConvert_v1alpha4_Network_To_v1alpha3_Network(inNet, outNet, s); err != nil {
+			return err
+		}
+		*out = append(*out, *outNet)
+	}
+	return nil
+}
+
+func Convert_Slice_v1alpha3_Network_To_Slice_v1alpha4_Network(in *[]Network, out *[]v1alpha4.Network, s conversion.Scope) error {
+	for i := range *in {
+		inNet := &(*in)[i]
+		outNet := new(v1alpha4.Network)
+		if err := autoConvert_v1alpha3_Network_To_v1alpha4_Network(inNet, outNet, s); err != nil {
+			return err
+		}
+		*out = append(*out, *outNet)
+	}
+	return nil
+}

api/v1alpha3/zz_generated.conversion.go

@@ -55,9 +55,13 @@ type OpenStackMachineSpec struct {
 	SSHKeyName string `json:"sshKeyName,omitempty"`
 
 	// A networks object. Required parameter when there are multiple networks defined for the tenant.
-	// When you do not specify the networks parameter, the server attaches to the only network created for the current tenant.
+	// When you do not specify both networks and ports parameters, the server attaches to the only network created for the current tenant.
 	Networks []NetworkParam `json:"networks,omitempty"`
 
+	// Ports to be attached to the server instance. They are created if a port with the given name does not already exist.
+	// When you do not specify both networks and ports parameters, the server attaches to the only network created for the current tenant.
+	Ports []PortOpts `json:"ports,omitempty"`
+
 	// UUID, IP address of a port from this subnet will be marked as AccessIPv4 on the created compute instance
 	Subnet string `json:"subnet,omitempty"`
 

api/v1alpha4/openstackmachine_types.go

@@ -58,7 +58,7 @@ type NetworkParam struct {
 	// The UUID of the network. Required if you omit the port attribute.
 	UUID string `json:"uuid,omitempty"`
 	// A fixed IPv4 address for the NIC.
-	FixedIP string `json:"fixedIp,omitempty"`
+	FixedIP string `json:"fixedIP,omitempty"`
 	// Filters for optional network query
 	Filter Filter `json:"filter,omitempty"`
 	// Subnet within a network to use
@@ -116,6 +116,38 @@ type SubnetFilter struct {
 	NotTagsAny      string `json:"notTagsAny,omitempty"`
 }
 
+type PortOpts struct {
+	// ID of the OpenStack network on which to create the port. If unspecified, create the port on the default cluster network.
+	NetworkID string `json:"networkId,omitempty"`
+	// Used to make the name of the port unique. If unspecified, instead the 0-based index of the port in the list is used.
+	NameSuffix   string `json:"nameSuffix,omitempty"`
+	Description  string `json:"description,omitempty"`
+	AdminStateUp *bool  `json:"adminStateUp,omitempty"`
+	MACAddress   string `json:"macAddress,omitempty"`
+	// Specify pairs of subnet and/or IP address. These should be subnets of the network with the given NetworkID.
+	FixedIPs            []FixedIP     `json:"fixedIPs,omitempty"`
+	TenantID            string        `json:"tenantId,omitempty"`
+	ProjectID           string        `json:"projectId,omitempty"`
+	SecurityGroups      *[]string     `json:"securityGroups,omitempty"`
+	AllowedAddressPairs []AddressPair `json:"allowedAddressPairs,omitempty"`
+
+	// The ID of the host where the port is allocated
+	HostID string `json:"hostId,omitempty"`
+
+	// The virtual network interface card (vNIC) type that is bound to the neutron port.
+	VNICType string `json:"vnicType,omitempty"`
+}
+
+type FixedIP struct {
+	SubnetID  string `json:"subnetId"`
+	IPAddress string `json:"ipAddress,omitempty"`
+}
+
+type AddressPair struct {
+	IPAddress  string `json:"ipAddress,omitempty"`
+	MACAddress string `json:"macAddress,omitempty"`
+}
+
 type Instance struct {
 	ID             string            `json:"id,omitempty"`
 	Name           string            `json:"name,omitempty"`
@@ -145,16 +177,17 @@ type RootVolume struct {
 	Size       int    `json:"diskSize,omitempty"`
 }
 
-// Network represents basic information about the associated OpenStach Neutron Network.
+// Network represents basic information about an OpenStack Neutron Network associated with an instance's port.
 type Network struct {
 	Name string `json:"name"`
 	ID   string `json:"id"`
 
 	//+optional
 	Tags []string `json:"tags,omitempty"`
 
-	Subnet *Subnet `json:"subnet,omitempty"`
-	Router *Router `json:"router,omitempty"`
+	Subnet   *Subnet   `json:"subnet,omitempty"`
+	PortOpts *PortOpts `json:"port,omitempty"`
+	Router   *Router   `json:"router,omitempty"`
 
 	// Be careful when using APIServerLoadBalancer, because this field is optional and therefore not
 	// set in all cases