improve comments for aviunding misunderstanding from object graph nodes and Kubernetes nodes

Author: fabriziopandini

cmd/clusterctl/pkg/client/cluster/mover.go

@@ -120,8 +120,8 @@ func (o *objectMover) move(graph *objectGraph, toProxy Proxy) error {
 		return err
 	}
 
-	// Define the move sequence by processing the ownerReference chain, so we ensure that a node is moved only after its owners.
-	// Additionally nodes are grouped, so bulk of nodes can be moved in parallel. e.g.
+	// Define the move sequence by processing the ownerReference chain, so we ensure that a Kubernetes object is moved only after its owners.
+	// The sequence is bases on object graph nodes, each one representing a Kubernetes object; nodes are grouped, so bulk of nodes can be moved in parallel. e.g.
 	// - All the Clusters should be moved first (group 1, processed in parallel)
 	// - All the MachineDeployments should be moved second (group 1, processed in parallel)
 	// - then all the MachineSets, then all the Machines, etc.
@@ -155,7 +155,7 @@ type moveSequence struct {
 	nodesMap map[*node]empty
 }
 
-// moveGroup defines is a list of nodes that can be moved in parallel.
+// moveGroup defines is a list of nodes read from the object graph that can be moved in parallel.
 type moveGroup []*node
 
 func (s *moveSequence) addGroup(group moveGroup) {
@@ -260,10 +260,10 @@ const (
 	retryIntervalCreateTargetObject = 1 * time.Second
 )
 
-// createGroup creates all the nodes in a moveGroup into the target management cluster.
+// createGroup creates all the Kubernetes objects into the target management cluster corresponding to the object graph nodes in a moveGroup.
 func (o *objectMover) createGroup(group moveGroup, toProxy Proxy) error {
 
-	// Creates - in parallel - the all the nodes in the group.
+	// Creates - in parallel - all the nodes in the group.
 	var wg sync.WaitGroup
 	errList := []error{}
 	errCh := make(chan error)
@@ -299,7 +299,7 @@ func (o *objectMover) createGroup(group moveGroup, toProxy Proxy) error {
 	return nil
 }
 
-// createTargetObject creates the Kubernetes object corresponding to the node in the target Management cluster, taking care of restoring the OwnerReference with the owner node, if any.
+// createTargetObject creates the Kubernetes object in the target Management cluster corresponding to the object graph node, taking care of restoring the OwnerReference with the owner nodes, if any.
 func (o *objectMover) createTargetObject(nodeToCreate *node, toProxy Proxy) error {
 	o.log.V(1).Info("Creating", nodeToCreate.identity.Kind, nodeToCreate.identity.Name, "Namespace", nodeToCreate.identity.Namespace)
 
@@ -395,7 +395,7 @@ const (
 	retryIntervalDeleteSourceObject = 1 * time.Second
 )
 
-// deleteGroup deletes all the nodes in a moveGroup from the source management cluster.
+// deleteGroup deletes all the Kubernetes objects from the source management cluster corresponding to the object graph nodes in a moveGroup.
 func (o *objectMover) deleteGroup(group moveGroup) error {
 	// Deletes - in parallel - the dependents nodes and the softDependents nodes (with the respective object tree).
 	var wg sync.WaitGroup
@@ -435,7 +435,7 @@ var (
 	removeFinalizersPatch = client.ConstantPatch(types.MergePatchType, []byte("{\"metadata\":{\"finalizers\":[]}}"))
 )
 
-// deleteSourceObject deletes the object corresponding to the node from the source management cluster, taking care of removing all the finalizers so
+// deleteSourceObject deletes the Kubernetes object corresponding to the node from the source management cluster, taking care of removing all the finalizers so
 // the objects gets immediately deleted (force delete).
 func (o *objectMover) deleteSourceObject(nodeToDelete *node) error {
 	o.log.V(1).Info("Deleting", nodeToDelete.identity.Kind, nodeToDelete.identity.Name, "Namespace", nodeToDelete.identity.Namespace)