Treat failure to collect metric as fatal

Chris Pick · Chris Pick · commit c49aacebb23f · 2020-05-12T12:09:03.000-04:00
Exit on any failure to collect metrics.  This will give
`stackdriver_exporter` a chance to start over and perhaps recover from
transient errors.  For example: if it's Google credentials have expired
and have been replaced, it can restart to try to load up new ones.

Signed-off-by: Chris Pick &lt;chris@chrispick.com&gt;
diff --git a/collectors/monitoring_collector.go b/collectors/monitoring_collector.go
@@ -187,7 +187,7 @@ func (c *MonitoringCollector) Collect(ch chan<- prometheus.Metric) {
 	if err := c.reportMonitoringMetrics(ch); err != nil {
 		errorMetric = float64(1)
 		c.scrapeErrorsTotalMetric.Inc()
-		log.Errorf("Error while getting Google Stackdriver Monitoring metrics: %s", err)
+		log.Fatalf("Error while getting Google Stackdriver Monitoring metrics: %s", err)
 	}
 	c.scrapeErrorsTotalMetric.Collect(ch)
 

Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,7 @@ func (c *MonitoringCollector) Collect(ch chan<- prometheus.Metric) {`
`187`	`187`	`if err := c.reportMonitoringMetrics(ch); err != nil {`
`188`	`188`	`errorMetric = float64(1)`
`189`	`189`	`c.scrapeErrorsTotalMetric.Inc()`
`190`		`- log.Errorf("Error while getting Google Stackdriver Monitoring metrics: %s", err)`
	`190`	`+ log.Fatalf("Error while getting Google Stackdriver Monitoring metrics: %s", err)`
`191`	`191`	`}`
`192`	`192`	`c.scrapeErrorsTotalMetric.Collect(ch)`
`193`	`193`