From 9ea525a23be0df6a0f5821f51c2272add021160f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Janek=20=C5=81ukaszewicz?= <janluk@google.com>
Date: Tue, 23 Apr 2019 16:35:21 +0200
Subject: [PATCH] Use klog.Exit instead of Fatal when stacktraces are not
 needed.

---
 clusterloader2/cmd/clusterloader.go | 21 ++++++++++-----------
 1 file changed, 10 insertions(+), 11 deletions(-)

diff --git a/clusterloader2/cmd/clusterloader.go b/clusterloader2/cmd/clusterloader.go
index 240fb8613..1368015a7 100644
--- a/clusterloader2/cmd/clusterloader.go
+++ b/clusterloader2/cmd/clusterloader.go
@@ -177,23 +177,23 @@ func main() {
 	defer klog.Flush()
 	initFlags()
 	if err := flags.Parse(); err != nil {
-		klog.Fatalf("Flag parse failed: %v", err)
+		klog.Exitf("Flag parse failed: %v", err)
 	}
 	if errList := validateFlags(); !errList.IsEmpty() {
-		klog.Fatalf("Parsing flags error: %v", errList.String())
+		klog.Exitf("Parsing flags error: %v", errList.String())
 	}
 
 	mclient, err := framework.NewMultiClientSet(clusterLoaderConfig.ClusterConfig.KubeConfigPath, 1)
 	if err != nil {
-		klog.Fatalf("Client creation error: %v", err)
+		klog.Exitf("Client creation error: %v", err)
 	}
 
 	if err = completeConfig(mclient); err != nil {
-		klog.Fatalf("Config completing error: %v", err)
+		klog.Exitf("Config completing error: %v", err)
 	}
 
 	if err = createReportDir(); err != nil {
-		klog.Fatalf("Cannot create report directory: %v", err)
+		klog.Exitf("Cannot create report directory: %v", err)
 	}
 
 	if err = util.LogClusterNodes(mclient.GetClient()); err != nil {
@@ -201,7 +201,7 @@ func main() {
 	}
 
 	if err = verifyCluster(mclient.GetClient()); err != nil {
-		klog.Fatalf("Cluster verification error: %v", err)
+		klog.Exitf("Cluster verification error: %v", err)
 	}
 
 	f, err := framework.NewFramework(
@@ -209,18 +209,18 @@ func main() {
 		getClientsNumber(clusterLoaderConfig.ClusterConfig.Nodes),
 	)
 	if err != nil {
-		klog.Fatalf("Framework creation error: %v", err)
+		klog.Exitf("Framework creation error: %v", err)
 	}
 
 	var prometheusController *prometheus.PrometheusController
 	var prometheusFramework *framework.Framework
 	if clusterLoaderConfig.EnablePrometheusServer {
 		if prometheusController, err = prometheus.NewPrometheusController(&clusterLoaderConfig); err != nil {
-			klog.Fatalf("Error while creating Prometheus Controller: %v", err)
+			klog.Exitf("Error while creating Prometheus Controller: %v", err)
 		}
 		prometheusFramework = prometheusController.GetFramework()
 		if err := prometheusController.SetUpPrometheusStack(); err != nil {
-			klog.Fatalf("Error while setting up prometheus stack: %v", err)
+			klog.Exitf("Error while setting up prometheus stack: %v", err)
 		}
 	}
 
@@ -260,7 +260,6 @@ func main() {
 		}
 	}
 	if suiteSummary.NumberOfFailedSpecs > 0 {
-		klog.Errorf("%d tests have failed!", suiteSummary.NumberOfFailedSpecs)
-		os.Exit(1)
+		klog.Exitf("%d tests have failed!", suiteSummary.NumberOfFailedSpecs)
 	}
 }