Scale summaries from milliseconds to seconds.

Fixes #177 Updating exporter tests for new summary unit scaling behavior Signed-off-by: Spencer Malone <malone.spencer@gmail.com>
prometheus · Jan 9, 2019 · c73a7b2 · c73a7b2
1 parent 2d35b09
commit c73a7b2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/exporter.go b/exporter.go
@@ -420,7 +420,7 @@ func (b *Exporter) handleEvent(event Event) {
 				mapping,
 			)
 			if err == nil {
-				summary.Observe(event.Value())
+				summary.Observe(event.Value() / 1000) // prometheus presumes seconds, statsd millisecond
 				b.saveLabelValues(metricName, prometheusLabels, mapping.Ttl)
 				eventStats.WithLabelValues("timer").Inc()
 			} else {

diff --git a/exporter_test.go b/exporter_test.go
@@ -216,7 +216,7 @@ mappings:
 		// event with ttl = 2s from a mapping
 		&TimerEvent{
 			metricName: "bazqux.main",
-			value:      42,
+			value:      42000,
 		},
 	}