print all scheduler names if not uniq

Signed-off-by: kangclzjc <kangz@nvidia.com>

Author: kangclzjc

operator/api/config/v1alpha1/defaults.go

@@ -73,8 +73,7 @@ func SetDefaults_OperatorConfiguration(operatorConfig *OperatorConfiguration) {
 // Principle: respect all user-explicit values first.
 //
 //  1. If user did not include kube in profiles, add kube. Whether kube is default: only if no other profile is default, set kube as default.
-//  2. If user did not set any default (no profile has default: true), set kube as default (only when we added kube above — do not override user's explicit kube default: false).
-//     Otherwise (one non-kube default, or two+ defaults) we do not change; validation will reject invalid cases.
+//  2. If user did not set any default (no profile has default: true), set kube as default. Validation will reject invalid cases.
 func SetDefaults_SchedulerConfiguration(cfg *SchedulerConfiguration) {
 	if len(cfg.Profiles) == 0 {
 		cfg.Profiles = []SchedulerProfile{
@@ -110,8 +109,7 @@ func SetDefaults_SchedulerConfiguration(cfg *SchedulerConfiguration) {
 		return
 	}
 	// 2. No default → set kube as default.
-	// Only set kube as default if user did not explicitly set kube as default.
-	if defaultCount == 0 && !hasKube {
+	if defaultCount == 0 {
 		cfg.Profiles[kubeIdx].Default = true
 	}
 }

operator/internal/webhook/admission/pcs/validation/podcliqueset.go

@@ -150,26 +150,22 @@ func (v *pcsValidator) validatePodCliqueTemplates(fldPath *field.Path) ([]string
 			if def := schedulerbackend.GetDefault(); def != nil {
 				return def.Name()
 			}
-			return "default-scheduler" // fallback when backend not initialized (e.g. in tests)
+			return "default-scheduler"
 		}
 		return item
 	}))
 	if len(uniqueSchedulerNames) > 1 {
-		allErrs = append(allErrs, field.Invalid(fldPath.Child("spec").Child("podSpec").Child("schedulerName"), uniqueSchedulerNames[0], "the schedulerName for all pods have to be the same"))
+		allErrs = append(allErrs, field.Invalid(fldPath.Child("spec").Child("podSpec").Child("schedulerName"), strings.Join(uniqueSchedulerNames, ", "), "the schedulerName for all pods have to be the same"))
 	}
 
 	// Validate that the scheduler name is enabled (present in OperatorConfiguration profiles or default)
 	pcsSchedulerName := ""
 	if len(uniqueSchedulerNames) > 0 && uniqueSchedulerNames[0] != "" {
 		pcsSchedulerName = uniqueSchedulerNames[0]
 	}
-	if pcsSchedulerName == "" {
-		if def := schedulerbackend.GetDefault(); def != nil {
-			pcsSchedulerName = def.Name()
-		}
-	}
+
 	// default-scheduler is the pod-facing name for kube-scheduler and is always accepted when kube backend is enabled
-	if pcsSchedulerName != "" && pcsSchedulerName != "default-scheduler" && schedulerbackend.Get(pcsSchedulerName) == nil {
+	if pcsSchedulerName != "default-scheduler" && schedulerbackend.Get(pcsSchedulerName) == nil {
 		allErrs = append(allErrs, field.Invalid(
 			fldPath.Child("spec").Child("podSpec").Child("schedulerName"),
 			pcsSchedulerName,