minor changes

Author: Arvind Thirumurugan

tools/README.md

@@ -46,4 +46,4 @@ go run tools/uncordon-cluster/main/main.go --hubClusterContext <hub-cluster-cont
 the tool currently is a go program that also takes the hub cluster context and the member cluster name as arguments.
 
 The command internally removes all taints add to a `MemberCluster` resource of the member cluster and hence if any 
-`ClusterResourcePlacementEviction` object present which can propagate resources to the member cluster, it can continue to do so.
+`ClusterResourcePlacementEviction` object present which can propagate resources to the member cluster, it can continue to do so.

tools/cordon-cluster/main/main.go

@@ -128,8 +128,8 @@ func main() {
 		}
 	}
 
-	// wait until all evictions reach a terminal state.
 	// TODO: move isEvictionInTerminalState to a function in the pkg/utils package.
+	// wait until all evictions reach a terminal state.
 	for crpName := range crpNameMap {
 		err = wait.ExponentialBackoffWithContext(ctx, retry.DefaultBackoff, func(ctx context.Context) (bool, error) {
 			evictionName := fmt.Sprintf(testEvictionNameFormat, crpName, *clusterName)

tools/uncordon-cluster/main/main.go

@@ -65,5 +65,9 @@ func main() {
 		return hubClient.Update(ctx, &mc)
 	})
 
+	if err != nil {
+		log.Fatalf("failed to remove taints from member cluster %s: %v", *clusterName, err)
+	}
+
 	log.Printf("uncordoned member cluster %s", *clusterName)
 }