Merge pull request #7085 from FAUST-BENCHOU/fix/force-flag

cleanup: remove useless TODOs

Author: karmada-bot

pkg/estimator/server/estimate.go

@@ -102,8 +102,6 @@ func (es *AccurateSchedulerEstimatorServer) estimateComponents(ctx context.Conte
 		return maxSets, fmt.Errorf("estimate components plugins fails with %s", ret.Reasons())
 	}
 
-	// TODO - Node info has not be included in this implementation. Node Resource Estimation will be moved to a separate plugin.
-
 	// If no plugins were run (NoOperation), return maxSets value to prevent scheduling failure
 	if ret.IsSuccess() || ret.IsNoOperation() {
 		return maxSets, nil

pkg/karmadactl/interpret/edit.go

@@ -166,13 +166,9 @@ func (o *Options) runEdit() error {
 			file: file,
 		}
 
-		// TODO: validate edited customization
-
 		// not a syntax error as it turns out...
 		containsError = false
 
-		// TODO: add last-applied-configuration annotation
-
 		switch {
 		case info.Source != "":
 			err = o.saveToPath(info, editedCustomization, &results)

pkg/karmadactl/unregister/unregister.go

@@ -339,7 +339,6 @@ func (j *CommandUnregisterOption) RunUnregisterCluster() error {
 	j.Wait = j.Wait - time.Since(start)
 
 	// 2. delete the cluster object from the Karmada control plane
-	// TODO: add flag --force to implement force deletion.
 	if err = cmdutil.DeleteClusterObject(j.ControlPlaneKubeClient, j.ControlPlaneClient, j.ClusterName, j.Wait, j.DryRun, false); err != nil {
 		klog.Errorf("Failed to delete cluster object. cluster name: %s, error: %v", j.ClusterName, err)
 		return err