diff --git a/cmd/descheduler/app/options/options.go b/cmd/descheduler/app/options/options.go index a04c298ef..64d831c0e 100644 --- a/cmd/descheduler/app/options/options.go +++ b/cmd/descheduler/app/options/options.go @@ -62,9 +62,9 @@ func NewDeschedulerServer() (*DeschedulerServer, error) { } // Validation checks for DeschedulerServer. -func (s *DeschedulerServer) Validate() error { +func (s *DeschedulerServer) ValidateAndApply() error { var errs []error - errs = append(errs, s.Logs.Validate()...) + errs = append(errs, s.Logs.ValidateAndApply()) return utilerrors.NewAggregate(errs) } diff --git a/cmd/descheduler/app/server.go b/cmd/descheduler/app/server.go index 1c8b600e1..c34dc2483 100644 --- a/cmd/descheduler/app/server.go +++ b/cmd/descheduler/app/server.go @@ -49,7 +49,6 @@ func NewDeschedulerCommand(out io.Writer) *cobra.Command { Long: `The descheduler evicts pods which may be bound to less desired nodes`, Run: func(cmd *cobra.Command, args []string) { s.Logs.Config.Format = s.Logging.Format - s.Logs.Apply() // LoopbackClientConfig is a config for a privileged loopback connection var LoopbackClientConfig *restclient.Config @@ -59,7 +58,7 @@ func NewDeschedulerCommand(out io.Writer) *cobra.Command { return } - if err := s.Validate(); err != nil { + if err := s.ValidateAndApply(); err != nil { klog.ErrorS(err, "failed to validate server configuration") return }