puma-plugin-statsd 1.2.1 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bb512a32b788a2df3fc7f9f222e7dcb510fdbda09bb1bb278466978d2f5b511d
-  data.tar.gz: 6274f07b78d32a6bddbc5d6d823114f6171df7fb3131186e35fbdd9416244ce1
+  metadata.gz: 1ac5651e242dcf306ed0c5964de2c7178a142e9555333cfd203e043dada926a6
+  data.tar.gz: 9cc7a3b70cdcda54a824ca4d9b65714c16f7b20834bb348182eb787820b5e5e1
 SHA512:
-  metadata.gz: c568951db0c140e934d4cd40103c8b98160da02aeeda6b2549dd59a415ee2ee59ea171f6d818150fdbb66c99b88a912690c31f9bc3bc0715dc7478440136e76a
-  data.tar.gz: 2af0724eb7a83bae47caed6c3009a5fa5ed2d0ab98b2c98be250c03636750bdf48788f69fe8bd04fb83a7656fb15174ab356edd0581eed072e780b4fa0855aa8
+  metadata.gz: 1222d6580444972583bec535348958301a8c5aa4860d08e87bb1e735fc09d9b2764a38b4c3b9aa355878b9080ca30ef80b413c55c0f8f4b97edf750515162a89
+  data.tar.gz: 07e8987c8ce8f8f3db573241f824c80224aec9ffa2f5b1e8bbd474105981d6529c0c1a4a870995622f62c9edad1181aaced6c3382b8746aebf9149f51d0842c9

data/CHANGELOG.md

@@ -1,12 +1,18 @@
 # CHANGELOG
 
+## 2.0.0 2021-07-27
+
+* Require puma 5 or better
+* Split DD_TAGS environment variable by commas or spaces (PR #[31](https://github.com/yob/puma-plugin-statsd/pull/31))
+* Gracefully handle unexpecte errors when submitting to statsd (like DNS resolution failures) (PR #[35](https://github.com/yob/puma-plugin-statsd/pull/35))
+
 ## 1.2.1 2021-01-11
 
 * Remove json from the gemspec
 
 ## 1.2.0 2021-01-07
 
-* New metrics: old_workers (PR #[21](https://github.com/yob/puma-plugin-statsd/pull/21)) and requsts_count (PR #[28](https://github.com/yob/puma-plugin-statsd/pull/28))
+* New metrics: old_workers (PR #[21](https://github.com/yob/puma-plugin-statsd/pull/21)) and requests_count (PR #[28](https://github.com/yob/puma-plugin-statsd/pull/28))
 * Require json at runtime to be extra sure we don't load the wrong version before bundler has initialised the LOAD_PATH
 
 ## 1.1.0 2021-01-03

data/README.md

@@ -9,6 +9,8 @@ that puma can provide:
 * puma.backlog
 * puma.pool_capacity
 * puma.max_threads
+* puma.old_workers
+* puma.requests_count
 
   [puma]: https://github.com/puma/puma
   [statsd]: https://github.com/etsy/statsd
@@ -108,9 +110,17 @@ https://github.com/yob/puma-plugin-statsd.
 
 ## Testing the data being sent to statsd
 
-Start a pretend statsd server that listens for UDP packets on port 8125:
+Start a pretend statsd server that listens for UDP packets on port 8125.
 
-    ruby devtools/statsd-to-stdout.rb
+If you've installed the gem in your app:
+
+    # only need to install the binstub once
+    bundle binstubs puma-plugin-statsd
+    ./bin/statsd-to-stdout
+
+If you are developing/testing this gem locally:
+
+    ./bin/statsd-to-stdout
 
 Start puma:
 

data/bin/statsd-to-stdout

@@ -0,0 +1,15 @@
+#! /usr/bin/env ruby
+# encoding: utf-8
+
+RECV_BUFFER = 64*1024
+
+require 'socket'
+server = UDPSocket.new
+host, port = "127.0.0.1", 8125
+server.bind(host, port)
+
+while true
+  text, sender = server.recvfrom(RECV_BUFFER)
+  remote_host = sender[3]
+  STDOUT.puts "#{remote_host}:" + text
+end

data/lib/puma/plugin/statsd.rb

@@ -112,18 +112,6 @@ Puma::Plugin.create do
     in_background(&method(:stats_loop))
   end
 
-  if Puma.respond_to?(:stats_hash)
-    def fetch_stats
-      Puma.stats_hash
-    end
-  else
-    def fetch_stats
-      require "json"
-      stats = Puma.stats
-      JSON.parse(stats, symbolize_names: true)
-    end
-  end
-
   def environment_variable_tags
     # Tags are separated by spaces, and while they are normally a tag and
     # value separated by a ':', they can also just be tagged without any
@@ -147,7 +135,7 @@ Puma::Plugin.create do
     # https://docs.datadoghq.com/agent/docker/?tab=standard#global-options
     #
     if ENV.has_key?("DD_TAGS")
-      ENV["DD_TAGS"].split(/\s+/).each do |t|
+      ENV["DD_TAGS"].split(/\s+|,/).each do |t|
         tags << t
       end
     end
@@ -171,7 +159,7 @@ Puma::Plugin.create do
     loop do
       @launcher.events.debug "statsd: notify statsd"
       begin
-        stats = ::PumaStats.new(fetch_stats)
+        stats = ::PumaStats.new(Puma.stats_hash)
         @statsd.send(metric_name: prefixed_metric_name("puma.workers"), value: stats.workers, type: :gauge, tags: tags)
         @statsd.send(metric_name: prefixed_metric_name("puma.booted_workers"), value: stats.booted_workers, type: :gauge, tags: tags)
         @statsd.send(metric_name: prefixed_metric_name("puma.old_workers"), value: stats.old_workers, type: :gauge, tags: tags)
@@ -181,7 +169,7 @@ Puma::Plugin.create do
         @statsd.send(metric_name: prefixed_metric_name("puma.max_threads"), value: stats.max_threads, type: :gauge, tags: tags)
         @statsd.send(metric_name: prefixed_metric_name("puma.requests_count"), value: stats.requests_count, type: :gauge, tags: tags)
       rescue StandardError => e
-        @launcher.events.error "! statsd: notify stats failed:\n  #{e.to_s}\n  #{e.backtrace.join("\n    ")}"
+        @launcher.events.unknown_error e, nil, "! statsd: notify stats failed"
       ensure
         sleep 2
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puma-plugin-statsd
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 2.0.0
 platform: ruby
 authors:
 - James Healy
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-11 00:00:00.000000000 Z
+date: 2021-07-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: puma
@@ -16,7 +16,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.12'
+        version: '5.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '6'
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.12'
+        version: '5.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '6'
@@ -102,13 +102,15 @@ dependencies:
         version: '0'
 description: 
 email: james@yob.id.au
-executables: []
+executables:
+- statsd-to-stdout
 extensions: []
 extra_rdoc_files: []
 files:
 - CHANGELOG.md
 - MIT-LICENSE
 - README.md
+- bin/statsd-to-stdout
 - lib/puma/plugin/statsd.rb
 homepage: https://github.com/yob/puma-plugin-statsd
 licenses: