Skip to content
This repository has been archived by the owner on Dec 22, 2023. It is now read-only.

Commit

Permalink
Use prometheus client from rubygems
Browse files Browse the repository at this point in the history
The enormous rewrite from prometheus/client_ruby#95 has been merged,
and a prerelease version of it is now available on rubygems (as of
prometheus/client_ruby#124).  We should use the rubygems version
rather than the git version; when 0.10.0 is properly released we
should use that.
  • Loading branch information
philandstuff committed May 23, 2019
1 parent 04283c2 commit d51836e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 11 deletions.
6 changes: 3 additions & 3 deletions Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,9 @@ gem 'statsd-ruby', '~> 1.3.0'
# Use prometheus-client to expose metrics to prometheus
#
# prometheus/client_ruby had a massive rewrite in
# https://github.com/prometheus/client_ruby/pull/95 - it has landed in
# master but not been released yet
gem 'prometheus-client', git: 'https://github.com/prometheus/client_ruby.git'
# https://github.com/prometheus/client_ruby/pull/95
# This is the prerelease version of that work
gem 'prometheus-client', '~> 0.10.0.pre.alpha.1'

# Use sentry-raven for sending logs to Sentry via the raven protocol
gem 'sentry-raven'
Expand Down
11 changes: 3 additions & 8 deletions Gemfile.lock
Original file line number Diff line number Diff line change
@@ -1,10 +1,3 @@
GIT
remote: https://github.com/prometheus/client_ruby.git
revision: 460c2bbe70fb4434adaa2bdbeb7ff09e84abd0de
specs:
prometheus-client (0.9.0)
concurrent-ruby

GEM
remote: https://rubygems.org/
specs:
Expand Down Expand Up @@ -197,6 +190,8 @@ GEM
insist
mustache (= 0.99.8)
stud
prometheus-client (0.10.0.pre.alpha.1)
concurrent-ruby
psych (3.1.0)
public_suffix (3.0.3)
puma (3.12.1)
Expand Down Expand Up @@ -372,7 +367,7 @@ DEPENDENCIES
mini_racer
multi_json
pkgr (~> 1.5.1)
prometheus-client!
prometheus-client (~> 0.10.0.pre.alpha.1)
puma
rack-handlers
rack-test
Expand Down

0 comments on commit d51836e

Please sign in to comment.