]> git.openstreetmap.org Git - chef.git/blobdiff - cookbooks/munin/files/default/plugins/api_waits_
Fix some rubocop detected style issues
[chef.git] / cookbooks / munin / files / default / plugins / api_waits_
index 029c9f4a6d701abadd2ff11cd7ffffbcd0dde158..f74e3a5cdf2c7119ac02fbd562213c7fbf73a3d4 100755 (executable)
@@ -5,32 +5,32 @@ require 'date'
 require 'hpricot'
 require 'open-uri'
 
-def uri_and_times_from_status(server) 
+def uri_and_times_from_status(server)
   file = open("http://#{server}/server-status").read
   doc = Hpricot.parse(file)
   tables = doc / 'table'
   rows = (tables[0] / 'tr')[1..-1]
-  data = rows.collect {|r| (r  / 'td').collect {|x| x.inner_html} }
+  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 URI and number of seconds processing for each request
-  matching_data.collect {|r| [r[12], r[5].to_i]}
+  matching_data.collect { |r| [r[12], r[5].to_i] }
 end
 
 CALL_TYPES = {
-  :map => "Map API calls", 
-  :upload => "Changeset diff uploads", 
-  :amf => "AMF API calls", 
-  :history => "Element history fetches", 
+  :map => "Map API calls",
+  :upload => "Changeset diff uploads",
+  :amf => "AMF API calls",
+  :history => "Element history fetches",
   :full => "Full element fetches",
   :trkpts => "GPX trackpoints calls",
-  :web => "Web site traffic", 
+  :web => "Web site traffic",
   :other => "Other API calls"
 }
 
 def categorise_uri(line)
   uri = line.split(" ")[1]
-  
+
   case uri
   when /api\/0\.6\/map/ then :map
   when /api\/0\.6\/changeset\/[0-9]*\/upload/ then :upload
@@ -53,7 +53,7 @@ if ARGV[0] == 'config'
 
 else
   counts = uri_and_times_from_status(server).
-      collect {|x,y| [categorise_uri(x), y]}.
+      collect { |x, y| [categorise_uri(x), y] }.
       inject(Hash.new) do |h, e|
     category, time = e
     if h.has_key? category
@@ -63,10 +63,10 @@ else
     end
     h
   end
-  
+
   CALL_TYPES.keys.each do |type|
     count = counts[type] || [0]
-    avg = count.inject(0){|x,y|x+y} / (1.0 * count.length)
+    avg = count.inject(0) { |x, y|x + y } / (1.0 * count.length)
     puts "#{type}.value #{avg}"
   end
 end