polish based on the comments

Author: osswangxining

pkg/controller/launcher-populator/populator.go

@@ -520,54 +520,6 @@ func (ctl *controller) createLaunchers(ctx context.Context, node corev1.Node, ke
 	return nil
 }
 
-// deleteExcessLaunchers deletes the specified number of launcher pods
-func (ctl *controller) deleteExcessLaunchers(ctx context.Context, launchers []*corev1.Pod, count int) error {
-	logger := klog.FromContext(ctx)
-
-	deletedCount := 0
-	for i := 0; i < count && i < len(launchers); i++ {
-		pod := launchers[len(launchers)-1-i]
-		isBound, requesterPodName := ctl.isLauncherBoundToServerRequestingPod(pod)
-		if isBound {
-			logger.V(5).Info("Skipping deletion of launcher pod as it is bound to a server-requesting pod",
-				"pod", pod.Name, "server-requesting pod", requesterPodName)
-			continue
-		}
-
-		if err := ctl.coreclient.Pods(pod.Namespace).Delete(ctx, pod.Name, metav1.DeleteOptions{
-			Preconditions: &metav1.Preconditions{
-				ResourceVersion: &pod.ResourceVersion,
-			},
-		}); err != nil {
-			if apierrors.IsNotFound(err) {
-				logger.Info("Launcher pod already deleted", "pod", pod.Name)
-				deletedCount++ // Count as deletion target achieved
-				continue
-			}
-			if apierrors.IsConflict(err) {
-				logger.Info("Launcher pod version conflict, skipping deletion",
-					"pod", pod.Name, "error", err)
-				continue
-			}
-			return fmt.Errorf("failed to delete launcher pod %s: %w", pod.Name, err)
-		}
-		logger.Info("Deleted launcher pod", "pod", pod.Name)
-		deletedCount++
-	}
-
-	if deletedCount < count {
-		logger.Info("Fewer launcher pods were deleted than requested due to bound pods or concurrent changes",
-			"requested", count,
-			"deleted", deletedCount,
-			"skipped", count-deletedCount)
-	} else {
-		logger.Info("Deleted unbound launcher pods",
-			"deleted", deletedCount)
-	}
-
-	return nil
-}
-
 // isLauncherBoundToServerRequestingPod checks if the launcher pod is bound to any server-requesting pod
 func (ctl *controller) isLauncherBoundToServerRequestingPod(launcherPod *corev1.Pod) (bool, string) {
 	// Check if the launcher pod has annotations indicating assignment to a server-requesting pod