riemann-tools 1.0.0 → 1.1.0

data/tools/riemann-elasticsearch/bin/riemann-elasticsearch

@@ -1,166 +1,174 @@
 #!/usr/bin/env ruby
-Process.setproctitle($0)
+# frozen_string_literal: true
+
+Process.setproctitle($PROGRAM_NAME)
 
 require 'riemann/tools'
 
-class Riemann::Tools::Elasticsearch
-  include Riemann::Tools
-  require 'faraday'
-  require 'json'
-  require 'uri'
-
-  opt :read_timeout, 'Faraday read timeout', type: :int, default: 2
-  opt :open_timeout, 'Faraday open timeout', type: :int, default: 1
-  opt :path_prefix, 'Elasticsearch path prefix for proxied installations e.g. "els" for target http://localhost/els/_cluster/health', default: "/"
-  opt :es_host, 'Elasticsearch host', default: "localhost"
-  opt :es_port, 'Elasticsearch port', type: :int, default: 9200
-  opt :es_search_index, 'Elasticsearch index to fetch search statistics for', default: "_all"
-
-
-  # Handles HTTP connections and GET requests safely
-  def safe_get(uri)
-      # Handle connection timeouts
-      response = nil
-      begin
-        connection = Faraday.new(uri)
-        response = connection.get do |req|
-          req.options[:timeout] = options[:read_timeout]
-          req.options[:open_timeout] = options[:open_timeout]
+module Riemann
+  module Tools
+    class Elasticsearch
+      include Riemann::Tools
+      require 'faraday'
+      require 'json'
+      require 'uri'
+
+      opt :read_timeout, 'Faraday read timeout', type: :int, default: 2
+      opt :open_timeout, 'Faraday open timeout', type: :int, default: 1
+      opt :path_prefix,
+          'Elasticsearch path prefix for proxied installations e.g. "els" for target http://localhost/els/_cluster/health', default: '/'
+      opt :es_host, 'Elasticsearch host', default: 'localhost'
+      opt :es_port, 'Elasticsearch port', type: :int, default: 9200
+      opt :es_search_index, 'Elasticsearch index to fetch search statistics for', default: '_all'
+
+      # Handles HTTP connections and GET requests safely
+      def safe_get(uri)
+        # Handle connection timeouts
+        response = nil
+        begin
+          connection = Faraday.new(uri)
+          response = connection.get do |req|
+            req.options[:timeout] = options[:read_timeout]
+            req.options[:open_timeout] = options[:open_timeout]
+          end
+        rescue StandardError => e
+          report(
+            host: uri.host,
+            service: 'elasticsearch health',
+            state: 'critical',
+            description: "HTTP connection error: #{e.class} - #{e.message}",
+          )
         end
-      rescue => e
-        report(:host => uri.host,
-          :service => "elasticsearch health",
-          :state => "critical",
-          :description => "HTTP connection error: #{e.class} - #{e.message}"
-        )
+        response
       end
-      response
-  end
-
-  def make_es_url(path)
-    path_prefix = options[:path_prefix]
-    path_prefix[0] = '' if path_prefix[0]=='/'
-    path_prefix[path_prefix.length-1] = '' if path_prefix[path_prefix.length-1]=='/'
-    "http://#{options[:es_host]}:#{options[:es_port]}#{path_prefix.length>0?'/':''}#{path_prefix}/#{path}"
-  end
-
-  def health_url
-    make_es_url("_cluster/health")
-  end
-
-  def indices_url
-    make_es_url("_stats/store")
-  end
-
-  def search_url
-    es_search_index = options[:es_search_index]
-    make_es_url("#{es_search_index}/_stats/search")
-  end
-
-  def is_bad?(response, uri)
-    if response.success?
-      false
-    else
-      report(:host => uri.host,
-             :service => "elasticsearch health",
-             :state => "critical",
-             :description => response.nil? ? "HTTP response is empty!" : "HTTP connection error: #{response.status} - #{response.body}"
-      )
-    end
-  end
-
-  def tick_indices
-    uri = URI(indices_url)
-    response = safe_get(uri)
 
