tink-worker should initialize logger level early

tinkerbell · Aug 15, 2020 · f9500b2 · f9500b2
1 parent 211a0e5
commit f9500b2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/cmd/tink-worker/action.go b/cmd/tink-worker/action.go
@@ -276,6 +276,12 @@ func initializeDockerClient() (*client.Client, error) {
 	if err != nil {
 		return nil, errors.Wrap(err, "DOCKER CLIENT")
 	}
+
+	logger.SetFormatter(&logrus.JSONFormatter{})
+	return c, nil
+}
+
+func initializeLogger() {
 	level := os.Getenv("WORKER_LOG_LEVEL")
 	if level != "" {
 		switch strings.ToLower(level) {
@@ -301,6 +307,4 @@ func initializeDockerClient() (*client.Client, error) {
 		logger.SetLevel(logrus.InfoLevel)
 		logger.Warningln("Variable WORKER_LOG_LEVEL is not set. Default is Info")
 	}
-	logger.SetFormatter(&logrus.JSONFormatter{})
-	return c, nil
 }
diff --git a/cmd/tink-worker/main.go b/cmd/tink-worker/main.go
@@ -30,6 +30,7 @@ var (
 )
 
 func main() {
+	initializeLogger()
 	logger.Debug("Starting version " + version)
 	setupRetry()
 	if setupErr := client.Setup(); setupErr != nil {