From a905430d27ec7372267b1cf193f6aa6cda68adb6 Mon Sep 17 00:00:00 2001 From: richkun Date: Mon, 31 Jan 2022 04:24:22 +0800 Subject: [PATCH] embed: only log stream error with debug level (#13656) Co-authored-by: tangcong --- embed/config_logging.go | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/embed/config_logging.go b/embed/config_logging.go index e42103cb1..12dd2dd00 100644 --- a/embed/config_logging.go +++ b/embed/config_logging.go @@ -196,10 +196,14 @@ func (cfg *Config) setupLogging() error { grpcLogOnce.Do(func() { // debug true, enable info, warning, error // debug false, only discard info - var gl grpclog.LoggerV2 - gl, err = logutil.NewGRPCLoggerV2(copied) - if err == nil { - grpclog.SetLoggerV2(gl) + if cfg.LogLevel == "debug" { + var gl grpclog.LoggerV2 + gl, err = logutil.NewGRPCLoggerV2(copied) + if err == nil { + grpclog.SetLoggerV2(gl) + } + } else { + grpclog.SetLoggerV2(grpclog.NewLoggerV2(ioutil.Discard, os.Stderr, os.Stderr)) } }) return nil @@ -245,7 +249,11 @@ func (cfg *Config) setupLogging() error { c.loggerWriteSyncer = syncer grpcLogOnce.Do(func() { - grpclog.SetLoggerV2(logutil.NewGRPCLoggerV2FromZapCore(cr, syncer)) + if cfg.LogLevel == "debug" { + grpclog.SetLoggerV2(logutil.NewGRPCLoggerV2FromZapCore(cr, syncer)) + } else { + grpclog.SetLoggerV2(grpclog.NewLoggerV2(ioutil.Discard, os.Stderr, os.Stderr)) + } }) return nil }