Merge pull request #1386 from tahsinrahman/explicit-client-ms

🏃 MachineSetReconciler: make Client a proper field
kubernetes-sigs · Sep 5, 2019 · c5952ee · c5952ee
2 parents 44cf8e2 + 7636978
commit c5952ee
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/controllers/machineset_controller.go b/controllers/machineset_controller.go
@@ -62,8 +62,8 @@ var (
 
 // MachineSetReconciler reconciles a MachineSet object
 type MachineSetReconciler struct {
-	client.Client
-	Log logr.Logger
+	Client client.Client
+	Log    logr.Logger
 
 	recorder record.EventRecorder
 }
@@ -88,7 +88,7 @@ func (r *MachineSetReconciler) Reconcile(req ctrl.Request) (ctrl.Result, error)
 	_ = r.Log.WithValues("machineset", req.NamespacedName)
 
 	machineSet := &clusterv1.MachineSet{}
-	if err := r.Get(ctx, req.NamespacedName, machineSet); err != nil {
+	if err := r.Client.Get(ctx, req.NamespacedName, machineSet); err != nil {
 		if apierrors.IsNotFound(err) {
 			// Object not found, return. Created objects are automatically garbage collected.
 			// For additional cleanup logic use finalizers.