Merge pull request #1388 from tahsinrahman/explicit-client-machine

🏃 MachineReconciler: make Client a proper field

Author: k8s-ci-robot

controllers/machine_controller.go

@@ -54,8 +54,8 @@ var (
 
 // MachineReconciler reconciles a Machine object
 type MachineReconciler struct {
-	client.Client
-	Log logr.Logger
+	Client client.Client
+	Log    logr.Logger
 
 	controller       controller.Controller
 	recorder         record.EventRecorder
@@ -91,7 +91,7 @@ func (r *MachineReconciler) Reconcile(req ctrl.Request) (_ ctrl.Result, reterr e
 	}
 
 	// Initialize the patch helper
-	patchHelper, err := patch.NewHelper(m, r)
+	patchHelper, err := patch.NewHelper(m, r.Client)
 	if err != nil {
 		return ctrl.Result{}, err
 	}
@@ -310,7 +310,7 @@ func (r *MachineReconciler) reconcileDeleteExternal(ctx context.Context, m *clus
 
 	// Issue a delete request for any object that has been found.
 	for _, obj := range objects {
-		if err := r.Delete(ctx, obj); err != nil && !apierrors.IsNotFound(err) {
+		if err := r.Client.Delete(ctx, obj); err != nil && !apierrors.IsNotFound(err) {
 			return false, errors.Wrapf(err,
 				"failed to delete %v %q for Machine %q in namespace %q",
 				obj.GroupVersionKind(), obj.GetName(), m.Name, m.Namespace)

controllers/machine_controller_phases.go

@@ -98,7 +98,7 @@ func (r *MachineReconciler) reconcileExternal(ctx context.Context, m *clusterv1.
 
 	if !util.HasOwnerRef(obj.GetOwnerReferences(), machineOwnerRef) {
 		obj.SetOwnerReferences(util.EnsureOwnerRef(obj.GetOwnerReferences(), machineOwnerRef))
-		if err := r.Patch(ctx, obj, objPatch); err != nil {
+		if err := r.Client.Patch(ctx, obj, objPatch); err != nil {
 			return nil, errors.Wrapf(err,
 				"failed to set OwnerReference on %v %q for Machine %q in namespace %q",
 				obj.GroupVersionKind(), ref.Name, m.Name, m.Namespace)
@@ -247,7 +247,7 @@ func (r *MachineReconciler) reconcileClusterStatus(ctx context.Context, cluster
 	// set the Status.ControlPlaneInitialized on the Cluster.
 	if util.IsControlPlaneMachine(m) && m.Status.NodeRef != nil {
 		if !cluster.Status.ControlPlaneInitialized {
-			patchHelper, err := patch.NewHelper(cluster, r)
+			patchHelper, err := patch.NewHelper(cluster, r.Client)
 			if err != nil {
 				return errors.Wrapf(err, "failed to create patch helper for Cluster %q in namespace %q",
 					cluster.Name, cluster.Namespace)