Merge pull request #20 from bytheway/bucket-sum

frodenas · web-flow · commit b6e7a8c01bdd · 2018-02-24T11:33:27.000+01:00
Stackdriver doesn't provide a sum, but it does provide a mean and a count.
diff --git a/collectors/monitoring_collector.go b/collectors/monitoring_collector.go
@@ -302,7 +302,7 @@ func (c *MonitoringCollector) reportTimeSeriesMetrics(
 				ch <- prometheus.MustNewConstHistogram(
 					metricDesc,
 					uint64(dist.Count),
-					0, // Stackdriver does not provide the sum
+					dist.Mean*float64(dist.Count), // Stackdriver does not provide the sum, but we can fake it
 					buckets,
 					labelValues...,
 				)

Original file line number	Diff line number	Diff line change
`@@ -302,7 +302,7 @@ func (c *MonitoringCollector) reportTimeSeriesMetrics(`
`302`	`302`	`ch <- prometheus.MustNewConstHistogram(`
`303`	`303`	`metricDesc,`
`304`	`304`	`uint64(dist.Count),`
`305`		`- 0, // Stackdriver does not provide the sum`
	`305`	`+ dist.Mean*float64(dist.Count), // Stackdriver does not provide the sum, but we can fake it`
`306`	`306`	`buckets,`
`307`	`307`	`labelValues...,`
`308`	`308`	`)`