Merge pull request #1429 from zhaxzhax/fix-#1428

lowzj · lowzj · commit d34775a2cda0 · 2020-07-16T11:40:09.000+08:00
bugfix: fix missing error message 'Unable to autodetect advertiser ip
diff --git a/cmd/supernode/app/root.go b/cmd/supernode/app/root.go
@@ -375,8 +375,8 @@ func setAdvertiseIP(cfg *config.Config) error {
 		return errors.Wrapf(errortypes.ErrSystemError, "failed to get ip list: %v", err)
 	}
 	if len(ipList) == 0 {
-		logrus.Debugf("get empty system's unicast interface addresses")
-		return nil
+		logrus.Errorf("get empty system's unicast interface addresses")
+		return errors.Wrapf(errortypes.ErrSystemError, "Unable to autodetect advertiser ip, please set it via --advertise-ip")
 	}
 
 	cfg.AdvertiseIP = ipList[0]

Original file line number	Diff line number	Diff line change
`@@ -375,8 +375,8 @@ func setAdvertiseIP(cfg *config.Config) error {`
`375`	`375`	`return errors.Wrapf(errortypes.ErrSystemError, "failed to get ip list: %v", err)`
`376`	`376`	`}`
`377`	`377`	`if len(ipList) == 0 {`
`378`		`- logrus.Debugf("get empty system's unicast interface addresses")`
`379`		`- return nil`
	`378`	`+ logrus.Errorf("get empty system's unicast interface addresses")`
	`379`	`+ return errors.Wrapf(errortypes.ErrSystemError, "Unable to autodetect advertiser ip, please set it via --advertise-ip")`
`380`	`380`	`}`
`381`	`381`
`382`	`382`	`cfg.AdvertiseIP = ipList[0]`