librato-rack 0.6.0 → 1.0.0.beta1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d8313a2999f2b563b827f3a5de57a141499b1d68
-  data.tar.gz: 1ba11001c43ed069bef6439ab86f8507790f57d5
+  metadata.gz: 925119ebbdc1da99d8586de3f7c98b0a02b637af
+  data.tar.gz: 2c584413df2ea05a8fab05eb9e5c5116ffd15dbd
 SHA512:
-  metadata.gz: 69e0b07c2f409f4a87150c2bb10a99dca3dabede720b82c07a7bd29575efe675a92254814218f70fb2f81399d21ca3163345ce3acb6638eb1039aad426b21884
-  data.tar.gz: 91c50a3833f4b99f0ad8deac4528f8549153d19f607402eeb8ebb90d4fa631653dcb376489e73e1dcebf9c74a29928a539504cf029f8456739f1691dbc593d5e
+  metadata.gz: 8871cf91b56d26f7ed25d54fb6f0e93e3f8b9f012bc34b053bd4b7957601b7086c24c968e941b1791207413d4ff92f8f05684468234d5e9072ee3cdd2c72a5e6
+  data.tar.gz: 95712a3eb73c604413ad33bc9a2f77cdb49de8e75ec58bfb80a5728d948dc38394362a23144b9e38f322485e0156dfefe738ebca7c9c6f625942a4394c4d0205

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+### Version 1.0.0.beta1
+* Add support for configurable metric suites
+
 ### Version 0.6.0
 * Add support for proxy configuration
 

data/README.md