-    return if is_bad?(response, uri)
-
-    # Assuming that a 200 will give json
-    json = JSON.parse(response.body)
+      def make_es_url(path)
+        path_prefix = options[:path_prefix]
+        path_prefix[0] = '' if path_prefix[0] == '/'
+        path_prefix[path_prefix.length - 1] = '' if path_prefix[path_prefix.length - 1] == '/'
+        "http://#{options[:es_host]}:#{options[:es_port]}#{path_prefix.length.positive? ? '/' : ''}#{path_prefix}/#{path}"
+      end
 
-    json["indices"].each_pair do |k,v|
-      report(:host => uri.host,
-             :service => "elasticsearch index/#{k}/primaries/size_in_bytes",
-             :metric => v["primaries"]["store"]["size_in_bytes"]
-      )
-      report(:host => uri.host,
-             :service => "elasticsearch index/#{k}/total/size_in_bytes",
-             :metric => v["total"]["store"]["size_in_bytes"]
-      )
-    end
-  end
+      def health_url
+        make_es_url('_cluster/health')
+      end
 
-  def tick_search
-    uri = URI(search_url)
-    response = safe_get(uri)
+      def indices_url
+        make_es_url('_stats/store')
+      end
 
-    return if is_bad?(response, uri)
+      def search_url
+        es_search_index = options[:es_search_index]
+        make_es_url("#{es_search_index}/_stats/search")
+      end
 
-    es_search_index = options[:es_search_index]
-    # Assuming that a 200 will give json
-    json = JSON.parse(response.body)
+      def bad?(response, uri)
+        if response.success?
+          false
+        else
+          report(
+            host: uri.host,
+            service: 'elasticsearch health',
+            state: 'critical',
+            description: response.nil? ? 'HTTP response is empty!' : "HTTP connection error: #{response.status} - #{response.body}",
+          )
+        end
+      end
 
-    json["_all"].each_pair do |type, data|
-      query = data["search"]["query_time_in_millis"].to_f / data["search"]["query_total"].to_f
-      fetch = data["search"]["fetch_time_in_millis"].to_f / data["search"]["fetch_total"].to_f
+      def tick_indices
+        uri = URI(indices_url)
+        response = safe_get(uri)
+
+        return if bad?(response, uri)
+
+        # Assuming that a 200 will give json
+        json = JSON.parse(response.body)
+
+        json['indices'].each_pair do |k, v|
+          report(
+            host: uri.host,
+            service: "elasticsearch index/#{k}/primaries/size_in_bytes",
+            metric: v['primaries']['store']['size_in_bytes'],
+          )
+          report(
+            host: uri.host,
+            service: "elasticsearch index/#{k}/total/size_in_bytes",
+            metric: v['total']['store']['size_in_bytes'],
+          )
+        end
+      end
 
-      report(:host => uri.host,
-             :service => "elasticsearch search/#{es_search_index}/query",
-             :metric => query
-      )
-      report(:host => uri.host,
-             :service => "elasticsearch search/#{es_search_index}/fetch",
-             :metric => fetch
-      )
-    end
-  end
+      def tick_search
+        uri = URI(search_url)
+        response = safe_get(uri)
+
+        return if bad?(response, uri)
+
+        es_search_index = options[:es_search_index]
+        # Assuming that a 200 will give json
+        json = JSON.parse(response.body)
+
+        json['_all'].each_pair do |_type, data|
+          query = data['search']['query_time_in_millis'].to_f / data['search']['query_total']
+          fetch = data['search']['fetch_time_in_millis'].to_f / data['search']['fetch_total']
+
+          report(
+            host: uri.host,
+            service: "elasticsearch search/#{es_search_index}/query",
+            metric: query,
+          )
+          report(
+            host: uri.host,
+            service: "elasticsearch search/#{es_search_index}/fetch",
+            metric: fetch,
+          )
+        end
+      end
 
