diff --git a/embed/config.go b/embed/config.go index ed647a0637b..2772f518ee7 100644 --- a/embed/config.go +++ b/embed/config.go @@ -405,8 +405,7 @@ func checkBindURLs(urls []url.URL) error { continue } if net.ParseIP(host) == nil { - err := fmt.Errorf("expected IP in URL for binding (%s)", url.String()) - plog.Warning(err) + return fmt.Errorf("expected IP in URL for binding (%s)", url.String()) } } return nil diff --git a/integration/embed_test.go b/integration/embed_test.go index defa611ba41..71e41cc10da 100644 --- a/integration/embed_test.go +++ b/integration/embed_test.go @@ -40,6 +40,8 @@ func TestEmbedEtcd(t *testing.T) { {wpeers: 1, wclients: 1}, {wpeers: 2, wclients: 1}, {wpeers: 1, wclients: 2}, + {werr: "expected IP"}, + {werr: "expected IP"}, } urls := newEmbedURLs(10) @@ -58,6 +60,10 @@ func TestEmbedEtcd(t *testing.T) { setupEmbedCfg(&tests[5].cfg, []url.URL{urls[4]}, []url.URL{urls[5], urls[6]}) setupEmbedCfg(&tests[6].cfg, []url.URL{urls[7], urls[8]}, []url.URL{urls[9]}) + dnsURL, _ := url.Parse("http://whatever.test:12345") + tests[7].cfg.LCUrls = []url.URL{*dnsURL} + tests[8].cfg.LPUrls = []url.URL{*dnsURL} + dir := path.Join(os.TempDir(), fmt.Sprintf("embed-etcd")) os.RemoveAll(dir) defer os.RemoveAll(dir)