Skip to content

Commit

Permalink
Merge pull request #13 from bcandrea/master
Browse files Browse the repository at this point in the history
Use `_count` instead of `_total` when exposing summaries
  • Loading branch information
grobie committed May 18, 2015
2 parents daaa680 + 481fc8b commit cba7b4b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion lib/prometheus/client/formats/text.rb
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ def self.summary(name, set, value)

l = labels(set)
yield metric("#{name}_sum", l, value.sum)
yield metric("#{name}_total", l, value.total)
yield metric("#{name}_count", l, value.total)
end

def self.metric(name, labels, value)
Expand Down
2 changes: 1 addition & 1 deletion spec/prometheus/client/formats/text_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@
qux{for="sake",code="1",quantile="0.9"} 8.32
qux{for="sake",code="1",quantile="0.99"} 15.3
qux_sum{for="sake",code="1"} 1243.21
qux_total{for="sake",code="1"} 93
qux_count{for="sake",code="1"} 93
TEXT
end
end
Expand Down

0 comments on commit cba7b4b

Please sign in to comment.