-  def tick
-    begin
-      tick_indices
-      tick_search
-    rescue Exception => e
-      report(:host => options[:es_host],
-             :service => "elasticsearch error",
-             :state => "critical",
-             :description => "Elasticsearch cluster error: #{e.message}")
-    end
-    uri = URI(health_url)
-    response = safe_get(uri)
-
-    return if is_bad?(response, uri)
-
-    # Assuming that a 200 will give json
-    json = JSON.parse(response.body)
-    cluster_name = json.delete("cluster_name")
-    cluster_status = json.delete("status")
-    state = case cluster_status
-              when "green"
-                "ok"
-              when "yellow"
-                "warning"
-              when "red"
-                "critical"
-            end
-
-    report(:host => uri.host,
-           :service => "elasticsearch health",
-           :state => state,
-           :description => "Elasticsearch cluster: #{cluster_name} - #{cluster_status}")
-
-    json.each_pair do |k,v|
-      report(:host => uri.host,
-             :service => "elasticsearch #{k}",
-             :metric => v,
-             :description => "Elasticsearch cluster #{k}"
-      )
+      def tick
+        begin
+          tick_indices
+          tick_search
+        rescue StandardError => e
+          report(
+            host: options[:es_host],
+            service: 'elasticsearch error',
+            state: 'critical',
+            description: "Elasticsearch cluster error: #{e.message}",
+          )
+        end
+        uri = URI(health_url)
+        response = safe_get(uri)
+
+        return if bad?(response, uri)
+
+        # Assuming that a 200 will give json
+        json = JSON.parse(response.body)
+        cluster_name = json.delete('cluster_name')
+        cluster_status = json.delete('status')
+        state = {
+          'green'  => 'ok',
+          'yellow' => 'warning',
+          'red'    => 'critical',
+        }[cluster_status]
+
+        report(
+          host: uri.host,
+          service: 'elasticsearch health',
+          state: state,
+          description: "Elasticsearch cluster: #{cluster_name} - #{cluster_status}",
+        )
 
+        json.each_pair do |k, v|
+          report(
+            host: uri.host,
+            service: "elasticsearch #{k}",
+            metric: v,
+            description: "Elasticsearch cluster #{k}",
+          )
+        end
+      end
     end
-
   end
-
-
-
 end
 Riemann::Tools::Elasticsearch.run
-

data/tools/riemann-marathon/{Rakefile.rb → Rakefile}

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rubygems'
 require 'rubygems/package_task'
 require 'rdoc/task'

data/tools/riemann-marathon/bin/riemann-marathon

@@ -1,142 +1,159 @@
 #!/usr/bin/env ruby
-Process.setproctitle($0)
+# frozen_string_literal: true
 
-require 'riemann/tools'
+Process.setproctitle($PROGRAM_NAME)
 
-class Riemann::Tools::Marathon
-  include Riemann::Tools
+require 'riemann/tools'
 
-  require 'faraday'
-  require 'json'
-  require 'uri'
+module Riemann
+  module Tools
+    class Marathon
+      include Riemann::Tools
+
+      require 'faraday'
+      require 'json'
+      require 'uri'
+
+      opt :read_timeout, 'Faraday read timeout', type: :int, default: 2
+      opt :open_timeout, 'Faraday open timeout', type: :int, default: 1
+      opt :path_prefix,
+          'Marathon path prefix for proxied installations e.g. "marathon" for target http://localhost/marathon/metrics', default: '/'
+      opt :marathon_host, 'Marathon host', default: 'localhost'
+      opt :marathon_port, 'Marathon port', type: :int, default: 8080
+
+      def initialize
+        options[:interval] = 60
+        options[:ttl] = 120
+      end
 
