Merge pull request #346 from mnotti/lb-strategy-logging-error

fix lb strategy logging error
bojand · Apr 10, 2022 · 310c25f · 310c25f
2 parents b97c010 + 4db2f0e
commit 310c25f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/cmd/ghz/main.go b/cmd/ghz/main.go
@@ -315,8 +315,8 @@ func main() {
 		options = append(options, runner.WithLogger(logger))
 	}
 
-	if isLBStrategySet && cfg.Host != "" && !strings.HasPrefix(cfg.Host, "dns:///") {
-		logger.Warn("Load balancing strategy set without using DNS (dns:///) scheme. ", "Strategy: ", cfg.LBStrategy, " Host: ", cfg.Host)
+	if logger != nil && isLBStrategySet && cfg.Host != "" && !strings.HasPrefix(cfg.Host, "dns:///") {
+		logger.Warnw("Load balancing strategy set without using DNS (dns:///) scheme", "strategy", cfg.LBStrategy, "host", cfg.Host)
 	}
 
 	if logger != nil {