fix setting of interfaceGeneration

Author: AshleyDumaine

internal/controller/linodemachine_controller.go

@@ -553,10 +553,6 @@ func (r *LinodeMachineReconciler) reconcilePreflightMetadataSupportConfigure(ctx
 }
 
 func (r *LinodeMachineReconciler) reconcilePreflightCreate(ctx context.Context, logger logr.Logger, machineScope *scope.MachineScope) (ctrl.Result, error) {
-	// default to legacy interface generation if not set for now
-	if machineScope.LinodeMachine.Spec.InterfaceGeneration == "" {
-		machineScope.LinodeMachine.Spec.InterfaceGeneration = linodego.GenerationLegacyConfig
-	}
 	// get the bootstrap data for the Linode instance and set it for create config
 	createOpts, err := newCreateConfig(ctx, machineScope, r.GzipCompressionEnabled, logger)
 	if err != nil {

internal/controller/linodemachine_controller_helpers.go

@@ -81,16 +81,23 @@ func retryIfTransient(err error, logger logr.Logger) (ctrl.Result, error) {
 }
 
 func fillCreateConfig(createConfig *linodego.InstanceCreateOptions, machineScope *scope.MachineScope) {
+	// default to legacy interface generation if not set for now unless linodeInterfaces are defined
+	if machineScope.LinodeMachine.Spec.InterfaceGeneration == "" {
+		if machineScope.LinodeMachine.Spec.LinodeInterfaces != nil {
+			createConfig.InterfaceGeneration = linodego.GenerationLinode
+		} else {
+			createConfig.InterfaceGeneration = linodego.GenerationLegacyConfig
+		}
+	}
 	if machineScope.LinodeMachine.Spec.PrivateIP != nil {
 		createConfig.PrivateIP = *machineScope.LinodeMachine.Spec.PrivateIP
 	} else {
-		if machineScope.LinodeMachine.Spec.LinodeInterfaces == nil && machineScope.LinodeMachine.Spec.InterfaceGeneration == linodego.GenerationLegacyConfig {
+		if createConfig.InterfaceGeneration == linodego.GenerationLegacyConfig {
 			// Supported only for legacy network interfaces.
 			createConfig.PrivateIP = true
 		} else {
 			// Network Helper is not supported for the new network interfaces.
 			createConfig.NetworkHelper = nil
-			createConfig.InterfaceGeneration = linodego.GenerationLinode
 		}
 	}
 
@@ -1043,15 +1050,16 @@ func constructLinodeInterfaceCreateOpts(createOpts []infrav1alpha2.LinodeInterfa
 // for converting LinodeMachineSpec to linodego.InstanceCreateOptions. Any defaulting should be done in fillCreateConfig instead
 func linodeMachineSpecToInstanceCreateConfig(machineSpec infrav1alpha2.LinodeMachineSpec, machineTags []string) *linodego.InstanceCreateOptions {
 	instCreateOpts := &linodego.InstanceCreateOptions{
-		Region:          machineSpec.Region,
-		Type:            machineSpec.Type,
-		AuthorizedKeys:  machineSpec.AuthorizedKeys,
-		AuthorizedUsers: machineSpec.AuthorizedUsers,
-		RootPass:        machineSpec.RootPass,
-		Image:           machineSpec.Image,
-		Tags:            machineTags,
-		FirewallID:      machineSpec.FirewallID,
-		DiskEncryption:  linodego.InstanceDiskEncryption(machineSpec.DiskEncryption),
+		Region:              machineSpec.Region,
+		Type:                machineSpec.Type,
+		AuthorizedKeys:      machineSpec.AuthorizedKeys,
+		AuthorizedUsers:     machineSpec.AuthorizedUsers,
+		RootPass:            machineSpec.RootPass,
+		Image:               machineSpec.Image,
+		Tags:                machineTags,
+		FirewallID:          machineSpec.FirewallID,
+		InterfaceGeneration: machineSpec.InterfaceGeneration,
+		DiskEncryption:      linodego.InstanceDiskEncryption(machineSpec.DiskEncryption),
 	}
 
 	if machineSpec.PrivateIP != nil {