fix comments

Signed-off-by: Wantong Jiang <[email protected]>

Author: jwtty

tools/fleet/README.md

@@ -1,11 +1,7 @@
 # kubectl-fleet
 
-A unified kubectl plugin for KubeFleet cluster management operations, providing draincluster, and uncordoncluster functionality for member clusters in a fleet,
-as well as staged update approval capability.
-
-## Description
-
-`kubectl-fleet` allows you to manage member clusters in a KubeFleet deployment by approving staged updates, draining workloads from clusters for maintenance, and uncordoning them when ready to accept workloads again.
+A kubectl plugin for KubeFleet cluster management operations, providing functionalities include draining workloads from member clusters for maintenance,
+uncordoning them when ready to accept workloads again, as well as approving staged update run stage execution.
 
 ## Installation
 
@@ -120,8 +116,7 @@ The approve command currently supports the following resource kinds:
 Drains a member cluster by performing the following actions:
 
 1. **Cordoning**: Adds a `Taint` to the `MemberCluster` resource to prevent any new resources from being propagated to the member cluster
-2. **Eviction**: Creates `Eviction` objects for all the `Placement` objects that have propagated resources to the member cluster
-3. **Resource Removal**: Waits for all evictions to complete and resources to be removed from the member cluster
+2. **Eviction**: Creates `Eviction` objects for all the `Placement` objects that have propagated resources to the member cluster and waits all evictions to complete
 
 **Note**: The `draincluster` command is a best-effort mechanism. Once the command runs successfully, you must verify that all resources propagated by `Placement` resources are removed from the member cluster. Re-running the command is safe and recommended if you notice any resources still present on the member cluster.
 

tools/fleet/cmd/approve/approve.go

@@ -48,9 +48,9 @@ func NewCmdApprove() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "approve <kind>",
 		Short: "Approve a resource",
-		Long: `Approve a resource by patching an "Approved" condition to its status.
+		Long: `Approve a resource by updating its status with an "Approved" condition.
 
-This command patches the resource with an "Approved" condition,
+This command updates the clusterApprovalRequest status with an "Approved" condition,
 allowing staged update runs to proceed to the next stage.
 
 Currently supported kinds:

tools/fleet/cmd/approve/approve_test.go

@@ -29,7 +29,6 @@ import (
 
 	"github.com/google/go-cmp/cmp"
 	"github.com/google/go-cmp/cmp/cmpopts"
-	clusterv1beta1 "github.com/kubefleet-dev/kubefleet/apis/cluster/v1beta1"
 	placementv1beta1 "github.com/kubefleet-dev/kubefleet/apis/placement/v1beta1"
 )
 
@@ -211,9 +210,6 @@ func TestRun(t *testing.T) {
 // setupScheme creates a scheme with the necessary APIs for testing.
 func setupScheme(t *testing.T) *runtime.Scheme {
 	scheme := runtime.NewScheme()
-	if err := clusterv1beta1.AddToScheme(scheme); err != nil {
-		t.Fatalf("failed to add cluster v1beta1 scheme: %v", err)
-	}
 	if err := placementv1beta1.AddToScheme(scheme); err != nil {
 		t.Fatalf("failed to add placement v1beta1 scheme: %v", err)
 	}