Add a munin plugin for API response codes

[chef.git] / cookbooks / munin / files / default / plugins / api_calls_

diff --git a/cookbooks/munin/files/default/plugins/api_calls_ b/cookbooks/munin/files/default/plugins/api_calls_
index 8b4dd770da199b8e63271a39c4c57827eb5f2232..74c5bbe16895dc53d5e28f6656edbc8f22242b57 100755 (executable)
--- a/cookbooks/munin/files/default/plugins/api_calls_
+++ b/cookbooks/munin/files/default/plugins/api_calls_
@@ -9,10 +9,10 @@ def uris_from_status(server)
   file = open("http://#{server}/server-status").read
   doc = Hpricot.parse(file)
   tables = doc / "table"
   file = open("http://#{server}/server-status").read
   doc = Hpricot.parse(file)
   tables = doc / "table"

-  rows = (tables[0] / "tr")[1..-1]
+  rows = (tables[1] / "tr")[1..-1]

   data = rows.collect { |r| (r  / "td").collect(&:inner_html) }
   # filter where the PID is numeric, status is 'W' and host matches the server
   data = rows.collect { |r| (r  / "td").collect(&:inner_html) }
   # filter where the PID is numeric, status is 'W' and host matches the server

-  matching_data = data.select { |r| (r[1].to_i > 0) && r[3].match(/W/) && r[11].match(server) }
+  matching_data = data.select { |r| r[1].to_i > 0 && r[3].match(/W/) && r[11].match(server) }

   # return only the URI part
   matching_data.collect { |r| r[12] }
 end
   # return only the URI part
   matching_data.collect { |r| r[12] }
 end


@@ -54,7 +54,7 @@ if ARGV[0] == "config"
 else
   counts = uris_from_status(server)
            .collect { |x| categorise_uri(x) }
 else
   counts = uris_from_status(server)
            .collect { |x| categorise_uri(x) }

-           .each_with_object({}) do |h, e|
+           .each_with_object({}) do |e, h|

     if h.key? e
       h[e] += 1
     else
     if h.key? e
       h[e] += 1
     else