@@ -83,6 +83,28 @@ NOTE: if Heroku idles your application no measurements will be sent until it rec
 
 When using in an evented context set LIBRATO_EVENT_MODE to `'eventmachine'` if using [EventMachine](https://github.com/eventmachine/eventmachine) or `'synchrony'` if using [EM Synchrony](https://github.com/igrigorik/em-synchrony) and/or [Rack::FiberPool](https://github.com/alebsack/rack-fiber_pool). We're interested in maturing this support, so please let us know if you have any issues.
 
+##### Metric Suites
+
+The metrics recorded by `librato-rack` are organized into named metric suites that can be selecitvely enabled/disabled:
+
+* `rack`: The `rack.request.total`, `rack.request.time`, `rack.request.slow`, and `rack.request.queue.time` metrics
+* `rack_status`: All of the `rack.request.status` metrics
+* `rack_method`: All of the `rack.request.method` metrics
+
+All three of the metric suites listed above are enabled by default.
+
+The metric suites can be configured via either the `LIBRATO_SUITES` environment variable or the `suites` attributes on the `Librato::Rack::Configuration` object.
+
+    LIBRATO_SUITES="rack,rack_method"  # use ONLY the rack and rack_method suites
+    LIBRATO_SUITES="+foo,+bar"         # + prefix indicates that you want the default suites plus foo and bar
+    LIBRATO_SUITES="-rack_status"      # - prefix indicates that you want the default suites removing rack_status
+    LIBRATO_SUITES="+foo,-rack_status" # Use all default suites except for rack_status while also adding foo
+    LIBRATO_SUITES="all"               # Enable all suites
+    LIBRATO_SUITES="none"              # Disable all suites
+    LIBRATO_SUITES=""                  # Use only the default suites (same as if env var is absent)
+
+Note that you should EITHER specify an explict list of suites to enable OR add/subtract individual suites from the default list (using the +/- prefixes). If you try to mix these two forms a `Librato::Rack::InvalidSuiteConfiguration` error will be raised.
+
 ## Custom Measurements
 
 Tracking anything that interests you is easy with Metrics. There are four primary helpers available:

data/lib/librato/rack.rb

@@ -37,6 +37,30 @@ module Librato
   #   run MyApp
   #
   class Rack
+    RECORD_RACK_BODY = <<-'EOS'
+      group.increment 'total'
+      group.timing    'time', duration
+      group.increment 'slow' if duration > 200.0
+    EOS
+
+    RECORD_RACK_STATUS_BODY = <<-'EOS'
+      group.group 'status' do |s|
+        s.increment status
+        s.increment "#{status.to_s[0]}xx"
+
+        s.timing "#{status}.time", duration
+        s.timing "#{status.to_s[0]}xx.time", duration
+      end
+    EOS
+
+    RECORD_RACK_METHOD_BODY = <<-'EOS'
+      group.group 'method' do |m|
+        http_method.downcase!
+        m.increment http_method
+        m.timing "#{http_method}.time", duration
+      end
+    EOS
+
     attr_reader :config, :tracker
 
     def initialize(app, options={})
@@ -54,15 +78,18 @@ module Librato
       if old_style
         @tracker.deprecate 'middleware setup no longer takes a single argument, use `use Librato::Rack :config => config` instead.'
       end
+
+      build_record_request_metrics_method
+      build_record_header_metrics_method
+      build_record_exception_method
     end
 
     def call(env)
       check_log_output(env) unless @log_target
       @tracker.check_worker
-      request_method = env["REQUEST_METHOD"]
       record_header_metrics(env)
       response, duration = process_request(env)
-      record_request_metrics(response.first, request_method, duration)
+      record_request_metrics(response.first, env["REQUEST_METHOD"], duration)
       response
     end
 
@@ -100,47 +127,72 @@ module Librato
       [response, duration]
     end
 
-    def record_header_metrics(env)
-      queue_start = env['HTTP_X_REQUEST_START'] || env['HTTP_X_QUEUE_START']
-      if queue_start
-        queue_start = queue_start.to_s.sub('t=', '').sub('.', '')
-        case queue_start.length
-        when 16 # microseconds
-          wait = ((Time.now.to_f * 1000000).to_i - queue_start.to_i) / 1000.0
-          tracker.timing 'rack.request.queue.time', wait, percentile: 95
-        when 13 # milliseconds
-          wait = (Time.now.to_f * 1000).to_i - queue_start.to_i
-          tracker.timing 'rack.request.queue.time', wait, percentile: 95
+
+    # Dynamically construct :record_request_metrics method based on
+    # configured metric suites
+    def build_record_request_metrics_method
+      body = "def record_request_metrics(status, http_method, duration)\n"
+      body << "return if config.disable_rack_metrics\n"
+
+      unless config.instance_of?(Librato::Rack::Configuration::SuitesNone)
+        body << "tracker.group 'rack.request' do |group|\n"
+
+        if tracker.suite_enabled?(:rack)
+          body << RECORD_RACK_BODY
         end
+
+        if tracker.suite_enabled?(:rack_status)
+          body << RECORD_RACK_STATUS_BODY
+        end
+
+        if tracker.suite_enabled?(:rack_method)
+          body << RECORD_RACK_METHOD_BODY
+        end
+
+        body << "end\n"
       end
-    end
 
-    def record_request_metrics(status, http_method, duration)
-      return if config.disable_rack_metrics
-      tracker.group 'rack.request' do |group|
-        group.increment 'total'
-        group.timing    'time', duration, percentile: 95
-        group.increment 'slow' if duration > 200.0
+      body << "end\n"
 
-        group.group 'status' do |s|
-          s.increment status
-          s.increment "#{status.to_s[0]}xx"
+      instance_eval(body)
+    end
 
-          s.timing "#{status}.time", duration
-          s.timing "#{status.to_s[0]}xx.time", duration
+    # Dynamically construct :record_header_metrics method based on
+    # configured metric suites
+    def build_record_header_metrics_method
+      if tracker.suite_enabled?(:rack)
+        define_singleton_method(:record_header_metrics) do |env|
+          queue_start = env['HTTP_X_REQUEST_START'] || env['HTTP_X_QUEUE_START']
+          if queue_start
+            queue_start = queue_start.to_s.sub('t=', '').sub('.', '')
+            case queue_start.length
+            when 16 # microseconds
+              wait = ((Time.now.to_f * 1000000).to_i - queue_start.to_i) / 1000.0
+              tracker.timing 'rack.request.queue.time', wait, percentile: 95
+            when 13 # milliseconds
+              wait = (Time.now.to_f * 1000).to_i - queue_start.to_i
+              tracker.timing 'rack.request.queue.time', wait, percentile: 95
+            end
+          end
         end
-
-        group.group 'method' do |m|
-          http_method.downcase!
-          m.increment http_method
-          m.timing "#{http_method}.time", duration
+      else
+        define_singleton_method(:record_header_metrics) do |env|
+          # no-op
         end
       end
     end
 
-    def record_exception(exception)
-      return if config.disable_rack_metrics
-      tracker.increment 'rack.request.exceptions'
+    def build_record_exception_method
+      if tracker.suite_enabled?(:rack)
+        define_singleton_method(:record_exception) do |exception|
+          return if config.disable_rack_metrics
+          tracker.increment 'rack.request.exceptions'
+        end
+      else
+        define_singleton_method(:record_exception) do |exception|
+          # no-op
+        end
+      end
     end
 
   end

data/lib/librato/rack/configuration.rb

@@ -1,3 +1,5 @@
+
+
 module Librato
   class Rack
     # Holds configuration for Librato::Rack middleware to use.
@@ -12,9 +14,11 @@ module Librato
     class Configuration
       EVENT_MODES = [:eventmachine, :synchrony]
 
+      DEFAULT_SUITES = [:rack, :rack_method, :rack_status]
+
       attr_accessor :user, :token, :autorun, :api_endpoint, :tracker,
                     :source_pids, :log_level, :log_prefix, :log_target,
-                    :disable_rack_metrics, :flush_interval, :proxy
+                    :disable_rack_metrics, :flush_interval, :proxy, :suites
       attr_reader :prefix, :source, :deprecations
 
       def initialize
@@ -61,6 +65,7 @@ module Librato
         self.log_level = ENV['LIBRATO_LOG_LEVEL'] || :info
         self.proxy = ENV['LIBRATO_PROXY'] || ENV['https_proxy'] || ENV['http_proxy']
         self.event_mode = ENV['LIBRATO_EVENT_MODE']
+        self.suites = ENV['LIBRATO_SUITES'] || ''
         check_deprecations
       end
 
@@ -80,14 +85,30 @@ module Librato
 
       def dump
         fields = {}
-        %w{user token log_level source prefix flush_interval source_pids}.each do |field|
+        %w{user token log_level source prefix flush_interval source_pids suites}.each do |field|
           fields[field.to_sym] = self.send(field)
         end
+        fields[:metric_suites] = metric_suites.fields
         fields
       end
 
+      def metric_suites
+        @metric_suites ||= case suites.downcase.strip
+                           when 'all'
+                             SuitesAll.new
+                           when 'none'
+                             SuitesNone.new
+                           else
+                             Suites.new(suites, default_suites)
+                           end
+      end
+
       private
 
+      def default_suites
+        DEFAULT_SUITES
+      end
+
       def check_deprecations
         %w{USER TOKEN PREFIX SOURCE}.each do |item|
           if ENV["LIBRATO_METRICS_#{item}"]
@@ -114,3 +135,5 @@ module Librato
     end
   end
 end
+
+require_relative 'configuration/suites'

data/lib/librato/rack/configuration/suites.rb

@@ -0,0 +1,58 @@
+module Librato
+  class Rack
+    class Configuration
+
+      class Suites
+        attr_reader :fields
+        def initialize(value, defaults)
+          @fields = if value.nil? || value.empty?
+                      defaults
+                    else
+                      resolve_suites(value, defaults)
+                    end
+        end
+
+        def include?(field)
+          fields.include?(field)
+        end
+
+        private
+
+        def resolve_suites(value, defaults)
+          suites = value.to_s.split(/\s*,\s*/)
+          adds = suites.select { |i| i.start_with?('+') }.map { |i| i[1..-1].to_sym }
+          subs = suites.select { |i| i.start_with?('-') }.map { |i| i[1..-1].to_sym }
+
+          if adds.any? || subs.any?
+
+            # Did they try to mix adds/subs with explicit config
+            if (adds.size + subs.size) != suites.size
+              raise InvalidSuiteConfiguration, "Invalid suite value #{value}"
+            end
+
+            (defaults | adds) - subs
+          else
+            suites.map(&:to_sym)
+          end
+        end
+      end
+
+      class SuitesAll
+        def fields; [:all]; end
+
+        def include?(value)
+          true
+        end
+      end
+
+      class SuitesNone
+        def fields; []; end
+
+        def include?(value)
+          false
+        end
+      end
+
+    end
+  end
+end

data/lib/librato/rack/errors.rb

@@ -3,5 +3,7 @@ module Librato
 
     class InvalidLogLevel < RuntimeError; end
 
+    class InvalidSuiteConfiguration < RuntimeError; end
+
   end
-end
+end

data/lib/librato/rack/tracker.rb

@@ -101,6 +101,10 @@ module Librato
         logger.outlet = target
       end
 
+      def suite_enabled?(suite)
+        config.metric_suites.include?(suite.to_sym)
+      end
+
       private
 
       # access to client instance

data/lib/librato/rack/version.rb

@@ -1,5 +1,5 @@
 module Librato
   class Rack
-    VERSION = "0.6.0"
+    VERSION = "1.0.0.beta1"
   end
 end

data/test/apps/basic.ru

@@ -1,6 +1,21 @@
 require 'bundler/setup'
 require 'librato-rack'
 
+# Simulate the environment variables Heroku passes along
+# with each request
+#
+class QueueWait
+  def initialize(app)
+    @app = app
+  end
+
+  def call(env)
+    env['HTTP_X_QUEUE_START'] = (Time.now.to_f * 1000).to_i.to_s
+    @app.call(env)
+  end
+end
+
+use QueueWait
 use Librato::Rack
 
 def application(env)
@@ -17,4 +32,4 @@ def application(env)
   end
 end
 
-run method(:application)
+run method(:application)

data/test/apps/custom_suites.ru

@@ -0,0 +1,18 @@
+require 'bundler/setup'
+require 'librato-rack'
+
+config = Librato::Rack::Configuration.new
+config.suites = '-rack_status,-rack_method'
+
+use Librato::Rack, :config => config
+
+def application(env)
+  case env['PATH_INFO']
+  when '/exception'
+    raise 'exception raised!'
+  else
+    [200, {"Content-Type" => 'text/html'}, ["Hello!"]]
+  end
+end
+
+run method(:application)

data/test/integration/no_suites_test.rb

@@ -0,0 +1,71 @@
+require 'test_helper'
+require 'rack/test'
+
+# Tests to ensure tracking is disabled when a suite is removed. These tests
+# largely ensure that behavior verified positively other suites doesn't
+# happen when specific suites are disabled.
+#
+class NoSuitesTest < Minitest::Test
+  include Rack::Test::Methods
+  include EnvironmentHelpers
+
+  def app
+    ENV['LIBRATO_SUITES'] = 'none'
+    Rack::Builder.parse_file('test/apps/basic.ru').first
+  end
+
+  def teardown
+    # clear metrics before each run
+    aggregate.delete_all
+    counters.delete_all
+    clear_config_env_vars
+  end
+
+  def test_increment_total
+    get '/'
+    assert last_response.ok?
+    assert_nil counters["rack.request.total"], 'should not increment'
+  end
+
+  def test_track_queue_time
+    get '/'
+    assert last_response.ok?
+    assert_equal nil, aggregate["rack.request.queue.time"]
+  end
+
+  def test_increment_status
+    get '/'
+    assert last_response.ok?
+    assert_nil counters["rack.request.status.200"], 'should not increment'
+    assert_nil counters["rack.request.status.2xx"], 'should not increment'
+  end
+
+  def test_track_http_method_info
+    get '/'
+    assert_nil counters['rack.request.method.get']
+
+    post '/'
+    assert_nil counters['rack.request.method.post']
+  end
+
+  def test_increment_exception
+    begin
+      get '/exception'
+    rescue RuntimeError => e
+      raise unless e.message == 'exception raised!'
+    end
+
+    assert_nil counters["rack.request.exceptions"], 'should not increment'
+  end
+
+  private
+
+  def aggregate
+    Librato.tracker.collector.aggregate
+  end
+
+  def counters
+    Librato.tracker.collector.counters
+  end
+
+end

data/test/integration/suites_test.rb

@@ -0,0 +1,48 @@
+require 'test_helper'
+require 'rack/test'
+
+# Tests to ensure config suites work
+#
+class SuitesTest < Minitest::Test
+  include Rack::Test::Methods
+
+  def app
+    Rack::Builder.parse_file('test/apps/custom_suites.ru').first
+  end
+
+  def teardown
+    # clear metrics before each run
+    aggregate.delete_all
+    counters.delete_all
+  end
+
+  def test_no_rack_status
+    get '/'
+    assert last_response.ok?
+
+    # rack.request metrics (rack suite) should get logged
+    assert_equal 1, counters["rack.request.total"]
+    assert_equal 1, aggregate["rack.request.time"][:count]
+
+    # rack.request.method metrics (rack_method suite) should not get logged
+    assert_equal nil, counters['rack.request.method.get']
+    assert_equal nil, aggregate['rack.request.method.get.time']
+
+    # rack.request.status metrics (rack_status suite) should not get logged
+    assert_equal nil, counters["rack.request.status.200"]
+    assert_equal nil, counters["rack.request.status.2xx"]
+    assert_equal nil, counters["rack.request.status.200.time"]
+    assert_equal nil, counters["rack.request.status.2xx.time"]
+  end
+
+  private
+
+  def aggregate
+    Librato.tracker.collector.aggregate
+  end
+
+  def counters
+    Librato.tracker.collector.counters
+  end
+
+end

data/test/support/environment_helpers.rb

@@ -0,0 +1,23 @@
+# Helper methods for environment management that are shared
+# between test files.
+#
+module EnvironmentHelpers
+
+  def clear_config_env_vars
+    ENV.delete('LIBRATO_USER')
+    ENV.delete('LIBRATO_TOKEN')
+    ENV.delete('LIBRATO_PROXY')
+    ENV.delete('LIBRATO_SOURCE')
+    ENV.delete('LIBRATO_PREFIX')
+    ENV.delete('LIBRATO_SUITES')
+    ENV.delete('LIBRATO_LOG_LEVEL')
+    ENV.delete('LIBRATO_EVENT_MODE')
+    # legacy - deprecated
+    ENV.delete('LIBRATO_METRICS_USER')
+    ENV.delete('LIBRATO_METRICS_TOKEN')
+    ENV.delete('LIBRATO_METRICS_SOURCE')
+    # system
+    ENV.delete('http_proxy')
+  end
+
+end

data/test/test_helper.rb

@@ -6,3 +6,4 @@ require 'minitest/autorun'
 
 require 'librato/rack'
 
+require_relative 'support/environment_helpers'

data/test/unit/rack/configuration/suites_test.rb

@@ -0,0 +1,94 @@
+require 'test_helper'
+
+module Librato
+  class Rack
+    class SuitesTest < Minitest::Test
+      include EnvironmentHelpers
+
+      def setup;    clear_config_env_vars; end
+      def teardown; clear_config_env_vars; end
+
+      def test_suites_defaults
+        config = Configuration.new
+        assert config.metric_suites.include?(:rack), "should include 'rack' by default"
+        refute config.metric_suites.include?(:foo), "should not include 'foo' by default"
+      end
+
+      def test_suites_configured_by_explicit_list
+        ENV['LIBRATO_SUITES'] = 'abc, jkl,prq , xyz'
+        config = Configuration.new
+        [:abc, :jkl, :prq, :xyz].each do |suite|
+          assert config.metric_suites.include?(suite), "expected '#{suite}' to be active"
+        end
+
+        Librato::Rack::Configuration::DEFAULT_SUITES.each do |suite|
+          refute config.metric_suites.include?(suite), "should not include '#{suite}' by default"
+        end
+
+        refute config.metric_suites.include?(:something_else), 'should not include unspecified'
+      end
+
+      def test_suites_configured_by_inclusion
+        ENV['LIBRATO_SUITES'] = '+abc, +jkl,+prq'
+        config = Configuration.new
+
+        [:abc, :jkl, :prq].each do |suite|
+          assert config.metric_suites.include?(suite), "expected '#{suite.to_s}' to be active"
+        end
+
+        Librato::Rack::Configuration::DEFAULT_SUITES.each do |suite|
+          assert config.metric_suites.include?(suite), "should include '#{suite}' by default"
+        end
+      end
+
+      def test_suites_configured_by_exclusion
+        ENV['LIBRATO_SUITES'] = '-rack_method,-jkl'
+        config = Configuration.new
+
+        [:rack_method, :jkl].each do |suite|
+          refute config.metric_suites.include?(suite), "expected '#{suite.to_s}' to be disabled"
+        end
+
+        assert config.metric_suites.include?(:rack), "should include 'rack' by default"
+        assert config.metric_suites.include?(:rack_status), "should include 'rack_status' by default"
+      end
+
+      def test_suites_configured_by_inclusion_and_exclusion
+        ENV['LIBRATO_SUITES'] = '-rack_method, +foo'
+        config = Configuration.new
+
+        assert config.metric_suites.include?(:rack), "should include 'rack' by default"
+        assert config.metric_suites.include?(:rack_status), "should include 'rack_status' by default"
+        assert config.metric_suites.include?(:foo), "expected 'foo' to be active"
+        refute config.metric_suites.include?(:rack_method), "expected 'rack_method' to be disabled"
+      end
+
+      def test_invalid_suite_config
+        ENV['LIBRATO_SUITES'] = '-rack_method, +foo ,bar'
+
+        assert_raises(Librato::Rack::InvalidSuiteConfiguration) {
+          Configuration.new.metric_suites
+        }
+      end
+
+      def test_suites_all
+        ENV['LIBRATO_SUITES'] = 'all'
+        config = Configuration.new
+
+        [:foo, :bar, :baz].each do |suite|
+          assert config.metric_suites.include?(suite), "expected '#{suite}' to be active"
+        end
+      end
+
+      def test_suites_none
+        ENV['LIBRATO_SUITES'] = 'NONE'
+        config = Configuration.new
+
+        [:foo, :bar, :baz].each do |suite|
+          refute config.metric_suites.include?(suite), "expected '#{suite.to_s}' to be disabled"
+        end
+      end
+
+    end
+  end
+end

data/test/unit/rack/configuration_test.rb

@@ -3,19 +3,16 @@ require 'test_helper'
 module Librato
   class Rack
     class ConfigurationTest < Minitest::Test
+      include EnvironmentHelpers
 
-      def setup
-        clear_env_vars
-      end
-
-      def teardown
-        clear_env_vars
-      end
+      def setup;    clear_config_env_vars; end
+      def teardown; clear_config_env_vars; end
 
       def test_defaults
         config = Configuration.new
         assert_equal 60, config.flush_interval
         assert_equal Librato::Metrics.api_endpoint, config.api_endpoint
+        assert_equal '', config.suites
       end
 
       def test_environment_variable_config
@@ -23,11 +20,13 @@ module Librato
         ENV['LIBRATO_TOKEN'] = 'api_key'
         ENV['LIBRATO_SOURCE'] = 'source'
         ENV['LIBRATO_PROXY'] = 'http://localhost:8080'
+        ENV['LIBRATO_SUITES'] = 'foo,bar'
         config = Configuration.new
         assert_equal 'foo@bar.com', config.user
         assert_equal 'api_key', config.token
         assert_equal 'source', config.source
         assert_equal 'http://localhost:8080', config.proxy
+        assert_equal 'foo,bar', config.suites
         #assert Librato::Rails.explicit_source, 'source is explicit'
       end
 
@@ -89,22 +88,6 @@ module Librato
 
       private
 
-      def clear_env_vars
-        ENV.delete('LIBRATO_USER')
-        ENV.delete('LIBRATO_TOKEN')
-        ENV.delete('LIBRATO_PROXY')
-        ENV.delete('LIBRATO_SOURCE')
-        ENV.delete('LIBRATO_PREFIX')
-        ENV.delete('LIBRATO_LOG_LEVEL')
-        ENV.delete('LIBRATO_EVENT_MODE')
-        # legacy
-        ENV.delete('LIBRATO_METRICS_USER')
-        ENV.delete('LIBRATO_METRICS_TOKEN')
-        ENV.delete('LIBRATO_METRICS_SOURCE')
-        # system
-        ENV.delete('http_proxy')
-      end
-
       def listener_object
         listener = Object.new
         def listener.prefix=(prefix)

data/test/unit/rack/tracker_test.rb

@@ -46,6 +46,17 @@ module Librato
         ENV.delete('LIBRATO_AUTORUN')
       end
 
+      def test_suite_configured
+        ENV['LIBRATO_SUITES'] = 'abc,prq'
+
+        tracker = Tracker.new(Configuration.new)
+        assert tracker.suite_enabled?(:abc)
+        assert tracker.suite_enabled?(:prq)
+        refute tracker.suite_enabled?(:xyz)
+      ensure
+        ENV.delete('LIBRATO_SUITES')
+      end
+
       private
 
       def buffer_lines

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: librato-rack
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 1.0.0.beta1
 platform: ruby
 authors:
 - Matt Sanders
@@ -29,7 +29,7 @@ cert_chain:
   hvjx0WJP8pzZMJPKJBRZQXJO5ifEPyKjZyMi5XMHmrtDclHLj3sx4RAvEZjGWkRP
   JSQ=
   -----END CERTIFICATE-----
-date: 2015-12-10 00:00:00.000000000 Z
+date: 2016-04-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: librato-metrics
@@ -93,6 +93,7 @@ files:
 - lib/librato/collector/group.rb
 - lib/librato/rack.rb
 - lib/librato/rack/configuration.rb
+- lib/librato/rack/configuration/suites.rb
 - lib/librato/rack/errors.rb
 - lib/librato/rack/logger.rb
 - lib/librato/rack/tracker.rb
@@ -101,20 +102,25 @@ files:
 - lib/librato/rack/worker.rb
 - test/apps/basic.ru
 - test/apps/custom.ru
+- test/apps/custom_suites.ru
 - test/apps/deprecated.ru
 - test/apps/no_stats.ru
 - test/apps/queue_wait.ru
 - test/integration/custom_test.rb
 - test/integration/deprecated_test.rb
 - test/integration/no_stats_test.rb
+- test/integration/no_suites_test.rb
 - test/integration/queue_wait_test.rb
 - test/integration/request_test.rb
+- test/integration/suites_test.rb
 - test/remote/tracker_test.rb
+- test/support/environment_helpers.rb
 - test/test_helper.rb
 - test/unit/collector/aggregator_test.rb
 - test/unit/collector/counter_cache_test.rb
 - test/unit/collector/group_test.rb
 - test/unit/collector_test.rb
+- test/unit/rack/configuration/suites_test.rb
 - test/unit/rack/configuration_test.rb
 - test/unit/rack/logger_test.rb
 - test/unit/rack/tracker_test.rb
@@ -134,9 +140,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.2.2
@@ -146,20 +152,25 @@ summary: Use Librato Metrics with your rack application
 test_files:
 - test/apps/basic.ru
 - test/apps/custom.ru
+- test/apps/custom_suites.ru
 - test/apps/deprecated.ru
 - test/apps/no_stats.ru
 - test/apps/queue_wait.ru
 - test/integration/custom_test.rb
 - test/integration/deprecated_test.rb
 - test/integration/no_stats_test.rb
+- test/integration/no_suites_test.rb
 - test/integration/queue_wait_test.rb
 - test/integration/request_test.rb
+- test/integration/suites_test.rb
 - test/remote/tracker_test.rb
+- test/support/environment_helpers.rb
 - test/test_helper.rb
 - test/unit/collector/aggregator_test.rb
 - test/unit/collector/counter_cache_test.rb
 - test/unit/collector/group_test.rb
 - test/unit/collector_test.rb
+- test/unit/rack/configuration/suites_test.rb
 - test/unit/rack/configuration_test.rb
 - test/unit/rack/logger_test.rb
 - test/unit/rack/tracker_test.rb