Do not print log level on CLI (#2638)

if it's not in debug log level

closes https://github.com/woodpecker-ci/woodpecker/issues/1764

cli/common/zerologger.go

@@ -21,6 +21,6 @@ import (
 )
 
 func SetupGlobalLogger(c *cli.Context) error {
-	common.SetupGlobalLogger(c)
+	common.SetupGlobalLogger(c, false)
 	return nil
 }

cmd/agent/agent.go

@@ -49,7 +49,7 @@ import (
 )
 
 func run(c *cli.Context) error {
-	common.SetupGlobalLogger(c)
+	common.SetupGlobalLogger(c, true)
 
 	agentConfigPath := c.String("agent-config")
 	hostname := c.String("hostname")

cmd/cli/app.go

@@ -41,6 +41,7 @@ func newApp() *cli.App {
 	app.Usage = "command line utility"
 	app.EnableBashCompletion = true
 	app.Flags = common.GlobalFlags
+	app.Before = common.SetupGlobalLogger
 	app.Commands = []*cli.Command{
 		pipeline.Command,
 		log.Command,
@@ -56,9 +57,5 @@ func newApp() *cli.App {
 		cron.Command,
 	}
 
-	for _, command := range app.Commands {
-		command.Before = common.SetupGlobalLogger
-	}
-
 	return app
 }

cmd/common/logger.go

@@ -51,7 +51,7 @@ var GlobalLoggerFlags = []cli.Flag{
 	},
 }
 
-func SetupGlobalLogger(c *cli.Context) {
+func SetupGlobalLogger(c *cli.Context, printLvl bool) {
 	logLevel := c.String("log-level")
 	pretty := c.Bool("pretty")
 	noColor := c.Bool("nocolor")
@@ -96,5 +96,7 @@ func SetupGlobalLogger(c *cli.Context) {
 		log.Logger = log.With().Caller().Logger()
 	}
 
-	log.Info().Msgf("LogLevel = %s", zerolog.GlobalLevel().String())
+	if printLvl {
+		log.Info().Msgf("LogLevel = %s", zerolog.GlobalLevel().String())
+	}
 }

cmd/server/server.go

@@ -55,7 +55,7 @@ import (
 )
 
 func run(c *cli.Context) error {
-	common.SetupGlobalLogger(c)
+	common.SetupGlobalLogger(c, true)
 
 	// set gin mode based on log level
 	if zerolog.GlobalLevel() > zerolog.DebugLevel {