From 183ead4255b450b3e166952a6923b2609a67af5e Mon Sep 17 00:00:00 2001 From: Grant Rodgers Date: Wed, 2 Oct 2019 16:55:07 -0700 Subject: [PATCH] Use the correct zap sugared logger syntax Fatalw needs either a zap.Field or key, value pairs in the variadic args. --- cmd/autoscaler/main.go | 2 +- cmd/webhook/main.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/cmd/autoscaler/main.go b/cmd/autoscaler/main.go index ee739bd07a2f..2ae51a222474 100644 --- a/cmd/autoscaler/main.go +++ b/cmd/autoscaler/main.go @@ -143,7 +143,7 @@ func main() { // Start all of the informers and wait for them to sync. if err := controller.StartInformers(ctx.Done(), informers...); err != nil { - logger.Fatalw("Failed to start informers", err) + logger.Fatalw("Failed to start informers", zap.Error(err)) } go controller.StartAll(ctx.Done(), controllers...) diff --git a/cmd/webhook/main.go b/cmd/webhook/main.go index 49c121e70bbd..0ac15e210a33 100644 --- a/cmd/webhook/main.go +++ b/cmd/webhook/main.go @@ -106,7 +106,7 @@ func main() { logger = logger.With(zap.String(logkey.ControllerType, component)) if err := version.CheckMinimumVersion(kubeClient.Discovery()); err != nil { - logger.Fatalw("Version check failed", err) + logger.Fatalw("Version check failed", zap.Error(err)) } profilingHandler := profiling.NewHandler(logger, false)