-  opt :read_timeout, 'Faraday read timeout', type: :int, default: 2
-  opt :open_timeout, 'Faraday open timeout', type: :int, default: 1
-  opt :path_prefix, 'Marathon path prefix for proxied installations e.g. "marathon" for target http://localhost/marathon/metrics', default: "/"
-  opt :marathon_host, 'Marathon host', default: "localhost"
-  opt :marathon_port, 'Marathon port', type: :int, default: 8080
+      # Handles HTTP connections and GET requests safely
+      def safe_get(uri)
+        # Handle connection timeouts
+        response = nil
+        begin
+          connection = Faraday.new(uri)
+          response = connection.get do |req|
+            req.options[:timeout] = options[:read_timeout]
+            req.options[:open_timeout] = options[:open_timeout]
+          end
+        rescue StandardError => e
+          report(
+            host: uri.host,
+            service: 'marathon health',
+            state: 'critical',
+            description: "HTTP connection error: #{e.class} - #{e.message}",
+          )
+        end
+        response
+      end
 
-  def initialize
-    options[:interval] = 60
-    options[:ttl] = 120
-  end
+      def health_url
+        path_prefix = options[:path_prefix]
+        path_prefix[0] = '' if path_prefix[0] == '/'
+        path_prefix[path_prefix.length - 1] = '' if path_prefix[path_prefix.length - 1] == '/'
+        "http://#{options[:marathon_host]}:#{options[:marathon_port]}#{path_prefix.length.positive? ? '/' : ''}#{path_prefix}/metrics"
+      end
 
-  # Handles HTTP connections and GET requests safely
-  def safe_get(uri)
-      # Handle connection timeouts
-      response = nil
-      begin
-        connection = Faraday.new(uri)
-        response = connection.get do |req|
-          req.options[:timeout] = options[:read_timeout]
-          req.options[:open_timeout] = options[:open_timeout]
-        end
-      rescue => e
-        report(:host => uri.host,
-          :service => "marathon health",
-          :state => "critical",
-          :description => "HTTP connection error: #{e.class} - #{e.message}"
-        )
+      def apps_url
+        path_prefix = options[:path_prefix]
+        path_prefix[0] = '' if path_prefix[0] == '/'
+        path_prefix[path_prefix.length - 1] = '' if path_prefix[path_prefix.length - 1] == '/'
+        "http://#{options[:marathon_host]}:#{options[:marathon_port]}#{path_prefix.length.positive? ? '/' : ''}#{path_prefix}/v2/apps"
       end
-      response
-  end
 
-  def health_url
-    path_prefix = options[:path_prefix]
-    path_prefix[0] = '' if path_prefix[0]=='/'
-    path_prefix[path_prefix.length-1] = '' if path_prefix[path_prefix.length-1]=='/'
-    "http://#{options[:marathon_host]}:#{options[:marathon_port]}#{path_prefix.length>0?'/':''}#{path_prefix}/metrics"
-  end
+      def tick
+        tick_health
+        tick_apps
+      end
 
-  def apps_url
-    path_prefix = options[:path_prefix]
-    path_prefix[0] = '' if path_prefix[0]=='/'
-    path_prefix[path_prefix.length-1] = '' if path_prefix[path_prefix.length-1]=='/'
-    "http://#{options[:marathon_host]}:#{options[:marathon_port]}#{path_prefix.length>0?'/':''}#{path_prefix}/v2/apps"
-  end
+      def tick_health
+        uri = URI(health_url)
+        response = safe_get(uri)
+
+        return if response.nil?
+
+        if response.status != 200
+          report(
+            host: uri.host,
+            service: 'marathon health',
+            state: 'critical',
+            description: "HTTP connection error: #{response.status} - #{response.body}",
+          )
+        else
+          # Assuming that a 200 will give json
+          json = JSON.parse(response.body)
+          state = 'ok'
+
+          report(
+            host: uri.host,
+            service: 'marathon health',
+            state: state,
+          )
+
+          json.each_pair do |t, d|
+            next unless d.respond_to? :each_pair
+
+            d.each_pair do |service, counters|
+              report(
+                host: uri.host,
+                service: "marathon_metric #{t} #{service}",
+                metric: 1,
+                tags: ['metric_name'],
+                ttl: 600,
+              )
+              next unless counters.respond_to? :each_pair
 
