Discard grpc logs by default

mlaventure · mlaventure · commit c9064df2cfd2 · 2017-06-14T11:58:13.000-07:00
Signed-off-by: Kenfe-Mickael Laventure &lt;mickael.laventure@gmail.com&gt;
diff --git a/linux/shim.go b/linux/shim.go
@@ -5,20 +5,17 @@ package linux
 import (
 	"context"
 	"fmt"
-	"io/ioutil"
-	"log"
 	"net"
 	"os/exec"
 	"path/filepath"
 	"syscall"
 	"time"
 
 	"google.golang.org/grpc"
-	"google.golang.org/grpc/grpclog"
 
 	"github.com/containerd/containerd/api/services/shim"
 	localShim "github.com/containerd/containerd/linux/shim"
-	containerdlog "github.com/containerd/containerd/log"
+	"github.com/containerd/containerd/log"
 	"github.com/containerd/containerd/reaper"
 	"github.com/containerd/containerd/sys"
 	"github.com/pkg/errors"
@@ -58,7 +55,7 @@ func newShim(ctx context.Context, shimName string, path, namespace string, remot
 			cmd.Process.Kill()
 			reaper.Default.Wait(cmd)
 		} else {
-			containerdlog.G(ctx).WithField("socket", socket).Infof("new shim started")
+			log.G(ctx).WithField("socket", socket).Infof("new shim started")
 		}
 	}()
 	if err = sys.SetOOMScore(cmd.Process.Pid, sys.OOMScoreMaxKillable); err != nil {
@@ -77,7 +74,6 @@ func loadShim(path, namespace string, remote bool) (shim.ShimClient, error) {
 
 func connectShim(socket string) (shim.ShimClient, error) {
 	// reset the logger for grpc to log to dev/null so that it does not mess with our stdio
-	grpclog.SetLogger(log.New(ioutil.Discard, "", log.LstdFlags))
 	dialOpts := []grpc.DialOption{grpc.WithInsecure(), grpc.WithTimeout(100 * time.Second)}
 	dialOpts = append(dialOpts,
 		grpc.WithDialer(func(addr string, timeout time.Duration) (net.Conn, error) {
diff --git a/log/grpc.go b/log/grpc.go
@@ -1,14 +1,12 @@
 package log
 
 import (
-	"context"
+	"io/ioutil"
+	"log"
 
 	"google.golang.org/grpc/grpclog"
 )
 
 func init() {
-	ctx := WithModule(context.Background(), "grpc")
-
-	// completely replace the grpc logger with the logrus logger.
-	grpclog.SetLogger(G(ctx))
+	grpclog.SetLogger(log.New(ioutil.Discard, "", log.LstdFlags))
 }
