fix: rebuild status when the config has been used

In order for us to get to this point the owner ref must be set by the
machine controller, which implies that the machine has a pointer to this
config object. So either it's an extremely unlikely user error, or we
mistakenly dropped the "ready" flag for this config.

Author: sethp-nr

controllers/kubeadmconfig_controller.go

@@ -137,10 +137,16 @@ func (r *KubeadmConfigReconciler) Reconcile(req ctrl.Request) (_ ctrl.Result, re
 	case config.Status.Ready && machine.Status.InfrastructureReady:
 		log.Info("ignoring config for an already ready machine")
 		return ctrl.Result{}, nil
-	// Ignore machines that already have bootstrap data we didn't generate
+	// Reconcile status for machines that have already copied bootstrap data
 	case machine.Spec.Bootstrap.Data != nil && !config.Status.Ready:
-		// TODO: mark the config as ready?
-		return ctrl.Result{}, nil
+		config.Status.Ready = true
+		// Initialize the patch helper
+		patchHelper, err := patch.NewHelper(config, r)
+		if err != nil {
+			return ctrl.Result{}, err
+		}
+		err = patchHelper.Patch(ctx, config)
+		return ctrl.Result{}, err
 	// If we've already embedded a time-limited join token into a config, but are still waiting for the token to be used, refresh it
 	case config.Status.Ready:
 		token := config.Spec.JoinConfiguration.Discovery.BootstrapToken.Token