-  def tick
-    tick_health
-    tick_apps
-  end
-  
-  def tick_health
-    uri = URI(health_url)
-    response = safe_get(uri)
-
-    return if response.nil?
-
-    if response.status != 200
-        report(:host => uri.host,
-          :service => "marathon health",
-          :state => "critical",
-          :description => "HTTP connection error: #{response.status} - #{response.body}"
-        )
-    else
-      # Assuming that a 200 will give json
-      json = JSON.parse(response.body)
-      state = "ok"
-
-      report(:host => uri.host,
-             :service => "marathon health",
-             :state => state)
-
-      json.each_pair do |t, d|
-        if d.respond_to? :each_pair
-          d.each_pair do |service, counters|
-            report(:host => uri.host,
-                   :service => "marathon_metric #{t} #{service}",
-                   :metric => 1,
-                   :tags => ["metric_name"],
-                   :ttl => 600
-            )
-            if counters.respond_to? :each_pair
               counters.each_pair do |k, v|
-                if v.is_a? Numeric
-                  report(:host => uri.host,
-                         :service => "marathon #{service} #{k}",
-                         :metric => v,
-                         :tags => ["metric", "#{t}"],
-                         :ttl => 600
-                  )
-                end
+                next unless v.is_a? Numeric
+
+                report(
+                  host: uri.host,
+                  service: "marathon #{service} #{k}",
+                  metric: v,
+                  tags: ['metric', t.to_s],
+                  ttl: 600,
+                )
               end
             end
           end
         end
       end
-    end
-  end
 
-  def tick_apps
-    uri = URI(apps_url)
-    response = safe_get(uri)
-
-    return if response.nil?
-
-    if response.status != 200
-        report(:host => uri.host,
-          :service => "marathon health",
-          :state => "critical",
-          :description => "HTTP connection error: #{response.status} - #{response.body}"
-        )
-    else
-      # Assuming that a 200 will give json
-      json = JSON.parse(response.body)
-      state = "ok"
-
-      report(:host => uri.host,
-             :service => "marathon health",
-             :state => state)
-
-      json["apps"].each do |app|
-        app.each_pair do |k, v|
-          if v.is_a? Numeric
-            report(:host => uri.host,
-                   :service => "marathon apps#{app["id"]}/#{k}",
-                   :metric => v,
-                   :ttl => 120
-            )
+      def tick_apps
+        uri = URI(apps_url)
+        response = safe_get(uri)
+
+        return if response.nil?
+
+        if response.status != 200
+          report(
+            host: uri.host,
+            service: 'marathon health',
+            state: 'critical',
+            description: "HTTP connection error: #{response.status} - #{response.body}",
+          )
+        else
+          # Assuming that a 200 will give json
+          json = JSON.parse(response.body)
+          state = 'ok'
+
+          report(
+            host: uri.host,
+            service: 'marathon health',
+            state: state,
+          )
+
+          json['apps'].each do |app|
+            app.each_pair do |k, v|
+              next unless v.is_a? Numeric
+
+              report(
+                host: uri.host,
+                service: "marathon apps#{app['id']}/#{k}",
+                metric: v,
+                ttl: 120,
+              )
+            end
           end
         end
       end
@@ -144,4 +161,3 @@ class Riemann::Tools::Marathon
   end
 end
 Riemann::Tools::Marathon.run
-

data/tools/riemann-mesos/{Rakefile.rb → Rakefile}

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rubygems'
 require 'rubygems/package_task'
 require 'rdoc/task'