need to requeue if bootstrap data not ready, secret must be b64encoded

cloud/scope/machine.go

@@ -2,6 +2,7 @@ package scope
 
 import (
 	"context"
+	b64 "encoding/base64"
 	"errors"
 	"fmt"
 	corev1 "k8s.io/api/core/v1"
@@ -101,5 +102,5 @@ func (m *MachineScope) GetBootstrapData() (string, error) {
 			m.LinodeMachine.Name,
 		)
 	}
-	return string(value), nil
+	return b64.StdEncoding.EncodeToString(value), nil
 }

controller/linodemachine_controller.go

@@ -119,7 +119,6 @@ func (r *LinodeMachineReconciler) Reconcile(ctx context.Context, req ctrl.Reques
 
 		return ctrl.Result{}, nil
 	case skippedMachinePhases[machine.Status.Phase]:
-		log.Info("Machine phase is not the one we are looking for, skipping reconciliation", "phase", machine.Status.Phase)
 
 		return ctrl.Result{}, nil
 	default:
@@ -213,12 +212,6 @@ func (r *LinodeMachineReconciler) reconcile(
 	// Add the finalizer if not already there
 	controllerutil.AddFinalizer(machineScope.LinodeMachine, infrav1.GroupVersion.String())
 
-	// Make sure bootstrap data is available and populated.
-	if machineScope.Machine.Spec.Bootstrap.DataSecretName == nil {
-		logger.Info("Bootstrap data secret is not yet available")
-		return res, nil
-	}
-
 	// Delete
 	if !machineScope.LinodeMachine.ObjectMeta.DeletionTimestamp.IsZero() {
 		failureReason = cerrs.DeleteMachineError
@@ -249,7 +242,11 @@ func (r *LinodeMachineReconciler) reconcile(
 
 	// Create
 	failureReason = cerrs.CreateMachineError
-
+	// Make sure bootstrap data is available and populated.
+	if machineScope.Machine.Spec.Bootstrap.DataSecretName == nil {
+		logger.Info("Bootstrap data secret is not yet available")
+		return ctrl.Result{RequeueAfter: 5 * time.Second}, nil
+	}
 	linodeInstance, err = r.reconcileCreate(ctx, machineScope, logger)
 
 	return
@@ -292,6 +289,8 @@ func (*LinodeMachineReconciler) reconcileCreate(ctx context.Context, machineScop
 		// get the bootstrap data for the Linode instance and set it for create config
 		bootstrapData, err := machineScope.GetBootstrapData()
 		if err != nil {
+			logger.Info("Failed to get bootstrap data", "error", err.Error())
+
 			return nil, err
 		}
 		createConfig.Metadata = &linodego.InstanceMetadataOptions{