logging: Joining the cluster is an info, not an error

See olivere#235

CONTRIBUTORS

@@ -27,6 +27,7 @@ Mara Kim [@autochthe](https://github.com/autochthe)
 Medhi Bechina [@mdzor](https://github.com/mdzor)
 Nicholas Wolff [@nwolff](https://github.com/nwolff)
 Orne Brocaar [@brocaar](https://github.com/brocaar)
+Ryan Schmukler [@rschmukler](https://github.com/rschmukler)
 Sacheendra talluri [@sacheendra](https://github.com/sacheendra)
 Sean DuBois [@Sean-Der](https://github.com/Sean-Der)
 Shalin LK [@shalinlk](https://github.com/shalinlk)

client.go

@@ -849,7 +849,7 @@ func (c *Client) updateConns(conns []*conn) {
 		}
 		if !found {
 			// New connection didn't exist, so add it to our list of new conns.
-			c.errorf("elastic: %s joined the cluster", conn.URL())
+			c.infof("elastic: %s joined the cluster", conn.URL())
 			newConns = append(newConns, conn)
 		}
 	}

client_test.go

@@ -683,7 +683,7 @@ func TestPerformRequestWithLogger(t *testing.T) {
 	var w bytes.Buffer
 	out := log.New(&w, "LOGGER ", log.LstdFlags)
 
-	client, err := NewClient(SetInfoLog(out))
+	client, err := NewClient(SetInfoLog(out), SetSniff(false))
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -722,7 +722,7 @@ func TestPerformRequestWithLoggerAndTracer(t *testing.T) {
 	var tw bytes.Buffer
 	tout := log.New(&tw, "TRACER ", log.LstdFlags)
 
-	client, err := NewClient(SetInfoLog(lout), SetTraceLog(tout))
+	client, err := NewClient(SetInfoLog(lout), SetTraceLog(tout), SetSniff(false))
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -765,7 +765,7 @@ func (l *customLogger) Printf(format string, v ...interface{}) {
 func TestPerformRequestWithCustomLogger(t *testing.T) {
 	logger := &customLogger{}
 
-	client, err := NewClient(SetInfoLog(logger))
+	client, err := NewClient(SetInfoLog(logger), SetSniff(false))
 	if err != nil {
 		t.Fatal(err)
 	}