From ab1d2e474a8933d30f844dd36335427700092d5b Mon Sep 17 00:00:00 2001
From: Priya Wadhwa <priyawadhwa@google.com>
Date: Fri, 4 Dec 2020 11:36:16 -0800
Subject: [PATCH] don't fail if one container runtime fails

---
 hack/metrics/metrics.go | 45 ++++++++++++++++++++++++-----------------
 1 file changed, 26 insertions(+), 19 deletions(-)

diff --git a/hack/metrics/metrics.go b/hack/metrics/metrics.go
index b69520a32cd0..e3e85078e0e7 100644
--- a/hack/metrics/metrics.go
+++ b/hack/metrics/metrics.go
@@ -82,28 +82,35 @@ func execute() error {
 		return errors.Wrap(err, "registering view")
 	}
 	for _, cr := range []string{"docker", "containerd", "crio"} {
-		sd, err := getExporter(projectID, cr)
-		if err != nil {
-			return errors.Wrap(err, "getting stackdriver exporter")
+		if err := exportMinikubeStart(ctx, projectID, cr); err != nil {
+			log.Printf("error exporting minikube start data for runtime %v: %v", cr, err)
 		}
-		// Register it as a trace exporter
-		trace.RegisterExporter(sd)
+	}
+	return nil
+}
 
-		if err := sd.StartMetricsExporter(); err != nil {
-			return errors.Wrap(err, "starting metric exporter")
-		}
-		// track minikube start time and record it to metrics collector
-		st, err := minikubeStartTime(ctx, projectID, tmpFile, cr)
-		if err != nil {
-			return errors.Wrap(err, "collecting start time")
-		}
-		fmt.Printf("Latency: %f\n", st)
-		stats.Record(ctx, latencyS.M(st))
-		time.Sleep(30 * time.Second)
-		sd.Flush()
-		sd.StopMetricsExporter()
-		trace.UnregisterExporter(sd)
+func exportMinikubeStart(ctx context.Context, projectID, containerRuntime string) error {
+	sd, err := getExporter(projectID, containerRuntime)
+	if err != nil {
+		return errors.Wrap(err, "getting stackdriver exporter")
+	}
+	// Register it as a trace exporter
+	trace.RegisterExporter(sd)
+
+	if err := sd.StartMetricsExporter(); err != nil {
+		return errors.Wrap(err, "starting metric exporter")
+	}
+	// track minikube start time and record it to metrics collector
+	st, err := minikubeStartTime(ctx, projectID, tmpFile, containerRuntime)
+	if err != nil {
+		return errors.Wrap(err, "collecting start time")
 	}
+	fmt.Printf("Latency: %f\n", st)
+	stats.Record(ctx, latencyS.M(st))
+	time.Sleep(30 * time.Second)
+	sd.Flush()
+	sd.StopMetricsExporter()
+	trace.UnregisterExporter(sd)
 	return nil
 }