haproxy2rpm 0.1.1 → 0.2.0

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## v 0.2.0
+
+* Introducing default route
+
 ## v 0.1.1
 
 * BUGFIX: hostnames with hyphens e.g s-app-1 break the syslog parser

data/README.md

@@ -27,6 +27,21 @@ Tell haproxy to log to syslog. e.g:
 
 Check the examples folder
 
+
+## Problems with haproxy timers
+
+Slow clients (e.g mobile clients over an edge connection) influence the
+measurements haproxy returns. The same happens for requests that do send
+a lot of data (e.g POST/PUT). In that case, the 'tr' value provided by
+haproxy is generally a bit higher. When you have many slow clients, it
+is significantely higher. For more information about the values haproxy
+provides, refer to the [manual](http://code.google.com/p/haproxy-docs/wiki/TimingEvents).
+
+## The issue of the default route
+
+When haproxy2rpm does not find a route, it will record it under a
+default name so that NewRelic is not flooded with millions of routes. You can provide by setting 'config.default_route'. 
+
 ## Node.js support
 
 Check the examples folder

data/lib/haproxy2rpm/rpm.rb

@@ -2,6 +2,7 @@ module Haproxy2Rpm
   class Rpm
     
     attr_accessor :routes, :queue_time_stats_engine, :stats_engine
+    attr_writer :default_route
     
     def initialize(options = {})
       agent_options = {:log => Haproxy2Rpm.logger}
@@ -21,7 +22,11 @@ module Haproxy2Rpm
     def config
       self
     end
-    
+
+    def default_route
+      @default_route ||= '/default'
+    end
+
     def process_and_send(line)
       begin
         message = message_parser.call(line)
@@ -88,7 +93,7 @@ module Haproxy2Rpm
           return route[:target]
         end
       end
-      path
+      default_route
     end
   end
 end

data/lib/haproxy2rpm/version.rb

@@ -1,3 +1,3 @@
 module Haproxy2Rpm
-  VERSION = "0.1.1"
+  VERSION = "0.2.0"
 end

data/test/rpm_test.rb

@@ -33,6 +33,7 @@ class RpmTest < Test::Unit::TestCase
     end
     
     should 'record the controller metric' do
+      @instance.routes = [{:pattern => %r{^/user/check$}, :target => '/user/check'}]
       NewRelic::Agent.expects(:record_transaction).with(anything, has_entry('metric', 'Controller/user/check'))
       @instance.process_and_send(log_entry(:http_path => "/user/check"))
     end
@@ -42,6 +43,13 @@ class RpmTest < Test::Unit::TestCase
       NewRelic::Agent.expects(:record_transaction).with(anything, has_entry('metric', 'Controller/user/update'))
       @instance.process_and_send(log_entry(:http_path => "/49339032093/update"))
     end
+
+    should 'fallback to default route when unknown' do
+      @instance.routes = []
+      @instance.default_route = '/default'
+      NewRelic::Agent.expects(:record_transaction).with(anything, has_entry('metric', 'Controller/default'))
+      @instance.process_and_send(log_entry(:http_path => "/unknown/route"))
+    end
         
     should 'record the queue time' do
       stats = mock

metadata

@@ -1,105 +1,81 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: haproxy2rpm
-version: !ruby/object:Gem::Version 
-  hash: 25
+version: !ruby/object:Gem::Version
+  version: 0.2.0
   prerelease: 
-  segments: 
-  - 0
-  - 1
-  - 1
-  version: 0.1.1
 platform: ruby
-authors: 
+authors:
 - Patrick Huesler
 - Martin Rehfeld
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2012-01-04 00:00:00 +01:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2012-01-05 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: newrelic_rpm
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
+  requirement: &70355528340020 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: eventmachine-tail
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  version_requirements: *70355528340020
+- !ruby/object:Gem::Dependency
+  name: eventmachine-tail
+  requirement: &70355528339240 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
-  name: rake
   prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  version_requirements: *70355528339240
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: &70355528338720 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
-  name: shoulda-context
   prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  version_requirements: *70355528338720
+- !ruby/object:Gem::Dependency
+  name: shoulda-context
+  requirement: &70355528338120 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
-  name: mocha
   prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  version_requirements: *70355528338120
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: &70355528337540 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id005
+  prerelease: false
+  version_requirements: *70355528337540
 description: Sending haproxy logs to new relic rpm
-email: 
+email:
 - patrick.huesler@wooga.com
 - martin.rehfeld@wooga.com
-executables: 
+executables:
 - haproxy2rpm
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
+files:
 - .gitignore
 - .travis.yml
 - CHANGELOG.md
@@ -126,41 +102,31 @@ files:
 - test/rpm_test.rb
 - test/syslog_test.rb
 - test/test_helper.rb
-has_rdoc: true
 homepage: https://github.com/wooga/haproxy2rpm
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.6.2
+rubygems_version: 1.8.6
 signing_key: 
 specification_version: 3
 summary: Sending haproxy logs to new relic rpm
-test_files: 
+test_files:
 - test/fixtures/config.rb
 - test/haproxy2pm_test.rb
 - test/rpm_test.rb