Merge pull request #63962 from hzxuzhonghu/fix-scheduler-options

Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. fix little bug in kube-scheduler options fix some little bugs in kube-scheduler options. **Release note**: ```release-note NONE ```

Merge pull request #63962 from hzxuzhonghu/fix-scheduler-options
e04ccf55 · Kubernetes Submit Queue · GitHub · 7909712c · be2a234a · e04ccf55
Unverified Commit e04ccf55 authored May 19, 2018 by Kubernetes Submit Queue Committed by GitHub May 19, 2018
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

deprecated.go cmd/kube-scheduler/app/options/deprecated.go +1 -1

insecure_serving.go cmd/kube-scheduler/app/options/insecure_serving.go +1 -1

No files found.
--- a/cmd/kube-scheduler/app/options/deprecated.go
+++ b/cmd/kube-scheduler/app/options/deprecated.go
@@ -72,7 +72,7 @@ func (o *DeprecatedOptions) AddFlags(fs *pflag.FlagSet, cfg *componentconfig.Kub
 // Validate validates the deprecated scheduler options.
 func (o *DeprecatedOptions) Validate() []error {
-	if o != nil {
+	if o == nil {
 		return nil
 	}
 	return nil

--- a/cmd/kube-scheduler/app/options/insecure_serving.go
+++ b/cmd/kube-scheduler/app/options/insecure_serving.go
@@ -153,7 +153,7 @@ func (o *CombinedInsecureServingOptions) Validate() []error {
 	errors := []error{}
 	if o.BindPort <= 0 || o.BindPort > 32767 {
-		errors = append(errors, fmt.Errorf("--insecure-port %v must be between 0 and 32767, inclusive. 0 for turning off insecure (HTTP) port", o.BindPort))
+		errors = append(errors, fmt.Errorf("--port %v must be between 0 and 32767, inclusive. 0 for turning off insecure (HTTP) port", o.BindPort))
 	}
 	if len(o.BindAddress) > 0 && net.ParseIP(o.BindAddress) == nil {