From 538248df0de7f255aa25c024089eb291ad5a19a0 Mon Sep 17 00:00:00 2001 From: kerthcet Date: Tue, 19 Dec 2023 19:13:30 +0800 Subject: [PATCH] More understandable error for scheduler configuration validation Signed-off-by: kerthcet --- pkg/scheduler/apis/config/validation/validation.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/scheduler/apis/config/validation/validation.go b/pkg/scheduler/apis/config/validation/validation.go index 4d6d977c177..1993ec8c933 100644 --- a/pkg/scheduler/apis/config/validation/validation.go +++ b/pkg/scheduler/apis/config/validation/validation.go @@ -273,11 +273,11 @@ func validateCommonQueueSort(path *field.Path, profiles []config.KubeSchedulerPr curr = profiles[i].Plugins.QueueSort } if !cmp.Equal(canon, curr) { - errs = append(errs, field.Invalid(path.Index(i).Child("plugins", "queueSort"), curr, "has to match for all profiles")) + errs = append(errs, field.Invalid(path.Index(i).Child("plugins", "queueSort"), curr, "queueSort must be the same for all profiles")) } for _, cfg := range profiles[i].PluginConfig { if cfg.Name == queueSortName && !cmp.Equal(queueSortArgs, cfg.Args) { - errs = append(errs, field.Invalid(path.Index(i).Child("pluginConfig", "args"), cfg.Args, "has to match for all profiles")) + errs = append(errs, field.Invalid(path.Index(i).Child("pluginConfig", "args"), cfg.Args, "queueSort must be the same for all profiles")) } } }