diff --git a/.rubocop.yml b/.rubocop.yml
new file mode 100644
index 00000000..31bda550
--- /dev/null
+++ b/.rubocop.yml
@@ -0,0 +1,7 @@
+# Allow "=begin"/"=end" multiline comments in order to protect munin's
+# magic markers ("#%#").
+Style/BlockComments:
+  Enabled: false
+
+AllCops:
+  NewCops: enable
diff --git a/plugins/bsd/netstat_bsd_m_ b/plugins/bsd/netstat_bsd_m_
index 82b79fa1..2e359a23 100755
--- a/plugins/bsd/netstat_bsd_m_
+++ b/plugins/bsd/netstat_bsd_m_
@@ -9,24 +9,22 @@ This plugin shows various statistics from 'netstat -m'
 Required privileges: none
 
 OS:
-	Supposed: BSD
-	Tested:   FreeBSD 8.2
+  Supposed: BSD
+  Tested:   FreeBSD 8.2
 
 Author: Artem Sheremet <dot.doom@gmail.com>
 
-
 #%# family=auto
 #%# capabilities=autoconf suggest
 
 =end
 
-
 # original filename
-PLUGIN_NAME = 'netstat_bsd_m_'
+PLUGIN_NAME = 'netstat_bsd_m_'.freeze
 
 class String
   def escape
-    self.gsub /[^\w]/, '_'
+    gsub(/[^\w]/, '_')
   end
 
   unless method_defined? :start_with?
@@ -37,17 +35,20 @@ class String
 end
 
 def netstat_m(filter = nil)
-  Hash[`netstat -m`.split($/).map { |line|
-    if line =~ /^([\d\/K]+) (.*) \(([\w\/+]+)\)$/
+  Hash[`netstat -m`.split($/).map do |line|
+    if line =~ %r{^([\d/K]+) (.*) \(([\w/+]+)\)$}
       # 7891K/22385K/30276K bytes allocated to network (current/cache/total)
-      values, desc, names = $1, $2, $3
-      [desc, names.split('/').zip(values.split '/')] if filter.nil? or desc.escape == filter
+      values = Regexp.last_match(1)
+      desc = Regexp.last_match(2)
+      names = Regexp.last_match(3)
+      [desc, names.split('/').zip(values.split('/'))] if filter.nil? || (desc.escape == filter)
     elsif line =~ /^(\d+) (.*)$/
       # 12327 requests for I/O initiated by sendfile
-      value, desc = $1, $2
-      [desc, [[:value, value]]] if filter.nil? or desc.escape == filter
+      value = Regexp.last_match(1)
+      desc = Regexp.last_match(2)
+      [desc, [[:value, value]]] if filter.nil? || (desc.escape == filter)
     end
-  }.compact]
+  end.compact]
 end
 
 stat_name = File.basename($0, '.*').escape
@@ -74,7 +75,7 @@ when 'config'
     CONFIG
     puts values.map { |name, _|
       esc_name = name.to_s.escape
-      "#{esc_name}.draw " + if %w(total max).include? name
+      "#{esc_name}.draw " + if %w[total max].include? name
                               'LINE'
                             elsif stack
                               if first
diff --git a/plugins/ejabberd/ejabberd_scanlog b/plugins/ejabberd/ejabberd_scanlog
index c46f6869..a07b8266 100755
--- a/plugins/ejabberd/ejabberd_scanlog
+++ b/plugins/ejabberd/ejabberd_scanlog
@@ -21,9 +21,9 @@ require 'yaml'
 #
 
 LOG_FILE = ENV['log'] || '/var/log/ejabberd/ejabberd.log'
-CACHE_FILE = '/tmp/ejabberd_scanlog_cache' # cache file position
+CACHE_FILE = '/tmp/ejabberd_scanlog_cache'.freeze # cache file position
 
-DEFAULT_CACHE = { :start => 0 }
+DEFAULT_CACHE = { start: 0 }.freeze
 
 $debug_mode = ARGV.first == 'debug'
 
@@ -32,7 +32,7 @@ if $debug_mode
 else
   begin
     log_info = YAML.load IO.read(CACHE_FILE)
-  rescue
+  rescue StandardError
     log_info = DEFAULT_CACHE
   end
 
@@ -43,7 +43,7 @@ else
 end
 
 if ARGV.first == 'reset'
-  log_info = { :start => File.size(LOG_FILE) - 1 }
+  log_info = { start: File.size(LOG_FILE) - 1 }
   puts 'Log reset'
 end
 
@@ -80,8 +80,8 @@ KNOWN_LOG_TYPES = [
    ['mysql',
     'Failed connecting to']],
   ['Timeout while running a hook',
-   ['ejabberd_hooks',
-    'timeout']],
+   %w[ejabberd_hooks
+      timeout]],
   ['SQL transaction restarts exceeded',
    ['SQL transaction restarts exceeded']],
   ['Unexpected info',
@@ -89,8 +89,8 @@ KNOWN_LOG_TYPES = [
   ['Other sql_cmd timeout',
    ['sql_cmd']],
   ['System limit hit: ports', # check with length(erlang:ports())., set in ejabberdctl config file
-   ['system_limit',
-    'open_port']],
+   %w[system_limit
+      open_port]],
   ['Other system limit hit', # processes? check with erlang:system_info(process_count)., erlang:system_info(process_limit)., set in ejabberdctl cfg
    ['system_limit']],
   ['Generic server terminating',
@@ -115,20 +115,22 @@ KNOWN_LOG_TYPES = [
    ['mysql_conn: init failed receiving data']],
   ['TCP Error',
    ['Failed TCP']]
-]
+].freeze
 
 def log_type(text)
-  KNOWN_LOG_TYPES.find_index { |entry|
+  KNOWN_LOG_TYPES.find_index do |entry|
     entry[1].all? { |substr| text.include? substr }
-  }
+  end
 end
 
-new_data.split("\n=").each { |report|
+new_data.split("\n=").each do |report|
   next if report.empty?
 
   report =~ /\A(\w+) REPORT==== (.*) ===\n(.*)\z/m
-  type, time, text = $1, $2, $3
-  next unless type and time and text
+  type = Regexp.last_match(1)
+  time = Regexp.last_match(2)
+  text = Regexp.last_match(3)
+  next unless type && time && text
 
   log_info[type] = (log_info[type] || 0) + 1
   if sub_type = log_type(text)
@@ -136,7 +138,7 @@ new_data.split("\n=").each { |report|
   elsif $debug_mode
     warn "Unparsed log entry #{type}: #{text} at #{time}"
   end
-}
+end
 
 log_info[:start] += new_data.size
 File.open(CACHE_FILE, 'w') { |f| f.write log_info.to_yaml } unless $debug_mode
@@ -150,7 +152,7 @@ if ARGV.first == 'config'
   CONFIG
 end
 
-(KNOWN_LOG_TYPES + %w(ERROR WARNING INFO DEBUG)).each.with_index { |log_type, index|
+(KNOWN_LOG_TYPES + %w[ERROR WARNING INFO DEBUG]).each.with_index do |log_type, index|
   label, index = if log_type.is_a? Array
                    [log_type.first, index]
                  else
@@ -162,4 +164,4 @@ end
   else
     puts "T#{index}.value #{log_info[index] or 0}"
   end
-}
+end
diff --git a/plugins/http/mongrel_memory b/plugins/http/mongrel_memory
index 3fb70303..9e3b2798 100755
--- a/plugins/http/mongrel_memory
+++ b/plugins/http/mongrel_memory
@@ -34,33 +34,32 @@ Contributors: Adam Jacob (<adam@hjksolutions.com>)
 
 =end
 
-
 module Munin
   class MongrelProcessMemory
     def run
-      pid_port_map = get_pids()
-      port_list = Hash.new
+      pid_port_map = get_pids
+      port_list = {}
       pid_port_map.sort.each do |pid, port|
-        rss = `pmap -x #{pid} | grep total`.split(" ")[3]
+        rss = `pmap -x #{pid} | grep total`.split(' ')[3]
         puts "mongrel_#{port}.value #{rss}"
       end
     end
 
     def get_pids
-      h = Hash.new
+      h = {}
       pids = []
       pids += `pgrep mongrel_rails`.split("\n")
       pids += `pgrep ruby`.split("\n")
-      pids.each { |pid|
+      pids.each do |pid|
         l = `pargs -l #{pid}`
         l =~ /-p (\d+)/
-        h[pid] = $1 if $1
-      }
+        h[pid] = Regexp.last_match(1) if Regexp.last_match(1)
+      end
       h
     end
 
     def autoconf
-      get_pids().length > 0
+      get_pids.length > 0
     end
   end
 end
@@ -68,25 +67,25 @@ end
 mpm = Munin::MongrelProcessMemory.new
 
 case ARGV[0]
-when "config"
-  puts "graph_title Mongrel Memory"
-  puts "graph_vlabel RSS"
-  puts "graph_category memory"
-  puts "graph_args --base 1024 -l 0"
-  puts "graph_scale yes"
-  puts "graph_info Tracks the size of individual mongrel processes"
+when 'config'
+  puts 'graph_title Mongrel Memory'
+  puts 'graph_vlabel RSS'
+  puts 'graph_category memory'
+  puts 'graph_args --base 1024 -l 0'
+  puts 'graph_scale yes'
+  puts 'graph_info Tracks the size of individual mongrel processes'
   mpm.get_pids.values.sort.each do |port|
     puts "mongrel_#{port}.label mongrel_#{port}"
     puts "mongrel_#{port}.info Process memory"
     puts "mongrel_#{port}.type GAUGE"
     puts "mongrel_#{port}.min 0"
   end
-when "autoconf"
+when 'autoconf'
   if mpm.autoconf
-    puts "yes"
+    puts 'yes'
     exit 0
   end
-  puts "no"
+  puts 'no'
   exit 0
 else
   mpm.run
diff --git a/plugins/http/mongrel_process_memory b/plugins/http/mongrel_process_memory
index fae85415..89030874 100755
--- a/plugins/http/mongrel_process_memory
+++ b/plugins/http/mongrel_process_memory
@@ -28,27 +28,24 @@ Contributors: Adam Jacob (<adam@hjksolutions.com>)
 
 =end
 
-
 module Munin
   class MongrelProcessMemory
     def run
-      h = get_pids()
-      ps_output = ""
+      h = get_pids
+      ps_output = ''
       # I have no doubt that this is a terrible way of doing this.
-      h.each do |k, v|
-        ps_output = ps_output + `ps --no-heading l #{k}`
+      h.each do |k, _v|
+        ps_output += `ps --no-heading l #{k}`
       end
 
       if ps_output
-        port_list = Hash.new
+        port_list = {}
         ps_output.each_line do |l|
-          if l =~ /-p (\d+)/
-            port = $1
-            l_ary = l.split(/\s+/)
-            if l_ary.length > 6
-              port_list[port] = l_ary[7].to_i * 1024
-            end
-          end
+          next unless l =~ /-p (\d+)/
+
+          port = Regexp.last_match(1)
+          l_ary = l.split(/\s+/)
+          port_list[port] = l_ary[7].to_i * 1024 if l_ary.length > 6
         end
         port_list.sort.each do |port|
           puts "mongrel_#{port[0]}.value #{port[1]}"
@@ -58,14 +55,14 @@ module Munin
     end
 
     def get_pids
-      h = Hash.new
+      h = {}
       pids = []
       pids = `pgrep mongrel_rails`
-      pids.each { |p|
+      pids.each do |p|
         l = `ps #{p}`
         l =~ /-p (\d+)/
-        h[p] = $1
-      }
+        h[p] = Regexp.last_match(1)
+      end
       h
     end
 
@@ -78,25 +75,25 @@ end
 mpm = Munin::MongrelProcessMemory.new
 
 case ARGV[0]
-when "config"
-  puts "graph_title Mongrel Memory"
-  puts "graph_vlabel RSS"
-  puts "graph_category memory"
-  puts "graph_args --base 1024 -l 0"
-  puts "graph_scale yes"
-  puts "graph_info Tracks the size of individual mongrel processes"
+when 'config'
+  puts 'graph_title Mongrel Memory'
+  puts 'graph_vlabel RSS'
+  puts 'graph_category memory'
+  puts 'graph_args --base 1024 -l 0'
+  puts 'graph_scale yes'
+  puts 'graph_info Tracks the size of individual mongrel processes'
   mpm.get_pids.values.sort.each do |port|
     puts "mongrel_#{port}.label mongrel_#{port}"
     puts "mongrel_#{port}.info Process memory"
     puts "mongrel_#{port}.type GAUGE"
     puts "mongrel_#{port}.min 0"
   end
-when "autoconf"
+when 'autoconf'
   if mpm.autoconf
-    puts "yes"
+    puts 'yes'
     exit 0
   end
-  puts "no"
+  puts 'no'
   exit 0
 else
   mpm.run
diff --git a/plugins/icecast/icecast2_simple b/plugins/icecast/icecast2_simple
index 543078e5..86c2b38b 100755
--- a/plugins/icecast/icecast2_simple
+++ b/plugins/icecast/icecast2_simple
@@ -18,10 +18,10 @@ require 'open-uri'
 
 def get_conf
   # Default values
-  conf = { :host => '127.0.0.1', :port => 8000,
-           :username => 'admin', :password => 'hackme' }
+  conf = { host: '127.0.0.1', port: 8000,
+           username: 'admin', password: 'hackme' }
   conf.keys.each do |key|
-    env_key = sprintf('icecast_%s', key)
+    env_key = format('icecast_%s', key)
     conf[key] = ENV[env_key] if ENV.has_key?(env_key)
   end
   conf
@@ -29,12 +29,12 @@ end
 
 def get_data(conf)
   begin
-    data = Hpricot(open(sprintf('http://%s:%s/admin/stats',
-                                conf[:host], conf[:port]),
-                        :http_basic_authentication => [conf[:username],
-                                                       conf[:password]]))
+    data = Hpricot(open(format('http://%s:%s/admin/stats',
+                               conf[:host], conf[:port]),
+                        http_basic_authentication: [conf[:username],
+                                                    conf[:password]]))
   rescue OpenURI::HTTPError
-    puts "Cannot connect: HTTP connection error"
+    puts 'Cannot connect: HTTP connection error'
     exit 1
   end
   data
@@ -42,13 +42,13 @@ end
 
 def get_values(data)
   vals = {}
-  [:sources, :clients].each do |key|
+  %i[sources clients].each do |key|
     elem = data / key
-    if elem.nil?
-      vals[key] = 0
-    else
-      vals[key] = elem.innerHTML
-    end
+    vals[key] = if elem.nil?
+                  0
+                else
+                  elem.innerHTML
+                end
   end
   vals
 end
@@ -56,15 +56,16 @@ end
 data = get_data(get_conf)
 vals = get_values(data)
 
-if ARGV[0] == 'autoconf'
+case ARGV[0]
+when 'autoconf'
   puts 'yes'
-elsif ARGV[0] == 'config'
-  puts "graph_title Total sources and clients for Icecast"
-  puts "graph_vlabel listeners"
-  puts "graph_category streaming"
-  puts "sources.label Total number of sources"
-  puts "clients.label Total number of clients"
+when 'config'
+  puts 'graph_title Total sources and clients for Icecast'
+  puts 'graph_vlabel listeners'
+  puts 'graph_category streaming'
+  puts 'sources.label Total number of sources'
+  puts 'clients.label Total number of clients'
 else
-  puts "sources.value " + vals[:sources]
-  puts "clients.value " + vals[:clients]
+  puts 'sources.value ' + vals[:sources]
+  puts 'clients.value ' + vals[:clients]
 end
diff --git a/plugins/minecraft/minecraft-users b/plugins/minecraft/minecraft-users
index 045937d5..c61d0aa8 100755
--- a/plugins/minecraft/minecraft-users
+++ b/plugins/minecraft/minecraft-users
@@ -8,29 +8,29 @@
 require 'socket'
 
 if ARGV[0] == 'config'
-  puts "graph_title Connected players"
-  puts "graph_vlabel players"
-  puts "players.label players"
-  puts "graph_info Number of players connected to Minecraft"
-  puts "graph_category games"
+  puts 'graph_title Connected players'
+  puts 'graph_vlabel players'
+  puts 'players.label players'
+  puts 'graph_info Number of players connected to Minecraft'
+  puts 'graph_category games'
   exit
 end
 
 host = ENV['host']
-host = 'localhost' unless host
+host ||= 'localhost'
 
 port = ENV['port']
-port = '25566' unless port
+port ||= '25566'
 
 socket = TCPSocket.new(host, port)
-socket.puts "QUERY"
+socket.puts 'QUERY'
 
 response = socket.read
 response = response.split("\n")
 
-server_port = response[0].split(" ", 2)[1].to_i
-player_count = response[1].split(" ", 2)[1].to_i
-max_players = response[2].split(" ", 2)[1].to_i
-player_list = response[3].split(" ", 2)[1].chomp[1..-2]
+server_port = response[0].split(' ', 2)[1].to_i
+player_count = response[1].split(' ', 2)[1].to_i
+max_players = response[2].split(' ', 2)[1].to_i
+player_list = response[3].split(' ', 2)[1].chomp[1..-2]
 
 puts "players.value #{player_count}"
diff --git a/plugins/moblock/moblock_connections b/plugins/moblock/moblock_connections
index 0c68f2ef..9489e953 100755
--- a/plugins/moblock/moblock_connections
+++ b/plugins/moblock/moblock_connections
@@ -24,44 +24,43 @@ Magic markers
 
 =end
 
-
 #
 # Initialize vars
 #
-$logfile = ENV['logfile'] || "/var/log/moblock.log"
+$logfile = ENV['logfile'] || '/var/log/moblock.log'
 
 #
 # Configure generated graph
 #
 def config
-  puts "graph_args --base 1000 -r --lower-limit 0"
-  puts "graph_title Moblock"
-  puts "graph_vlabel Blocked Connections"
-  puts "graph_category fw"
-  puts "graph_info This graph shows the number of connections blocked by Moblock"
+  puts 'graph_args --base 1000 -r --lower-limit 0'
+  puts 'graph_title Moblock'
+  puts 'graph_vlabel Blocked Connections'
+  puts 'graph_category fw'
+  puts 'graph_info This graph shows the number of connections blocked by Moblock'
 
-  puts "blocked_in.label Blocked In"
-  puts "blocked_in.draw LINE1"
-  puts "blocked_in.info Number of blocked incoming connections"
-  puts "blocked_in.type GAUGE"
+  puts 'blocked_in.label Blocked In'
+  puts 'blocked_in.draw LINE1'
+  puts 'blocked_in.info Number of blocked incoming connections'
+  puts 'blocked_in.type GAUGE'
 
-  puts "blocked_out.label Blocked Out"
-  puts "blocked_out.draw LINE1"
-  puts "blocked_out.info Number of blocked outgoing connections"
-  puts "blocked_out.type GAUGE"
+  puts 'blocked_out.label Blocked Out'
+  puts 'blocked_out.draw LINE1'
+  puts 'blocked_out.info Number of blocked outgoing connections'
+  puts 'blocked_out.type GAUGE'
 
-  puts "blocked_total.label Total Blocked"
-  puts "blocked_total.draw LINE1"
-  puts "blocked_total.info Total Number of blocked connections"
-  puts "blocked_total.type GAUGE"
+  puts 'blocked_total.label Total Blocked'
+  puts 'blocked_total.draw LINE1'
+  puts 'blocked_total.info Total Number of blocked connections'
+  puts 'blocked_total.type GAUGE'
 end
 
 #
 # Grep moblock logs for stats
 #
-def fetch(debug = false)
-  num_in = %x{cat #{$logfile} | grep --extended-regexp 'IN: ' | wc -l}
-  num_out = %x{cat #{$logfile} | grep --extended-regexp 'OUT: ' | wc -l}
+def fetch(_debug = false)
+  num_in = `cat #{$logfile} | grep --extended-regexp 'IN: ' | wc -l`
+  num_out = `cat #{$logfile} | grep --extended-regexp 'OUT: ' | wc -l`
   num_total = num_in.to_i + num_out.to_i
 
   puts "blocked_in.value #{num_in}"
@@ -73,11 +72,11 @@ end
 # If moblock executable on path then allow autoconfiguration
 #
 def autoconf
-  moblock_path = %x{which moblock}
+  moblock_path = `which moblock`
   if moblock_path.index('moblock')
-    puts "yes"
+    puts 'yes'
   else
-    puts "no"
+    puts 'no'
   end
 end
 
diff --git a/plugins/mssql/microsoft-sql b/plugins/mssql/microsoft-sql
index 42e12d48..e52348e3 100755
--- a/plugins/mssql/microsoft-sql
+++ b/plugins/mssql/microsoft-sql
@@ -21,34 +21,32 @@ You should have received a copy of the GNU General Public License along
 with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
-
 Prerequistes:
-	1) /etc/odbc.ini and /etc/freetds.conf
-	2) rubygems
-	3) ruby-dbi
+  1) /etc/odbc.ini and /etc/freetds.conf
+  2) rubygems
+  3) ruby-dbi
 
 Usage:
-	1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
-	2) chmod to allow executable to others
-	3) create symbolic link in /etc/munin/plugins
+  1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
+  2) chmod to allow executable to others
+  3) create symbolic link in /etc/munin/plugins
           ln -s /usr/share/munin/plugins/mssql_transaction.rb /etc/munin/plugins/mssql_transaction.rb
 
 Parameters:
-	autoconf
-	config (required)
+  autoconf
+  config (required)
 
 Config variables:
-	sqluser : mssql user who has view server state privilege
-	sqlpass : password for the mssql user
-	dsn     : datasource name as defined in /etc/odbc.ini
-	instance: instance to monitor
+  sqluser : mssql user who has view server state privilege
+  sqlpass : password for the mssql user
+  dsn     : datasource name as defined in /etc/odbc.ini
+  instance: instance to monitor
 
 #%# family=auto
 #%# capabilities=autoconf
 
 =end
 
-
 require 'rubygems'
 require 'dbi'
 
@@ -74,7 +72,7 @@ transaction_query = "select cntr_value from sys.dm_os_performance_counters
                      and object_name = 'SQLServer:Databases'
                      and instance_name = ?"
 
-all_instance_names = Array.new
+all_instance_names = []
 sth = dbh.execute(instance_name_query)
 sth.fetch do |row|
   all_instance_names.push(row[0].strip)
@@ -84,25 +82,26 @@ sth.finish
 #
 # autoconf
 #
-if ARGV[0] == "autoconf"
+case ARGV[0]
+when 'autoconf'
   if all_instance_names.length > 1 && sqluser.length > 1 && sqlpass.length > 1
-    puts "yes"
+    puts 'yes'
   else
-    puts "no"
+    puts 'no'
     puts "Usage: #{__FILE__} autoconf|conf"
   end
   exit 0
 #
 # config definition
 #
-elsif ARGV[0] == "config"
-  puts "graph_args --base 1000 -r --lower-limit 0"
-  puts "graph_title MSSQL Transactions/s"
-  puts "graph_category db"
-  puts "graph_info This graph shows transactions/s"
-  puts "graph_vlabel transactions/s"
-  puts "graph_scale no"
-  puts "graph_period second"
+when 'config'
+  puts 'graph_args --base 1000 -r --lower-limit 0'
+  puts 'graph_title MSSQL Transactions/s'
+  puts 'graph_category db'
+  puts 'graph_info This graph shows transactions/s'
+  puts 'graph_vlabel transactions/s'
+  puts 'graph_scale no'
+  puts 'graph_period second'
 
   all_instance_names.sort.each do |s|
     puts "#{s}.label #{s}"
@@ -122,7 +121,7 @@ all_instance_names.sort.each do |k|
   sth.execute(k)
   sth.fetch do |row|
     # since type is DERIVE, need to convert value to integer then to string
-    puts "#{k.to_s}.value #{row[0].to_i.to_s}"
+    puts "#{k}.value #{row[0].to_i}"
   end
 end
 sth.finish
diff --git a/plugins/mssql/microsoft-sql-buffer-cache-hit-ratio b/plugins/mssql/microsoft-sql-buffer-cache-hit-ratio
index 8e225617..b7b83d89 100755
--- a/plugins/mssql/microsoft-sql-buffer-cache-hit-ratio
+++ b/plugins/mssql/microsoft-sql-buffer-cache-hit-ratio
@@ -21,33 +21,31 @@ You should have received a copy of the GNU General Public License along
 with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
-
 Prerequistes:
-	1) /etc/odbc.ini and /etc/freetds.conf
-	2) rubygems
-	3) ruby-dbi
+  1) /etc/odbc.ini and /etc/freetds.conf
+  2) rubygems
+  3) ruby-dbi
 
 Usage:
-	1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
-	2) chmod to allow executable to others
-	3) create symbolic link in /etc/munin/plugins
+  1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
+  2) chmod to allow executable to others
+  3) create symbolic link in /etc/munin/plugins
            ln -s /usr/share/munin/plugins/mssql_buffercachehitratio.rb /etc/munin/plugins/mssql_buffercachehitratio.rb
 
 Parameters:
-	autoconf
-	config (required)
+  autoconf
+  config (required)
 
 Config variables:
-	sqluser : mssql user who has view server state privilege
-	sqlpass : password for the mssql user
-	dsn     : datasource name as defined in /etc/odbc.ini
+  sqluser : mssql user who has view server state privilege
+  sqlpass : password for the mssql user
+  dsn     : datasource name as defined in /etc/odbc.ini
 
 #%# family=auto
 #%# capabilities=autoconf
 
 =end
 
-
 require 'rubygems'
 require 'dbi'
 
@@ -74,37 +72,38 @@ buffercachehitratio_query = "select (a.cntr_value * 1.0 / b.cntr_value) * 100.0
 #
 # autoconf
 #
-if ARGV[0] == "autoconf"
+case ARGV[0]
+when 'autoconf'
   if all_instance_names.length > 1 && sqluser.length > 1 && sqlpass.length > 1
-    puts "yes"
+    puts 'yes'
   else
-    puts "no"
+    puts 'no'
     puts "Usage: #{__FILE__} autoconf|conf"
   end
   exit 0
 #
 # config definition
 #
-elsif ARGV[0] == "config"
-  puts "graph_args --base 1000 -r --lower-limit 0"
-  puts "graph_title MSSQL Buffer Cache Hit Ratio "
-  puts "graph_category db"
-  puts "graph_info This graph shows Buffer Cache Hit Ratio"
-  puts "graph_vlabel %"
-  puts "graph_scale no"
-  puts "graph_period second"
+when 'config'
+  puts 'graph_args --base 1000 -r --lower-limit 0'
+  puts 'graph_title MSSQL Buffer Cache Hit Ratio '
+  puts 'graph_category db'
+  puts 'graph_info This graph shows Buffer Cache Hit Ratio'
+  puts 'graph_vlabel %'
+  puts 'graph_scale no'
+  puts 'graph_period second'
 
-  puts "bc_hitratio.label BufferCacheHitRatio"
-  puts "bc_hitratio.info BufferCacheHitRatio"
-  puts "bc_hitratio.type GAUGE"
-  puts "bc_hitratio.draw LINE1"
+  puts 'bc_hitratio.label BufferCacheHitRatio'
+  puts 'bc_hitratio.info BufferCacheHitRatio'
+  puts 'bc_hitratio.type GAUGE'
+  puts 'bc_hitratio.draw LINE1'
 
   exit 0
 end
 
 sth = dbh.execute(buffercachehitratio_query)
 sth.fetch do |row|
-  puts "bc_hitratio.value #{row[0].strip.to_s}"
+  puts "bc_hitratio.value #{row[0].strip}"
 end
 sth.finish
 dbh.disconnect
diff --git a/plugins/mssql/microsoft-sql-data-file-sizes b/plugins/mssql/microsoft-sql-data-file-sizes
index 4d4b78db..915ebc2d 100755
--- a/plugins/mssql/microsoft-sql-data-file-sizes
+++ b/plugins/mssql/microsoft-sql-data-file-sizes
@@ -21,34 +21,32 @@ You should have received a copy of the GNU General Public License along
 with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
-
 Prerequistes:
-	1) /etc/odbc.ini and /etc/freetds.conf
-	2) rubygems
-	3) ruby-dbi
+  1) /etc/odbc.ini and /etc/freetds.conf
+  2) rubygems
+  3) ruby-dbi
 
 Usage:
-	1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
-	2) chmod to allow executable to others
-	3) create symbolic link in /etc/munin/plugins
+  1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
+  2) chmod to allow executable to others
+  3) create symbolic link in /etc/munin/plugins
            ln -s /usr/share/munin/plugins/mssql_datafilesizes.rb /etc/munin/plugins/mssql_datafilesizes.rb
 
 Parameters:
-	autoconf
-	config (required)
+  autoconf
+  config (required)
 
 Config variables:
-	sqluser : mssql user who has view server state privilege
-	sqlpass : password for the mssql user
-	dsn     : datasource name as defined in /etc/odbc.ini
-	instance: instance to monitor
+  sqluser : mssql user who has view server state privilege
+  sqlpass : password for the mssql user
+  dsn     : datasource name as defined in /etc/odbc.ini
+  instance: instance to monitor
 
 #%# family=auto
 #%# capabilities=autoconf
 
 =end
 
-
 require 'rubygems'
 require 'dbi'
 
@@ -72,7 +70,7 @@ transaction_query = "select cntr_value/1024.0 from sys.dm_os_performance_counter
                      and object_name = 'SQLServer:Databases'
                      and instance_name = ?"
 
-all_instance_names = Array.new
+all_instance_names = []
 sth = dbh.execute(instance_name_query)
 sth.fetch do |row|
   all_instance_names.push(row[0].strip)
@@ -82,25 +80,26 @@ sth.finish
 #
 # autoconf
 #
-if ARGV[0] == "autoconf"
+case ARGV[0]
+when 'autoconf'
   if all_instance_names.length > 1 && sqluser.length > 1 && sqlpass.length > 1
-    puts "yes"
+    puts 'yes'
   else
-    puts "no"
+    puts 'no'
     puts "Usage: #{__FILE__} autoconf|conf"
   end
   exit 0
 #
 # config definition
 #
-elsif ARGV[0] == "config"
-  puts "graph_args --base 1024k -r --lower-limit 0"
-  puts "graph_title MSSQL DB File Sizes"
-  puts "graph_category db"
-  puts "graph_info This graph shows DB File Sizes (MB)"
-  puts "graph_vlabel MB"
-  puts "graph_scale no"
-  puts "graph_period second"
+when 'config'
+  puts 'graph_args --base 1024k -r --lower-limit 0'
+  puts 'graph_title MSSQL DB File Sizes'
+  puts 'graph_category db'
+  puts 'graph_info This graph shows DB File Sizes (MB)'
+  puts 'graph_vlabel MB'
+  puts 'graph_scale no'
+  puts 'graph_period second'
 
   all_instance_names.sort.each do |s|
     puts "#{s}.label #{s}"
@@ -119,7 +118,7 @@ sth = dbh.prepare(transaction_query)
 all_instance_names.sort.each do |k|
   sth.execute(k)
   sth.fetch do |row|
-    puts "#{k.to_s}.value #{row[0].to_s}"
+    puts "#{k}.value #{row[0]}"
   end
 end
 sth.finish
diff --git a/plugins/mssql/microsoft-sql-log-file-size b/plugins/mssql/microsoft-sql-log-file-size
index 46e777e1..813fac01 100755
--- a/plugins/mssql/microsoft-sql-log-file-size
+++ b/plugins/mssql/microsoft-sql-log-file-size
@@ -21,34 +21,32 @@ You should have received a copy of the GNU General Public License along
 with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
-
 Prerequistes:
-	1) /etc/odbc.ini and /etc/freetds.conf
-	2) rubygems
-	3) ruby-dbi
+  1) /etc/odbc.ini and /etc/freetds.conf
+  2) rubygems
+  3) ruby-dbi
 
 Usage:
-	1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
-	2) chmod to allow executable to others
-	3) create symbolic link in /etc/munin/plugins
+  1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
+  2) chmod to allow executable to others
+  3) create symbolic link in /etc/munin/plugins
            ln -s /usr/share/munin/plugins/mssql_logfilesizes.rb /etc/munin/plugins/mssql_logfilesizes.rb
 
 Parameters:
-	autoconf
-	config (required)
+  autoconf
+  config (required)
 
 Config variables:
-	sqluser : mssql user who has view server state privilege
-	sqlpass : password for the mssql user
-	dsn     : datasource name as defined in /etc/odbc.ini
-	instance: instance to monitor
+  sqluser : mssql user who has view server state privilege
+  sqlpass : password for the mssql user
+  dsn     : datasource name as defined in /etc/odbc.ini
+  instance: instance to monitor
 
 #%# family=auto
 #%# capabilities=autoconf
 
 =end
 
-
 require 'rubygems'
 require 'dbi'
 
@@ -72,7 +70,7 @@ logfilesize_query = "SELECT cntr_value/1024.0 from sys.dm_os_performance_counter
                      AND object_name = 'SQLServer:Databases'
                      AND instance_name = ?"
 
-all_instance_names = Array.new
+all_instance_names = []
 sth = dbh.execute(instance_name_query)
 sth.fetch do |row|
   all_instance_names.push(row[0].strip)
@@ -82,25 +80,26 @@ sth.finish
 #
 # autoconf
 #
-if ARGV[0] == "autoconf"
+case ARGV[0]
+when 'autoconf'
   if all_instance_names.length > 1 && sqluser.length > 1 && sqlpass.length > 1
-    puts "yes"
+    puts 'yes'
   else
-    puts "no"
+    puts 'no'
     puts "Usage: #{__FILE__} autoconf|conf"
   end
   exit 0
 #
 # config definition
 #
-elsif ARGV[0] == "config"
-  puts "graph_args --base 1024k -r --lower-limit 0"
-  puts "graph_title MSSQL DB Log File Sizes"
-  puts "graph_category db"
-  puts "graph_info This graph shows DB Log File Sizes (MB)"
-  puts "graph_vlabel MB"
-  puts "graph_scale no"
-  puts "graph_period second"
+when 'config'
+  puts 'graph_args --base 1024k -r --lower-limit 0'
+  puts 'graph_title MSSQL DB Log File Sizes'
+  puts 'graph_category db'
+  puts 'graph_info This graph shows DB Log File Sizes (MB)'
+  puts 'graph_vlabel MB'
+  puts 'graph_scale no'
+  puts 'graph_period second'
 
   all_instance_names.sort.each do |s|
     puts "#{s}.label #{s}"
@@ -116,7 +115,7 @@ sth = dbh.prepare(logfilesize_query)
 all_instance_names.sort.each do |k|
   sth.execute(k)
   sth.fetch do |row|
-    puts "#{k.to_s}.value #{row[0].to_s}"
+    puts "#{k}.value #{row[0]}"
   end
 end
 sth.finish
diff --git a/plugins/network/http__tp_link b/plugins/network/http__tp_link
index 2214b8e6..06fa489b 100755
--- a/plugins/network/http__tp_link
+++ b/plugins/network/http__tp_link
@@ -1,42 +1,40 @@
 #!/usr/bin/env ruby
 
-if $0 =~ /^(?:|.*\/)http_([^_]+)_/
-  host = $1
-end
+host = Regexp.last_match(1) if $0 =~ %r{^(?:|.*/)http_([^_]+)_}
 abort "# Error: couldn't understand what I'm supposed to monitor." unless host
 
 user = ENV['user'] || 'user'
 password = ENV['password'] || 'user'
 
-if (ARGV[0] == 'config')
+if ARGV[0] == 'config'
   puts "host_name #{host}" unless host == 'localhost'
 
-  puts "multigraph dsl_rate"
-  puts "graph_title DSL line speed"
-  puts "graph_args --base 1000 -l 0"
-  puts "graph_vlabel bps"
-  puts "graph_category network"
-  puts "downstream.label downstream"
-  puts "downstream.type GAUGE"
-  puts "downstream.min 0"
-  puts "downstream.cdef downstream,1000,*"
-  puts "upstream.label upstream"
-  puts "upstream.type GAUGE"
-  puts "upstream.min 0"
-  puts "upstream.cdef upstream,1000,*"
+  puts 'multigraph dsl_rate'
+  puts 'graph_title DSL line speed'
+  puts 'graph_args --base 1000 -l 0'
+  puts 'graph_vlabel bps'
+  puts 'graph_category network'
+  puts 'downstream.label downstream'
+  puts 'downstream.type GAUGE'
+  puts 'downstream.min 0'
+  puts 'downstream.cdef downstream,1000,*'
+  puts 'upstream.label upstream'
+  puts 'upstream.type GAUGE'
+  puts 'upstream.min 0'
+  puts 'upstream.cdef upstream,1000,*'
 
-  puts "multigraph dsl_snr"
-  puts "graph_title DSL SNR"
-  puts "graph_args --base 1000 -l 0"
-  puts "graph_vlabel dB"
-  puts "graph_scale no"
-  puts "graph_category network"
-  puts "downstream.label downstream"
-  puts "downstream.type GAUGE"
-  puts "downstream.min 0"
-  puts "upstream.label upstream"
-  puts "upstream.type GAUGE"
-  puts "upstream.min 0"
+  puts 'multigraph dsl_snr'
+  puts 'graph_title DSL SNR'
+  puts 'graph_args --base 1000 -l 0'
+  puts 'graph_vlabel dB'
+  puts 'graph_scale no'
+  puts 'graph_category network'
+  puts 'downstream.label downstream'
+  puts 'downstream.type GAUGE'
+  puts 'downstream.min 0'
+  puts 'upstream.label upstream'
+  puts 'upstream.type GAUGE'
+  puts 'upstream.min 0'
 
   exit 0
 end
@@ -56,21 +54,21 @@ class TPAdslStats
 
   def field_values(label)
     if @html =~ />#{label}.*?<td>([0-9.]+).*?([0-9.]+)/m
-      [$1, $2]
+      [Regexp.last_match(1), Regexp.last_match(2)]
     else
-      ['U', 'U']
+      %w[U U]
     end
   end
 end
 
 stats = TPAdslStats.new(host, user, password)
 
-puts "multigraph dsl_rate"
+puts 'multigraph dsl_rate'
 rate = stats.field_values('Rate')
 puts "downstream.value #{rate[0]}"
 puts "upstream.value #{rate[1]}"
 
-puts "multigraph dsl_snr"
+puts 'multigraph dsl_snr'
 snr = stats.field_values('SNR')
 puts "downstream.value #{snr[0]}"
 puts "upstream.value #{snr[1]}"
diff --git a/plugins/network/netstat_s_ b/plugins/network/netstat_s_
index d5764ef7..2aa209de 100755
--- a/plugins/network/netstat_s_
+++ b/plugins/network/netstat_s_
@@ -9,27 +9,26 @@ This plugin shows various statistics from 'netstat -s'
 Required privileges: none
 
 OS:
-	Supposed: BSD, Linux (only a few items, see netstat_multi for more)
-	Tested:   FreeBSD: 8.2, 8.3, 9.1
-	          Linux  : Debian 6 (kernel 2.6.32), Arch (kernel 3.11.6), CentOS 6
+  Supposed: BSD, Linux (only a few items, see netstat_multi for more)
+  Tested:   FreeBSD: 8.2, 8.3, 9.1
+            Linux  : Debian 6 (kernel 2.6.32), Arch (kernel 3.11.6), CentOS 6
 
 Author: Artem Sheremet <dot.doom@gmail.com>
 
-
 #%# family=auto
 #%# capabilities=autoconf suggest
 
 =end
 
 # original filename
-PLUGIN_NAME = 'netstat_s_'
+PLUGIN_NAME = 'netstat_s_'.freeze
 
 $os = `uname -s`.strip.downcase.to_sym
 $debug_mode = ARGV.first == 'debug'
 
 class String
   def escape
-    self.gsub /[^\w]/, '_'
+    gsub(/[^\w]/, '_')
   end
 
   unless method_defined? :start_with?
@@ -40,14 +39,16 @@ class String
 
   unless method_defined? :lines
     def lines
-      self.split($/).to_enum
+      split($/).to_enum
     end
   end
 end
 
 class Graph
   def initialize(name, protocol, parse_expr)
-    @name, @protocol, @parse_expr = name, protocol, parse_expr
+    @name = name
+    @protocol = protocol
+    @parse_expr = parse_expr
   end
 
   def config
@@ -56,31 +57,31 @@ class Graph
     # first, build a list of multigraphs (one graph per unit)
     # Hash key is unit, and the value is array of labels
     multigraphs = {}
-    @parse_expr.each { |expr, descr|
+    @parse_expr.each do |_expr, descr|
       next unless descr # no label - skip this entry
 
-      descr.each { |entry|
+      descr.each do |entry|
         labels_array = (multigraphs[entry[0]] ||= [])
         labels_array.push [entry[1], entry[2]]
-      }
-    }
+      end
+    end
 
-    multigraphs.each_pair { |unit, labels_and_negatives|
+    multigraphs.each_pair do |unit, labels_and_negatives|
       # now just add options to the config
 
       config_options.concat [
         "multigraph #{name(unit)}",
         "graph_title Netstat: #{@protocol}: #{@name}#{" (#{unit})" if multigraphs.size > 1}",
-        "graph_category network",
+        'graph_category network',
         "graph_order #{labels_and_negatives.map { |label, _negative| label.escape }.join(' ')}"
       ]
 
-      config_options.push "graph_args --base 1024" if unit == :bytes
+      config_options.push 'graph_args --base 1024' if unit == :bytes
       has_negatives = false
 
-      labels_and_negatives.each { |label, negative|
+      labels_and_negatives.each do |label, negative|
         label_esc = label.escape
-        has_negatives = true unless negative == nil
+        has_negatives = true unless negative.nil?
 
         if negative == true
           # the value has no opposite and is negative
@@ -114,10 +115,10 @@ class Graph
             "#{negative_esc}.graph no"
           ]
         end
-      }
+      end
 
-      config_options.push "graph_vlabel per second#{" in (-) / out (+)" if has_negatives}"
-    }
+      config_options.push "graph_vlabel per second#{' in (-) / out (+)' if has_negatives}"
+    end
 
     config_options
   end
@@ -128,25 +129,25 @@ class Graph
     # first build a set of multigraphs, one per unit.
     # Hash key is unit, and the value is a hash of 'escaped label' => 'value'
     multigraphs = {}
-    @parse_expr.each { |expr, descr|
+    @parse_expr.each do |expr, descr|
       next unless descr # no label - skip this entry
 
       index = data.index { |line| line =~ expr }
       if index
         data.delete_at index
-        $~[1..-1].zip(descr).each { |value, info|
+        $~[1..-1].zip(descr).each do |value, info|
           unit, label = info
           (multigraphs[unit] ||= {})[label.escape] = value
-        }
+        end
       else
         warn "no line found for #{expr}, #{descr}" if $debug_mode
       end
-    }
+    end
 
-    multigraphs.each_pair { |unit, values|
+    multigraphs.each_pair do |unit, values|
       output_data.push "multigraph #{name(unit)}"
       output_data += values.map { |label, value| "#{label}.value #{value}" }
-    }
+    end
 
     output_data
   end
@@ -161,214 +162,230 @@ def graphs_for(protocol)
   # Order of the graps in each section is important for parsing.
   # At the same time, it is not important for munin, so we are OK placing it in parsing order here.
   when 'tcp'
-    $os == :linux ? [
-      Graph.new('sent', protocol, [
-                  # Description of the elements of arrays below:
-                  # 0: regexp to parse the line
-                  # 1: Array<Array[3]> for each matching group in the regular expression.
-                  #    0: unit name
-                  #    1: label
-                  #    2 (optional): negative label
-                  #    It could be reasonable to add more elements as warning and critical values.
+    if $os == :linux
+      [
+        Graph.new('sent', protocol, [
+                    # Description of the elements of arrays below:
+                    # 0: regexp to parse the line
+                    # 1: Array<Array[3]> for each matching group in the regular expression.
+                    #    0: unit name
+                    #    1: label
+                    #    2 (optional): negative label
+                    #    It could be reasonable to add more elements as warning and critical values.
 
-                  [/(\d+) segments send out$/, [[:segments, 'total']]],
-                  [/(\d+) segments retransmitted$/, [[:segments, 'retransmitted']]]
-                ]),
+                    [/(\d+) segments send out$/, [[:segments, 'total']]],
+                    [/(\d+) segments retransmitted$/, [[:segments, 'retransmitted']]]
+                  ]),
 
-      Graph.new('received', protocol, [
-                  [/(\d+) segments received$/, [[:segments, 'total']]],
-                  [/(\d+) bad segments received.$/, [[:segments, 'bad']]]
-                ]),
+        Graph.new('received', protocol, [
+                    [/(\d+) segments received$/, [[:segments, 'total']]],
+                    [/(\d+) bad segments received.$/, [[:segments, 'bad']]]
+                  ]),
 
-      Graph.new('connections', protocol, [
-                  [/(\d+) active connections openings$/, [[:connections, 'active openings']]],
-                  [/(\d+) passive connection openings$/, [[:connections, 'passive openings']]],
-                  [/(\d+) failed connection attempts$/, [[:connections, 'failed attempts']]],
-                  [/(\d+) connection resets received$/, [[:connections, 'RST received']]],
-                  [/(\d+) connections established$/, [[:connections, 'established']]],
-                  [/(\d+) resets sent$/, [[:connections, 'RST sent']]]
-                ]),
+        Graph.new('connections', protocol, [
+                    [/(\d+) active connections openings$/, [[:connections, 'active openings']]],
+                    [/(\d+) passive connection openings$/, [[:connections, 'passive openings']]],
+                    [/(\d+) failed connection attempts$/, [[:connections, 'failed attempts']]],
+                    [/(\d+) connection resets received$/, [[:connections, 'RST received']]],
+                    [/(\d+) connections established$/, [[:connections, 'established']]],
+                    [/(\d+) resets sent$/, [[:connections, 'RST sent']]]
+                  ]),
 
-      Graph.new('timeouts', protocol, [
-                  [/(\d+) timeouts after SACK recovery$/, [[:segments, 'after SACK recovery']]],
-                  [/(\d+) other TCP timeouts$/, [[:segments, 'other TCP']]],
-                  [/(\d+) timeouts in loss state$/, [[:segments, 'in a loss state']]]
-                ])
-    ] : [
-      Graph.new('sent', protocol, [
-                  [/(\d+) packets sent$/, [[:packets, 'total']]],
-                  [/(\d+) data packets \((\d+) bytes\)$/, [[:packets, 'data'], [:bytes, 'data']]],
-                  [/(\d+) data packets \((\d+) bytes\) retransmitted$/, [[:packets, 'retransmitted'], [:bytes, 'retransmitted']]],
-                  [/(\d+) data packets unnecessarily retransmitted$/, [[:packets, 'unnecessarily retransmitted']]],
-                  [/(\d+) resends initiated by MTU discovery$/, [[:packets, 'resends initiated by MTU discovery']]],
-                  [/(\d+) ack-only packets \((\d+) delayed\)$/, [[:packets, 'ack-only'], [:packets, 'ack-only delayed']]],
-                  [/(\d+) URG only packets$/, [[:packets, 'URG only']]],
-                  [/(\d+) window probe packets$/, [[:packets, 'window probe']]],
-                  [/(\d+) window update packets$/, [[:packets, 'window update']]],
-                  [/(\d+) control packets$/, [[:packets, 'control']]]
-                ]),
+        Graph.new('timeouts', protocol, [
+                    [/(\d+) timeouts after SACK recovery$/, [[:segments, 'after SACK recovery']]],
+                    [/(\d+) other TCP timeouts$/, [[:segments, 'other TCP']]],
+                    [/(\d+) timeouts in loss state$/, [[:segments, 'in a loss state']]]
+                  ])
+      ]
+    else
+      [
+        Graph.new('sent', protocol, [
+                    [/(\d+) packets sent$/, [[:packets, 'total']]],
+                    [/(\d+) data packets \((\d+) bytes\)$/, [[:packets, 'data'], [:bytes, 'data']]],
+                    [/(\d+) data packets \((\d+) bytes\) retransmitted$/, [[:packets, 'retransmitted'], [:bytes, 'retransmitted']]],
+                    [/(\d+) data packets unnecessarily retransmitted$/, [[:packets, 'unnecessarily retransmitted']]],
+                    [/(\d+) resends initiated by MTU discovery$/, [[:packets, 'resends initiated by MTU discovery']]],
+                    [/(\d+) ack-only packets \((\d+) delayed\)$/, [[:packets, 'ack-only'], [:packets, 'ack-only delayed']]],
+                    [/(\d+) URG only packets$/, [[:packets, 'URG only']]],
+                    [/(\d+) window probe packets$/, [[:packets, 'window probe']]],
+                    [/(\d+) window update packets$/, [[:packets, 'window update']]],
+                    [/(\d+) control packets$/, [[:packets, 'control']]]
+                  ]),
 
-      Graph.new('received', protocol, [
-                  [/(\d+) packets received$/, [[:packets, 'total']]],
-                  [/(\d+) acks \(for (\d+) bytes\)$/, [[:packets, 'acks'], [:bytes, 'acks']]],
-                  [/(\d+) duplicate acks$/, [[:packets, 'duplicate acks']]],
-                  [/(\d+) acks for unsent data$/, [[:packets, 'acks for unsent data']]],
-                  [/(\d+) packets \((\d+) bytes\) received in-sequence$/, [[:packets, 'in-sequence'], [:bytes, 'in-sequence']]],
-                  [/(\d+) completely duplicate packets \((\d+) bytes\)$/, [[:packets, 'completely duplicate'], [:bytes, 'completely duplicate']]],
-                  [/(\d+) old duplicate packets$/, [[:packets, 'old duplicate']]],
-                  [/(\d+) packets with some dup\. data \((\d+) bytes duped\)$/, [[:packets, 'some dup. data'], [:bytes, 'partial dups']]],
-                  [/(\d+) out-of-order packets \((\d+) bytes\)$/, [[:packets, 'out-of-order'], [:bytes, 'out-of-order']]],
-                  [/(\d+) packets \((\d+) bytes\) of data after window$/, [[:packets, 'data after window'], [:bytes, 'data after window']]],
-                  [/(\d+) window probes$/, [[:packets, 'window probes']]],
-                  [/(\d+) window update packets$/, [[:packets, 'window update']]],
-                  [/(\d+) packets received after close$/, [[:packets, 'after close']]],
-                  [/(\d+) discarded for bad checksums$/, [[:packets, 'bad checksums']]],
-                  [/(\d+) discarded for bad header offset fields?$/, [[:packets, 'bad header offset flds']]],
-                  [/(\d+) discarded because packet too short$/, [[:packets, 'too short']]],
-                  [/(\d+) discarded due to memory problems$/, [[:packets, 'discarded: memory problems']]],
-                  [/(\d+) ignored RSTs in the windows$/, [[:packets, 'ignored RSTs in windows']]],
-                  [/(\d+) segments updated rtt \(of (\d+) attempts\)$/, [[:packets, 'RTT: updated'], [:packets, 'RTT: attempts to update']]]
-                ]),
+        Graph.new('received', protocol, [
+                    [/(\d+) packets received$/, [[:packets, 'total']]],
+                    [/(\d+) acks \(for (\d+) bytes\)$/, [[:packets, 'acks'], [:bytes, 'acks']]],
+                    [/(\d+) duplicate acks$/, [[:packets, 'duplicate acks']]],
+                    [/(\d+) acks for unsent data$/, [[:packets, 'acks for unsent data']]],
+                    [/(\d+) packets \((\d+) bytes\) received in-sequence$/, [[:packets, 'in-sequence'], [:bytes, 'in-sequence']]],
+                    [/(\d+) completely duplicate packets \((\d+) bytes\)$/, [[:packets, 'completely duplicate'], [:bytes, 'completely duplicate']]],
+                    [/(\d+) old duplicate packets$/, [[:packets, 'old duplicate']]],
+                    [/(\d+) packets with some dup\. data \((\d+) bytes duped\)$/, [[:packets, 'some dup. data'], [:bytes, 'partial dups']]],
+                    [/(\d+) out-of-order packets \((\d+) bytes\)$/, [[:packets, 'out-of-order'], [:bytes, 'out-of-order']]],
+                    [/(\d+) packets \((\d+) bytes\) of data after window$/, [[:packets, 'data after window'], [:bytes, 'data after window']]],
+                    [/(\d+) window probes$/, [[:packets, 'window probes']]],
+                    [/(\d+) window update packets$/, [[:packets, 'window update']]],
+                    [/(\d+) packets received after close$/, [[:packets, 'after close']]],
+                    [/(\d+) discarded for bad checksums$/, [[:packets, 'bad checksums']]],
+                    [/(\d+) discarded for bad header offset fields?$/, [[:packets, 'bad header offset flds']]],
+                    [/(\d+) discarded because packet too short$/, [[:packets, 'too short']]],
+                    [/(\d+) discarded due to memory problems$/, [[:packets, 'discarded: memory problems']]],
+                    [/(\d+) ignored RSTs in the windows$/, [[:packets, 'ignored RSTs in windows']]],
+                    [/(\d+) segments updated rtt \(of (\d+) attempts\)$/, [[:packets, 'RTT: updated'], [:packets, 'RTT: attempts to update']]]
+                  ]),
 
-      Graph.new('connections', protocol, [
-                  [/(\d+) connection requests$/, [[:connections, 'requests']]],
-                  [/(\d+) connection accepts$/, [[:connections, 'accepts']]],
-                  [/(\d+) bad connection attempts$/, [[:connections, 'bad attempts']]],
-                  [/(\d+) listen queue overflows$/, [[:connections, 'listen queue overflows']]],
-                  [/(\d+) connections established \(including accepts\)$/, [[:connections, 'established']]],
-                  [/(\d+) connections closed \(including (\d+) drops\)$/, [[:connections, 'closed'], [:connections, 'dropped']]],
-                  [/(\d+) connections updated cached RTT on close$/, [[:connections, 'closed & upd cached RTT']]],
-                  [/(\d+) connections updated cached RTT variance on close$/, [[:connections, 'closed & upd cached RTT variance']]],
-                  [/(\d+) connections updated cached ssthresh on close$/, [[:connections, 'closed & upd cached ssthresh']]],
-                  [/(\d+) embryonic connections dropped$/, [[:connections, 'embryonic dropped']]]
-                ]),
+        Graph.new('connections', protocol, [
+                    [/(\d+) connection requests$/, [[:connections, 'requests']]],
+                    [/(\d+) connection accepts$/, [[:connections, 'accepts']]],
+                    [/(\d+) bad connection attempts$/, [[:connections, 'bad attempts']]],
+                    [/(\d+) listen queue overflows$/, [[:connections, 'listen queue overflows']]],
+                    [/(\d+) connections established \(including accepts\)$/, [[:connections, 'established']]],
+                    [/(\d+) connections closed \(including (\d+) drops\)$/, [[:connections, 'closed'], [:connections, 'dropped']]],
+                    [/(\d+) connections updated cached RTT on close$/, [[:connections, 'closed & upd cached RTT']]],
+                    [/(\d+) connections updated cached RTT variance on close$/, [[:connections, 'closed & upd cached RTT variance']]],
+                    [/(\d+) connections updated cached ssthresh on close$/, [[:connections, 'closed & upd cached ssthresh']]],
+                    [/(\d+) embryonic connections dropped$/, [[:connections, 'embryonic dropped']]]
+                  ]),
 
-      Graph.new('timeouts', protocol, [
-                  [/(\d+) retransmit timeouts$/, [[:connections, 'retransmit']]],
-                  [/(\d+) connections dropped by rexmit timeout$/, [[:connections, 'retransmit: dropped']]],
-                  [/(\d+) persist timeouts$/, [[:connections, 'persist']]],
-                  [/(\d+) connections dropped by persist timeout$/, [[:connections, 'persist: dropped']]],
-                  [/(\d+) Connections \(fin_wait_2\) dropped because of timeout$/, [[:connections, 'fin_wait_2: dropped']]],
-                  [/(\d+) keepalive timeouts$/, [[:connections, 'keepalive']]],
-                  [/(\d+) keepalive probes sent$/, [[:connections, 'keepalive: probes sent']]],
-                  [/(\d+) connections dropped by keepalive$/, [[:connections, 'keepalive: dropped']]]
-                ]),
+        Graph.new('timeouts', protocol, [
+                    [/(\d+) retransmit timeouts$/, [[:connections, 'retransmit']]],
+                    [/(\d+) connections dropped by rexmit timeout$/, [[:connections, 'retransmit: dropped']]],
+                    [/(\d+) persist timeouts$/, [[:connections, 'persist']]],
+                    [/(\d+) connections dropped by persist timeout$/, [[:connections, 'persist: dropped']]],
+                    [/(\d+) Connections \(fin_wait_2\) dropped because of timeout$/, [[:connections, 'fin_wait_2: dropped']]],
+                    [/(\d+) keepalive timeouts$/, [[:connections, 'keepalive']]],
+                    [/(\d+) keepalive probes sent$/, [[:connections, 'keepalive: probes sent']]],
+                    [/(\d+) connections dropped by keepalive$/, [[:connections, 'keepalive: dropped']]]
+                  ]),
 
-      Graph.new('correct predictions', protocol, [
-                  [/(\d+) correct ACK header predictions$/, [[:predictions, 'ACK header']]],
-                  [/(\d+) correct data packet header predictions$/, [[:predictions, 'data packet header']]]
-                ]),
+        Graph.new('correct predictions', protocol, [
+                    [/(\d+) correct ACK header predictions$/, [[:predictions, 'ACK header']]],
+                    [/(\d+) correct data packet header predictions$/, [[:predictions, 'data packet header']]]
+                  ]),
 
-      Graph.new('SYN', protocol, [
-                  [/(\d+) syncache entries added$/, [[:entries, 'cache added']]],
-                  [/(\d+) cookies sent$/, [[:entries, 'cookies sent']]],
-                  [/(\d+) cookies received$/, [[:entries, 'cookies received']]],
-                  [/(\d+) retransmitted$/, [[:entries, 'retransmitted']]],
-                  [/(\d+) dupsyn$/, [[:entries, 'duplicates']]],
-                  [/(\d+) dropped$/, [[:entries, 'dropped']]],
-                  [/(\d+) completed$/, [[:entries, 'completed']]],
-                  [/(\d+) bucket overflow$/, [[:entries, 'bucket overflow']]],
-                  [/(\d+) cache overflow$/, [[:entries, 'cache overflow']]],
-                  [/(\d+) reset$/, [[:entries, 'reset']]],
-                  [/(\d+) stale$/, [[:entries, 'stale']]],
-                  [/(\d+) aborted$/, [[:entries, 'aborted']]],
-                  [/(\d+) badack$/, [[:entries, 'bad ACK']]],
-                  [/(\d+) unreach$/, [[:entries, 'unreachable']]],
-                  [/(\d+) zone failures$/, [[:entries, 'zone failures']]],
-                  [/(\d+) hostcache entries added$/, [[:entries, 'hostcache added']]],
-                  [/(\d+) bucket overflow$/, [[:entries, 'hostcache overflow']]]
-                ]),
+        Graph.new('SYN', protocol, [
+                    [/(\d+) syncache entries added$/, [[:entries, 'cache added']]],
+                    [/(\d+) cookies sent$/, [[:entries, 'cookies sent']]],
+                    [/(\d+) cookies received$/, [[:entries, 'cookies received']]],
+                    [/(\d+) retransmitted$/, [[:entries, 'retransmitted']]],
+                    [/(\d+) dupsyn$/, [[:entries, 'duplicates']]],
+                    [/(\d+) dropped$/, [[:entries, 'dropped']]],
+                    [/(\d+) completed$/, [[:entries, 'completed']]],
+                    [/(\d+) bucket overflow$/, [[:entries, 'bucket overflow']]],
+                    [/(\d+) cache overflow$/, [[:entries, 'cache overflow']]],
+                    [/(\d+) reset$/, [[:entries, 'reset']]],
+                    [/(\d+) stale$/, [[:entries, 'stale']]],
+                    [/(\d+) aborted$/, [[:entries, 'aborted']]],
+                    [/(\d+) badack$/, [[:entries, 'bad ACK']]],
+                    [/(\d+) unreach$/, [[:entries, 'unreachable']]],
+                    [/(\d+) zone failures$/, [[:entries, 'zone failures']]],
+                    [/(\d+) hostcache entries added$/, [[:entries, 'hostcache added']]],
+                    [/(\d+) bucket overflow$/, [[:entries, 'hostcache overflow']]]
+                  ]),
 
-      Graph.new('SACK', protocol, [
-                  [/(\d+) SACK recovery episodes$/, [[:packets, 'recovery episodes']]],
-                  [/(\d+) segment rexmits in SACK recovery episodes$/, [[:packets, 'segment rexmits']]],
-                  [/(\d+) byte rexmits in SACK recovery episodes$/, [[:bytes, 'bytes rexmitted']]],
-                  [/(\d+) SACK options \(SACK blocks\) received$/, [[:packets, 'options blocks rcvd']]],
-                  [/(\d+) SACK options \(SACK blocks\) sent$/, [[:packets, 'options blocks sent']]],
-                  [/(\d+) SACK scoreboard overflow$/, [[:packets, 'scoreboard overflow']]]
-                ]),
+        Graph.new('SACK', protocol, [
+                    [/(\d+) SACK recovery episodes$/, [[:packets, 'recovery episodes']]],
+                    [/(\d+) segment rexmits in SACK recovery episodes$/, [[:packets, 'segment rexmits']]],
+                    [/(\d+) byte rexmits in SACK recovery episodes$/, [[:bytes, 'bytes rexmitted']]],
+                    [/(\d+) SACK options \(SACK blocks\) received$/, [[:packets, 'options blocks rcvd']]],
+                    [/(\d+) SACK options \(SACK blocks\) sent$/, [[:packets, 'options blocks sent']]],
+                    [/(\d+) SACK scoreboard overflow$/, [[:packets, 'scoreboard overflow']]]
+                  ]),
 
-      Graph.new('ECN', protocol, [
-                  [/(\d+) packets with ECN CE bit set$/, [[:packets, 'CE bit']]],
-                  [/(\d+) packets with ECN ECT\(0\) bit set$/, [[:packets, 'ECT(0) bit']]],
-                  [/(\d+) packets with ECN ECT\(1\) bit set$/, [[:packets, 'ECT(1) bit']]],
-                  [/(\d+) successful ECN handshakes$/, [[:packets, 'successful handshakes']]],
-                  [/(\d+) times ECN reduced the congestion window$/, [[:packets, 'congestion window reduced']]]
-                ])
-    ]
+        Graph.new('ECN', protocol, [
+                    [/(\d+) packets with ECN CE bit set$/, [[:packets, 'CE bit']]],
+                    [/(\d+) packets with ECN ECT\(0\) bit set$/, [[:packets, 'ECT(0) bit']]],
+                    [/(\d+) packets with ECN ECT\(1\) bit set$/, [[:packets, 'ECT(1) bit']]],
+                    [/(\d+) successful ECN handshakes$/, [[:packets, 'successful handshakes']]],
+                    [/(\d+) times ECN reduced the congestion window$/, [[:packets, 'congestion window reduced']]]
+                  ])
+      ]
+end
   when 'udp'
-    $os == :linux ? [] : [
-      Graph.new('received', protocol, [
-                  [/(\d+) datagrams received$/, [[:packets, 'total']]],
-                  [/(\d+) with incomplete header$/, [[:packets, 'incomplete header']]],
-                  [/(\d+) with bad data length field$/, [[:packets, 'bad data length field']]],
-                  [/(\d+) with bad checksum$/, [[:packets, 'bad checksum']]],
-                  [/(\d+) with no checksum$/, [[:packets, 'no checksum']]],
-                  [/(\d+) dropped due to no socket$/, [[:packets, 'dropped: no socket']]],
-                  [/(\d+) broadcast\/multicast datagrams undelivered$/, [[:packets, '*cast undelivered']]],
-                  [/(\d+) dropped due to full socket buffers$/, [[:packets, 'dropped: no buffers']]],
-                  [/(\d+) not for hashed pcb$/, [[:packets, 'not for hashed pcb']]],
-                  [/(\d+) delivered$/, [[:packets, 'delivered']]]
-                ]),
+    if $os == :linux
+      []
+    else
+      [
+        Graph.new('received', protocol, [
+                    [/(\d+) datagrams received$/, [[:packets, 'total']]],
+                    [/(\d+) with incomplete header$/, [[:packets, 'incomplete header']]],
+                    [/(\d+) with bad data length field$/, [[:packets, 'bad data length field']]],
+                    [/(\d+) with bad checksum$/, [[:packets, 'bad checksum']]],
+                    [/(\d+) with no checksum$/, [[:packets, 'no checksum']]],
+                    [/(\d+) dropped due to no socket$/, [[:packets, 'dropped: no socket']]],
+                    [%r{(\d+) broadcast/multicast datagrams undelivered$}, [[:packets, '*cast undelivered']]],
+                    [/(\d+) dropped due to full socket buffers$/, [[:packets, 'dropped: no buffers']]],
+                    [/(\d+) not for hashed pcb$/, [[:packets, 'not for hashed pcb']]],
+                    [/(\d+) delivered$/, [[:packets, 'delivered']]]
+                  ]),
 
-      Graph.new('sent', protocol, [
-                  [/(\d+) datagrams output$/, [[:packets, 'total']]],
-                  [/(\d+) times multicast source filter matched$/, [[:packets, 'multicast src filter match']]]
-                ])
-    ]
+        Graph.new('sent', protocol, [
+                    [/(\d+) datagrams output$/, [[:packets, 'total']]],
+                    [/(\d+) times multicast source filter matched$/, [[:packets, 'multicast src filter match']]]
+                  ])
+      ]
+end
   when 'ip'
-    $os == :linux ? [] : [
-      Graph.new('received', protocol, [
-                  [/(\d+) total packets received$/, [[:packets, 'total']]],
-                  [/(\d+) bad header checksums$/, [[:packets, 'bad header checksum']]],
-                  [/(\d+) with size smaller than minimum$/, [[:packets, 'size smaller than min']]],
-                  [/(\d+) with data size < data length$/, [[:packets, 'data size < data length']]],
-                  [/(\d+) with ip length > max ip packet size$/, [[:packets, 'ip length > max ip packet sz']]],
-                  [/(\d+) with header length < data size$/, [[:packets, 'header length < data size']]],
-                  [/(\d+) with data length < header length$/, [[:packets, 'data length < header length']]],
-                  [/(\d+) with bad options$/, [[:packets, 'bad options']]],
-                  [/(\d+) with incorrect version number$/, [[:packets, 'incorrect version']]],
-                  [/(\d+) fragments? received$/, [[:packets, 'fragments']]],
-                  [/(\d+) fragments? dropped \(dup or out of space\)$/, [[:packets, 'frags dropped: dup/out of spc']]],
-                  [/(\d+) fragments? dropped after timeout$/, [[:packets, 'frags dropped: timeout']]],
-                  [/(\d+) packets? reassembled ok$/, [[:packets, 'reassembled ok']]],
-                  [/(\d+) packets? for this host$/, [[:packets, 'for this host']]],
-                  [/(\d+) packets? for unknown\/unsupported protocol$/, [[:packets, 'for unknown/unsup protocol']]],
-                  [/(\d+) packets? forwarded \((\d+) packets fast forwarded\)$/, [[:packets, 'forwarded'], [:packets, 'fast forwarded']]],
-                  [/(\d+) packets? not forwardable$/, [[:packets, 'not forwardable']]],
-                  [/(\d+) packets? received for unknown multicast group$/, [[:packets, 'unknown multicast grp']]]
-                ]),
+    if $os == :linux
+      []
+    else
+      [
+        Graph.new('received', protocol, [
+                    [/(\d+) total packets received$/, [[:packets, 'total']]],
+                    [/(\d+) bad header checksums$/, [[:packets, 'bad header checksum']]],
+                    [/(\d+) with size smaller than minimum$/, [[:packets, 'size smaller than min']]],
+                    [/(\d+) with data size < data length$/, [[:packets, 'data size < data length']]],
+                    [/(\d+) with ip length > max ip packet size$/, [[:packets, 'ip length > max ip packet sz']]],
+                    [/(\d+) with header length < data size$/, [[:packets, 'header length < data size']]],
+                    [/(\d+) with data length < header length$/, [[:packets, 'data length < header length']]],
+                    [/(\d+) with bad options$/, [[:packets, 'bad options']]],
+                    [/(\d+) with incorrect version number$/, [[:packets, 'incorrect version']]],
+                    [/(\d+) fragments? received$/, [[:packets, 'fragments']]],
+                    [/(\d+) fragments? dropped \(dup or out of space\)$/, [[:packets, 'frags dropped: dup/out of spc']]],
+                    [/(\d+) fragments? dropped after timeout$/, [[:packets, 'frags dropped: timeout']]],
+                    [/(\d+) packets? reassembled ok$/, [[:packets, 'reassembled ok']]],
+                    [/(\d+) packets? for this host$/, [[:packets, 'for this host']]],
+                    [%r{(\d+) packets? for unknown/unsupported protocol$}, [[:packets, 'for unknown/unsup protocol']]],
+                    [/(\d+) packets? forwarded \((\d+) packets fast forwarded\)$/, [[:packets, 'forwarded'], [:packets, 'fast forwarded']]],
+                    [/(\d+) packets? not forwardable$/, [[:packets, 'not forwardable']]],
+                    [/(\d+) packets? received for unknown multicast group$/, [[:packets, 'unknown multicast grp']]]
+                  ]),
 
-      Graph.new('sent', protocol, [
-                  [/(\d+) packets? sent from this host$/, [[:packets, 'total']]],
-                  [/(\d+) redirects? sent$/, [[:packets, 'redirect']]],
-                  [/(\d+) packets? sent with fabricated ip header$/, [[:packets, 'fabricated IP head']]],
-                  [/(\d+) output packets? dropped due to no bufs, etc\.$/, [[:packets, 'dropped: no bufs, etc']]],
-                  [/(\d+) output packets? discarded due to no route$/, [[:packets, 'discarded: no route']]],
-                  [/(\d+) output datagrams? fragmented$/, [[:packets, 'fragmented']]],
-                  [/(\d+) fragments? created$/, [[:packets, 'fragments created']]],
-                  [/(\d+) datagrams? that can't be fragmented$/, [[:packets, "can't be fragmented"]]],
-                  [/(\d+) tunneling packets? that can't find gif$/, [[:packets, 'tunneling, gif not found']]],
-                  [/(\d+) datagrams? with bad address in header$/, [[:packets, 'bad address in header']]]
-                ])
-    ]
+        Graph.new('sent', protocol, [
+                    [/(\d+) packets? sent from this host$/, [[:packets, 'total']]],
+                    [/(\d+) redirects? sent$/, [[:packets, 'redirect']]],
+                    [/(\d+) packets? sent with fabricated ip header$/, [[:packets, 'fabricated IP head']]],
+                    [/(\d+) output packets? dropped due to no bufs, etc\.$/, [[:packets, 'dropped: no bufs, etc']]],
+                    [/(\d+) output packets? discarded due to no route$/, [[:packets, 'discarded: no route']]],
+                    [/(\d+) output datagrams? fragmented$/, [[:packets, 'fragmented']]],
+                    [/(\d+) fragments? created$/, [[:packets, 'fragments created']]],
+                    [/(\d+) datagrams? that can't be fragmented$/, [[:packets, "can't be fragmented"]]],
+                    [/(\d+) tunneling packets? that can't find gif$/, [[:packets, 'tunneling, gif not found']]],
+                    [/(\d+) datagrams? with bad address in header$/, [[:packets, 'bad address in header']]]
+                  ])
+      ]
+end
   when 'arp'
-    $os == :linux ? [] : [
-      Graph.new('packets', protocol, [
-                  # This is just a total, so ignore the value but keep regexp to avoid 'not parsed' warning.
-                  [/(\d+) ARP packets? received$/],
-                  [/(\d+) ARP requests? received$/, [[:packets, 'requests received']]],
-                  [/(\d+) ARP repl(?:y|ies) received$/, [[:packets, 'replies received']]],
-                  [/(\d+) ARP requests? sent$/, [[:packets, 'requests', 'requests received']]],
-                  [/(\d+) ARP repl(?:y|ies) sent$/, [[:packets, 'replies', 'replies received']]],
-                  [/(\d+) total packets? dropped due to no ARP entry$/, [[:packets, 'no entry']]]
-                ]),
+    if $os == :linux
+      []
+    else
+      [
+        Graph.new('packets', protocol, [
+                    # This is just a total, so ignore the value but keep regexp to avoid 'not parsed' warning.
+                    [/(\d+) ARP packets? received$/],
+                    [/(\d+) ARP requests? received$/, [[:packets, 'requests received']]],
+                    [/(\d+) ARP repl(?:y|ies) received$/, [[:packets, 'replies received']]],
+                    [/(\d+) ARP requests? sent$/, [[:packets, 'requests', 'requests received']]],
+                    [/(\d+) ARP repl(?:y|ies) sent$/, [[:packets, 'replies', 'replies received']]],
+                    [/(\d+) total packets? dropped due to no ARP entry$/, [[:packets, 'no entry']]]
+                  ]),
 
-      Graph.new('entries', protocol, [
-                  [/(\d+) ARP entrys? timed out$/, [[:entries, 'timed out']]],
-                  [/(\d+) Duplicate IPs seen$/, [[:entries, 'duplicate IPs seen']]]
-                ])
-    ]
+        Graph.new('entries', protocol, [
+                    [/(\d+) ARP entrys? timed out$/, [[:entries, 'timed out']]],
+                    [/(\d+) Duplicate IPs seen$/, [[:entries, 'duplicate IPs seen']]]
+                  ])
+      ]
+end
   end
 end
 
@@ -379,9 +396,11 @@ proto_name = 'tcp' if proto_name.empty?
 
 def netstat_s(protocol)
   if $os == :linux
-    %w(tcp udp).include?(protocol) ?
-      `netstat -s --#{protocol}` :
+    if %w[tcp udp].include?(protocol)
+      `netstat -s --#{protocol}`
+    else
       `netstat -s --raw`
+end
   else
     `netstat -sp #{protocol}`
   end.lines.reject { |line| line =~ /^\w+:/ }
@@ -389,20 +408,22 @@ end
 
 case ARGV.first
 when 'autoconf'
-  puts [:linux, :freebsd].include?($os) ? 'yes' : 'no'
+  puts %i[linux freebsd].include?($os) ? 'yes' : 'no'
 when 'suggest'
-  puts $os == :linux ? %w(tcp) : %w(tcp udp ip arp)
+  puts $os == :linux ? %w[tcp] : %w[tcp udp ip arp]
 when 'config'
-  graphs_for(proto_name).each { |graph|
+  graphs_for(proto_name).each do |graph|
     puts graph.config.join $/
-  }
+  end
 else
   data = netstat_s(proto_name)
-  graphs_for(proto_name).each { |graph|
+  graphs_for(proto_name).each do |graph|
     puts graph.fetch(data).join $/
-  }
+  end
 
-  warn "not parsed:\n#{data.join}" unless data.empty? if $debug_mode
+  if $debug_mode
+    warn "not parsed:\n#{data.join}" unless data.empty?
+end
 end
 
 # awful performance when scrolling through those regexps above
diff --git a/plugins/oracle/oracle-pga-monitor b/plugins/oracle/oracle-pga-monitor
index a6929ad8..90920a37 100755
--- a/plugins/oracle/oracle-pga-monitor
+++ b/plugins/oracle/oracle-pga-monitor
@@ -21,39 +21,36 @@ You should have received a copy of the GNU General Public License along
 with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
-
 Prerequistes:
-	1) env.ORACLE_HOME set in munin-node
-	2) rubygems
-	3) oci8 - DBI gem for connecting to Oracle
-	   * instruction of installing oci8 is available here:
-	     http://ruby-oci8.rubyforge.org/en/InstallBinaryPackage.html
+  1) env.ORACLE_HOME set in munin-node
+  2) rubygems
+  3) oci8 - DBI gem for connecting to Oracle
+     * instruction of installing oci8 is available here:
+       http://ruby-oci8.rubyforge.org/en/InstallBinaryPackage.html
 
 Usage:
-	1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
-	2) chmod to allow executable to others
-	3) create symbolic link in /etc/munin/plugins
+  1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
+  2) chmod to allow executable to others
+  3) create symbolic link in /etc/munin/plugins
            ln -s /usr/share/munin/plugins/oracle_<sid>_pga.rb /etc/munin/plugins/oracle_<sid>_pga.rb
            ** replace <sid> with your oralce sid
 
 Parameters:
-	autoconf
-	config (required)
+  autoconf
+  config (required)
 
 Configurable variables:
-	orauser : oracle user who has select privilege to query v$pgastat view
-	orapass : password for the oracle user
-	dbport  : port used by the monitored instance (notice: numeric value)
-	dbname  : database to be monitored
-	dbhost  : host or ip address of db instance
-
+  orauser : oracle user who has select privilege to query v$pgastat view
+  orapass : password for the oracle user
+  dbport  : port used by the monitored instance (notice: numeric value)
+  dbname  : database to be monitored
+  dbhost  : host or ip address of db instance
 
 #%# family=auto
 #%# capabilities=autoconf
 
 =end
 
-
 require 'rubygems'
 require 'oci8'
 
@@ -69,7 +66,7 @@ tnsname = "(DESCRIPTION =
 
 def runQuery(name, query)
   rows = $conn.exec(query)
-  puts "#{name}.value #{rows.fetch().to_s}"
+  puts "#{name}.value #{rows.fetch}"
   rows.close
 end
 
@@ -82,31 +79,32 @@ pga_target_query = "SELECT TO_CHAR(ROUND(decode(unit,'bytes',(value)/(1024*1024)
 pga_query = "SELECT TO_CHAR(ROUND(decode(unit,'bytes',(value)/(1024*1024),value),2)) pga
                      from V$PGASTAT where name = 'total PGA inuse'"
 
-pga_components = { "pga_target" => pga_target_query,
-                   "pga_in_use" => pga_query }
+pga_components = { 'pga_target' => pga_target_query,
+                   'pga_in_use' => pga_query }
 
 #
 # autoconf
 #
-if ARGV[0] == "autoconf"
+case ARGV[0]
+when 'autoconf'
   if tnsname.length > 1 && orauser.length > 1 && orapass.length > 1
-    puts "yes"
+    puts 'yes'
   else
-    puts "no"
+    puts 'no'
     puts "Usage: #{__FILE__} autoconf|conf"
   end
   exit 0
 #
 # config definition
 #
-elsif ARGV[0] == "config"
-  puts "graph_args --base 1024k -r --lower-limit 0"
+when 'config'
+  puts 'graph_args --base 1024k -r --lower-limit 0'
   puts "graph_title Oracle PGA from #{dbname}"
-  puts "graph_category db"
-  puts "graph_info This graph shows the PGA memory usage (in MB)"
-  puts "graph_vlabel MB"
-  puts "graph_scale no"
-  puts "graph_period second"
+  puts 'graph_category db'
+  puts 'graph_info This graph shows the PGA memory usage (in MB)'
+  puts 'graph_vlabel MB'
+  puts 'graph_scale no'
+  puts 'graph_period second'
 
   pga_components.keys.each do |p|
     puts "#{p}.label #{p}"
diff --git a/plugins/oracle/oracle-sga b/plugins/oracle/oracle-sga
index c1aa5d12..3fc21f03 100755
--- a/plugins/oracle/oracle-sga
+++ b/plugins/oracle/oracle-sga
@@ -21,38 +21,35 @@ You should have received a copy of the GNU General Public License along
 with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
-
 Prerequistes:
-	1) env.ORACLE_HOME set in munin-node
-	2) rubygems
-	3) oci8 - DBI gem for connecting to Oracle
-	   * instruction of installing oci8 is available here:
-	     http://ruby-oci8.rubyforge.org/en/InstallBinaryPackage.html
+  1) env.ORACLE_HOME set in munin-node
+  2) rubygems
+  3) oci8 - DBI gem for connecting to Oracle
+     * instruction of installing oci8 is available here:
+       http://ruby-oci8.rubyforge.org/en/InstallBinaryPackage.html
 
 Usage:
-	1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
-	2) chmod to allow executable to others
-	3) create symbolic link in /etc/munin/plugins
+  1) copy this script to the munin install plugins directory (e.g. /usr/share/munin/plugins)
+  2) chmod to allow executable to others
+  3) create symbolic link in /etc/munin/plugins
            ln -s /usr/share/munin/plugins/oracle_orcl_sga.rb /etc/munin/plugins/oracle_orcl_sga.rb
 
 Parameters:
-	autoconf
-	config (required)
+  autoconf
+  config (required)
 
 Configurable variables:
-	orauser : oracle user who has select privilege to query v$sgastat view
-	orapass : password for the oracle user
-	dbport  : port used by the monitored instance (notice: numeric value)
-	dbname  : database to be monitored
-	dbhost  : host or ip address of db instance
-
+  orauser : oracle user who has select privilege to query v$sgastat view
+  orapass : password for the oracle user
+  dbport  : port used by the monitored instance (notice: numeric value)
+  dbname  : database to be monitored
+  dbhost  : host or ip address of db instance
 
 #%# family=auto
 #%# capabilities=autoconf
 
 =end
 
-
 require 'rubygems'
 require 'oci8'
 
@@ -68,7 +65,7 @@ tnsname = "(DESCRIPTION =
 
 def runQuery(name, query)
   rows = $conn.exec(query)
-  puts "#{name}.value #{rows.fetch().to_s}"
+  puts "#{name}.value #{rows.fetch}"
   rows.close
 end
 
@@ -102,35 +99,36 @@ log_buffer_query = "SELECT TO_CHAR(ROUND(SUM(decode(pool, NULL,
                    decode(name, 'log_buffer', (bytes)/(1024*1024),0),0)),2)) sga_lbuffer
                    from V$SGASTAT"
 
-memory_components = { "fixed_area" => fixed_area_query,
-                      "buffer_cache" => buffer_cache_query,
-                      "java_pool" => java_pool_query,
-                      "large_pool" => large_pool_query,
-                      "log_buffer" => log_buffer_query,
-                      "shared_pool" => shared_pool_query }
+memory_components = { 'fixed_area' => fixed_area_query,
+                      'buffer_cache' => buffer_cache_query,
+                      'java_pool' => java_pool_query,
+                      'large_pool' => large_pool_query,
+                      'log_buffer' => log_buffer_query,
+                      'shared_pool' => shared_pool_query }
 
 #
 # autoconf
 #
-if ARGV[0] == "autoconf"
+case ARGV[0]
+when 'autoconf'
   if tnsname.length > 1 && orauser.length > 1 && orapass.length > 1
-    puts "yes"
+    puts 'yes'
   else
-    puts "no"
+    puts 'no'
     puts "Usage: #{__FILE__} autoconf|conf"
   end
   exit 0
 #
 # config definition
 #
-elsif ARGV[0] == "config"
-  puts "graph_args --base 1024k -r --lower-limit 0"
+when 'config'
+  puts 'graph_args --base 1024k -r --lower-limit 0'
   puts "graph_title Oracle SGA from #{dbname}"
-  puts "graph_category db"
-  puts "graph_info This graph shows the SGA memory usage (in MB)"
-  puts "graph_vlabel MB"
-  puts "graph_scale no"
-  puts "graph_period second"
+  puts 'graph_category db'
+  puts 'graph_info This graph shows the SGA memory usage (in MB)'
+  puts 'graph_vlabel MB'
+  puts 'graph_scale no'
+  puts 'graph_period second'
 
   memory_components.keys.each do |m|
     puts "#{m}.label #{m}"
@@ -138,7 +136,7 @@ elsif ARGV[0] == "config"
     puts "#{m}.type GAUGE"
 
     # make sure fixed_area is at the bottom of the stack
-    if (m == 'fixed_area')
+    if m == 'fixed_area'
       puts "#{m}.draw AREA"
     else
       puts "#{m}.draw STACK"
diff --git a/plugins/other/port_ b/plugins/other/port_
index 585166f5..81fd7f47 100755
--- a/plugins/other/port_
+++ b/plugins/other/port_
@@ -16,53 +16,53 @@ require 'rubygems'
 require 'munin'
 
 SERVICE   = $0.split('_').last
-SERVICE_F = '/etc/services'
-PORT      = /^[\d]+(\.[\d]+){0,1}$/ === SERVICE ? SERVICE : %x[grep #{SERVICE} #{SERVICE_F}].split("\t\t")[1].split('/')[0]
+SERVICE_F = '/etc/services'.freeze
+PORT      = SERVICE =~ /^\d+(\.\d+){0,1}$/ ? SERVICE : `grep #{SERVICE} #{SERVICE_F}`.split("\t\t")[1].split('/')[0]
 
 class PortMonit < Munin::Plugin
   graph_attributes "#{SERVICE} port usage, known as #{PORT}",
-                   :category => 'network',
-                   :info => 'This graph shows connection split by the state of the socket.',
-                   :vlabel => 'Current connections'
+                   category: 'network',
+                   info: 'This graph shows connection split by the state of the socket.',
+                   vlabel: 'Current connections'
 
   declare_field :ESTABLISHED,
-                :label => 'Established', :draw => :AREA,
-                :type => :GAUGE, :min => 0
+                label: 'Established', draw: :AREA,
+                type: :GAUGE, min: 0
 
   declare_field :CLOSE_WAIT,
-                :label => 'Waiting close', :draw => :STACK,
-                :type => :GAUGE, :min => 0
+                label: 'Waiting close', draw: :STACK,
+                type: :GAUGE, min: 0
 
   declare_field :TIME_WAIT,
-                :label => 'Waiting after close', :draw => :STACK,
-                :type => :GAUGE, :min => 0
+                label: 'Waiting after close', draw: :STACK,
+                type: :GAUGE, min: 0
 
   declare_field :CLOSING,
-                :label => 'Closing', :draw => :STACK,
-                :type => :GAUGE, :min => 0
+                label: 'Closing', draw: :STACK,
+                type: :GAUGE, min: 0
 
   declare_field :LAST_ACK,
-                :label => 'Waiting for acknowledgement', :draw => :STACK,
-                :type => :GAUGE, :min => 0
+                label: 'Waiting for acknowledgement', draw: :STACK,
+                type: :GAUGE, min: 0
 
   declare_field :FIN_WAIT_1,
-                :label => 'Socket closed, connection shutting down', :draw => :STACK,
-                :type => :GAUGE, :min => 0
+                label: 'Socket closed, connection shutting down', draw: :STACK,
+                type: :GAUGE, min: 0
 
   declare_field :FIN_WAIT_2,
-                :label => 'Connection closed, Socket still waiting', :draw => :STACK,
-                :type => :GAUGE, :min => 0
+                label: 'Connection closed, Socket still waiting', draw: :STACK,
+                type: :GAUGE, min: 0
 
   def retrieve_values
-    @_netstat = %x[netstat -n -P tcp | egrep "\.#{PORT} "].split("\n")
+    @_netstat = `netstat -n -P tcp | egrep "\.#{PORT} "`.split("\n")
 
-    { :ESTABLISHED => count(@_netstat, 'ESTABLISHED'),
-      :CLOSE_WAIT => count(@_netstat, 'CLOSE_WAIT'),
-      :CLOSING => count(@_netstat, 'CLOSING'),
-      :LAST_ACK => count(@_netstat, 'LAST_ACK'),
-      :FIN_WAIT_1 => count(@_netstat, 'FIN_WAIT_1'),
-      :FIN_WAIT_2 => count(@_netstat, 'FIN_WAIT_2'),
-      :TIME_WAIT => count(@_netstat, 'TIME_WAIT') }
+    { ESTABLISHED: count(@_netstat, 'ESTABLISHED'),
+      CLOSE_WAIT: count(@_netstat, 'CLOSE_WAIT'),
+      CLOSING: count(@_netstat, 'CLOSING'),
+      LAST_ACK: count(@_netstat, 'LAST_ACK'),
+      FIN_WAIT_1: count(@_netstat, 'FIN_WAIT_1'),
+      FIN_WAIT_2: count(@_netstat, 'FIN_WAIT_2'),
+      TIME_WAIT: count(@_netstat, 'TIME_WAIT') }
   end
 
   private
@@ -72,7 +72,7 @@ class PortMonit < Munin::Plugin
 
     source.each { |obj| @_result += 1 if obj.match(regex) }
 
-    return @_result
+    @_result
   end
 end
 
diff --git a/plugins/passenger/passenger_memory b/plugins/passenger/passenger_memory
index c1138457..171db640 100755
--- a/plugins/passenger/passenger_memory
+++ b/plugins/passenger/passenger_memory
@@ -11,11 +11,11 @@
 memory_stats_command = ENV['memory_stats_command'] || '/opt/ruby-enterprise-1.8.6-20080810/bin/passenger-memory-stats'
 
 if ARGV.length > 0 && ARGV[0] == 'config'
-  puts "graph_title Passenger Memory Usage"
-  puts "graph_category webserver"
-  puts "graph_vlabel MB"
-  puts "apache_rss.label Apache Dirty RSS"
-  puts "passenger_rss.label Passenger Dirty RSS"
+  puts 'graph_title Passenger Memory Usage'
+  puts 'graph_category webserver'
+  puts 'graph_vlabel MB'
+  puts 'apache_rss.label Apache Dirty RSS'
+  puts 'passenger_rss.label Passenger Dirty RSS'
   exit(0)
 end
 
diff --git a/plugins/passenger/passenger_processes b/plugins/passenger/passenger_processes
index 1af4e27b..8b4b5a4b 100755
--- a/plugins/passenger/passenger_processes
+++ b/plugins/passenger/passenger_processes
@@ -8,13 +8,13 @@
 process_stats_command = ENV['process_stats_command'] || '/opt/ruby-enterprise-1.8.6-20080810/bin/passenger-status'
 
 if ARGV.length > 0 && ARGV[0] == 'config'
-  puts "graph_title Passenger Processes"
-  puts "graph_category webserver"
-  puts "graph_vlabel processes"
-  puts "max.label Max processes"
-  puts "count.label Total processes"
-  puts "active.label Active processes"
-  puts "queued.label Queued requests"
+  puts 'graph_title Passenger Processes'
+  puts 'graph_category webserver'
+  puts 'graph_vlabel processes'
+  puts 'max.label Max processes'
+  puts 'count.label Total processes'
+  puts 'active.label Active processes'
+  puts 'queued.label Queued requests'
   exit(0)
 end
 
@@ -24,13 +24,14 @@ active = nil
 queued = nil
 
 `#{process_stats_command}`.each_line do |line|
-  if /max\s+=\s+(\d+)/.match(line)
+  case line
+  when /max\s+=\s+(\d+)/
     max = $~[1]
-  elsif /count\s+=\s+(\d+)/.match(line)
+  when /count\s+=\s+(\d+)/
     count = $~[1]
-  elsif /^active\s+=\s+(\d+)/.match(line)
+  when /^active\s+=\s+(\d+)/
     active = $~[1]
-  elsif /Waiting on global queue\s+=\s+(\d+)/.match(line)
+  when /Waiting on global queue\s+=\s+(\d+)/
     queued = $~[1]
   end
 end
diff --git a/plugins/passenger/passenger_status b/plugins/passenger/passenger_status
index 8c992c54..9ac5738a 100755
--- a/plugins/passenger/passenger_status
+++ b/plugins/passenger/passenger_status
@@ -28,27 +28,27 @@ end
 def output_values
   status = `sudo passenger-status`
   unless $?.success?
-    $stderr.puts "failed executing passenger-status"
+    warn 'failed executing passenger-status'
     exit 1
   end
   status =~ /max\s+=\s+(\d+)/
-  puts "max.value #{$1}"
+  puts "max.value #{Regexp.last_match(1)}"
 
   status =~ /count\s+=\s+(\d+)/
-  puts "running.value #{$1}"
+  puts "running.value #{Regexp.last_match(1)}"
 
   status =~ /active\s+=\s+(\d+)/
-  puts "active.value #{$1}"
+  puts "active.value #{Regexp.last_match(1)}"
 
   status =~ /Waiting on global queue:\s+(\d+)/
-  puts "waiting.value #{$1}"
+  puts "waiting.value #{Regexp.last_match(1)}"
 
   total_sessions = 0
   status.scan(/Sessions: (\d+)/).flatten.each { |count| total_sessions += count.to_i }
   puts "sessions.value #{total_sessions}"
 end
 
-if ARGV[0] == "config"
+if ARGV[0] == 'config'
   output_config
 else
   output_values
diff --git a/plugins/php/php_eaccelerator b/plugins/php/php_eaccelerator
index 1d89df71..e5e93354 100755
--- a/plugins/php/php_eaccelerator
+++ b/plugins/php/php_eaccelerator
@@ -31,7 +31,7 @@ user  = ENV['user'] || 'user'
 pwd   = ENV['password'] || 'password'
 url   = ENV['url'] || 'http://127.0.0.1/control.php'
 
-if ARGV[0] == "config"
+if ARGV[0] == 'config'
   print "EAccelerator Monitoring\n"
   print "graph_title PHP Eaccelerator\n"
   print "graph_category webserver\n"
@@ -49,54 +49,48 @@ end
 
 one_liners = 0
 three_liners = 0
-key = ""
+key = ''
 
-open(url, :http_basic_authentication => [user, pwd]) do |f|
+open(url, http_basic_authentication: [user, pwd]) do |f|
   f.each do |line|
     if three_liners > 0
-      three_liners = three_liners + 1
+      three_liners += 1
 
-      if three_liners == 2
-        print "Memoryusagepercentage.value "
-      end
+      print 'Memoryusagepercentage.value ' if three_liners == 2
 
-      if three_liners == 3
-        print "Memoryusage.value "
-      end
+      print 'Memoryusage.value ' if three_liners == 3
 
-      if three_liners == 4
-        print "Memorymax.value "
-      end
+      print 'Memorymax.value ' if three_liners == 4
 
-      print line.gsub!(/[^0-9.]/s, "")
+      print line.gsub!(/[^0-9.]/s, '')
       print "\n"
     end
 
     if one_liners > 0
-      one_liners = one_liners + 1
+      one_liners += 1
       print "#{key}.value "
-      print line.gsub!(/[^0-9.]/s, "")
+      print line.gsub!(/[^0-9.]/s, '')
       print "\n"
     end
 
     if one_liners > 1
-      line = ""
+      line = ''
       one_liners = 0
     end
 
     if three_liners > 3
-      line = ""
+      line = ''
       three_liners = 0
     end
 
     if line =~ /Memory usage/
-      key = line.gsub!(/(<[^>]*>)|\n|\t| /s, "")
-      three_liners = three_liners + 1
+      key = line.gsub!(/(<[^>]*>)|\n|\t| /s, '')
+      three_liners += 1
     end
 
     if line =~ /<td class="e">Free memory/ || line =~ /<td class="e">Cached scripts/ || line =~ /<td class="e">Removed scripts/ || line =~ /<td class="e">Cached keys/
-      key = line.gsub!(/(<[^>]*>)|\n|\t| /s, "")
-      one_liners = one_liners + 1
+      key = line.gsub!(/(<[^>]*>)|\n|\t| /s, '')
+      one_liners += 1
     end
   end
 end
diff --git a/plugins/puppet/puppet_runtime b/plugins/puppet/puppet_runtime
index 620ac06e..30720582 100755
--- a/plugins/puppet/puppet_runtime
+++ b/plugins/puppet/puppet_runtime
@@ -15,29 +15,29 @@
 
 # reports how long the puppet agent took to apply the catalog
 def get_runtime
-  logfile = ENV["puppet_logfile"] || "/var/log/messages"
+  logfile = ENV['puppet_logfile'] || '/var/log/messages'
   t = Time.now
-  dateformat = ENV["puppet_logformat"] || "^%b %d"
+  dateformat = ENV['puppet_logformat'] || '^%b %d'
   today = t.strftime(dateformat)
   File.open(logfile).grep(/#{today}/).grep(/Finished catalog run in/).reverse_each do |line|
     if line =~ /in (.*) seconds/
-      puts "runtime.value #{$1}"
+      puts "runtime.value #{Regexp.last_match(1)}"
       exit 0
     end
   end
 end
 
 case ARGV[0]
-when "config"
-  puts "graph_category other"
-  puts "graph_args --base 1000 -l 0"
-  puts "graph_scale no"
-  puts "graph_title puppet catalog run time"
-  puts "graph_vlabel Seconds"
-  puts "runtime.label Catalog application time"
+when 'config'
+  puts 'graph_category other'
+  puts 'graph_args --base 1000 -l 0'
+  puts 'graph_scale no'
+  puts 'graph_title puppet catalog run time'
+  puts 'graph_vlabel Seconds'
+  puts 'runtime.label Catalog application time'
   exit 0
-when "autoconf"
-  puts "yes"
+when 'autoconf'
+  puts 'yes'
   exit 0
 else
   get_runtime
diff --git a/plugins/router/ag241-adsl b/plugins/router/ag241-adsl
index 1aba2300..e4063276 100755
--- a/plugins/router/ag241-adsl
+++ b/plugins/router/ag241-adsl
@@ -35,7 +35,6 @@ Some magical munin foo...
 
 =end
 
-
 # Require this module, it is part of the standard ruby lib AFAIK
 require 'net/http'
 
@@ -49,7 +48,7 @@ stat = nil
 # Check executable "name" for parameter count
 params = $0.split('_')
 if params.size != 3
-  puts "Incorrect number of parameters"
+  puts 'Incorrect number of parameters'
   exit 1
 end
 
@@ -59,11 +58,11 @@ host = params[1]
 stat = params[2]
 
 unless ENV['debug'].nil?
-  puts "user = " + user
-  puts "pass = " + pass
-  puts "host = " + host
-  puts "port = " + port
-  puts "stat = " + stat
+  puts 'user = ' + user
+  puts 'pass = ' + pass
+  puts 'host = ' + host
+  puts 'port = ' + port
+  puts 'stat = ' + stat
 end
 
 # Dump the graph configuration data
@@ -101,7 +100,7 @@ s = response.body
 
 # Make sure we got the page successfully
 if response.code != '200'
-  puts "Getting web page failed:"
+  puts 'Getting web page failed:'
   case response.code
   when '401'
     puts 'Probably because the username and password are incorrect'
@@ -124,12 +123,12 @@ when 'syncrate'
 when 'attenuation'
   a = s.scan(/.*share\.lineatt.*\n.*share\.down[^0-9]*([0-9]+).*share\.up[^0-9]*([0-9]+).*$/)
   b, c = a[0]
-  puts 'down.value ' + (b.to_i).to_s + "\n" + 'up.value ' + (c.to_i).to_s
+  puts 'down.value ' + b.to_i.to_s + "\n" + 'up.value ' + c.to_i.to_s
   exit 0
 when 'margin', 'noise'
   a = s.scan(/.*share\.noise.*\n.*share\.down[^0-9]*([0-9]+).*share\.up[^0-9]*([0-9]+).*$/)
   b, c = a[0]
-  puts 'down.value ' + (b.to_i).to_s + "\n" + 'up.value ' + (c.to_i).to_s
+  puts 'down.value ' + b.to_i.to_s + "\n" + 'up.value ' + c.to_i.to_s
   exit 0
 else
   puts 'Statistic ' + stat.to_s + ' not known, would you like me to fabricate it for you?'
diff --git a/plugins/router/d-link-dir-655-router-statistics-plugin b/plugins/router/d-link-dir-655-router-statistics-plugin
index b8712d49..af8705a5 100755
--- a/plugins/router/d-link-dir-655-router-statistics-plugin
+++ b/plugins/router/d-link-dir-655-router-statistics-plugin
@@ -32,13 +32,13 @@ require 'digest/md5'
 require 'nokogiri'
 
 def output
-  nics = Hash.new
-  nics["LAN"] = Hash.new
-  nics["WAN"] = Hash.new
-  nics["WLAN"] = Hash.new
-  password = ENV['router_password'] || ""
-  router_path = ENV['router_ip_address'] || "10.0.0.1"
-  router_path = "http://" + router_path
+  nics = {}
+  nics['LAN'] = {}
+  nics['WAN'] = {}
+  nics['WLAN'] = {}
+  password = ENV['router_password'] || ''
+  router_path = ENV['router_ip_address'] || '10.0.0.1'
+  router_path = 'http://' + router_path
   agent = Mechanize.new
   x = agent.get(router_path)
   salt = x.body.match(/salt = "(.*)"/)[1]
@@ -48,7 +48,7 @@ def output
   padded_password = password + "\x01" * pad_size
 
   # pad it the rest of the way, length 64 for user
-  salted_password = salt + padded_password + ("\x01" * (63 - salt.length - padded_password.length)) + "U"
+  salted_password = salt + padded_password + ("\x01" * (63 - salt.length - padded_password.length)) + 'U'
   login_hash = salt + Digest::MD5.hexdigest(salted_password)
 
   # authenticate against the router using the hash that we just built
@@ -61,87 +61,87 @@ def output
   doc.xpath('//interface').each do |interface|
     children = interface.children
     name = children.search('name')[0].text
-    nics[name]["packets_sent"] = children.search('packets_sent')[0].text
-    nics[name]["packets_received"] = children.search('packets_received')[0].text
-    nics[name]["tx_dropped"] = children.search('tx_dropped')[0].text
+    nics[name]['packets_sent'] = children.search('packets_sent')[0].text
+    nics[name]['packets_received'] = children.search('packets_received')[0].text
+    nics[name]['tx_dropped'] = children.search('tx_dropped')[0].text
     begin
-      nics[name]["tx_collisions"] = children.search('tx_collisions')[0].text
+      nics[name]['tx_collisions'] = children.search('tx_collisions')[0].text
     rescue Exception
-      nics[name]["tx_collisions"] = "0"
+      nics[name]['tx_collisions'] = '0'
     end
-    nics[name]["rx_dropped"] = children.search('rx_dropped')[0].text
-    nics[name]["rx_errors"] = children.search('rx_errors')[0].text
+    nics[name]['rx_dropped'] = children.search('rx_dropped')[0].text
+    nics[name]['rx_errors'] = children.search('rx_errors')[0].text
   end
 
   # get wifi associations and print out info for munin graph
-  puts "multigraph clients"
+  puts 'multigraph clients'
   clients_xml = agent.get("#{router_path}/wifi_assoc.xml").body
   j = 0
   doc = Nokogiri::XML(clients_xml.to_s)
-  doc.xpath('//assoc').each do |assoc|
+  doc.xpath('//assoc').each do |_assoc|
     j += 1
   end
-  puts "wifi_assoc.value " + j.to_s
+  puts 'wifi_assoc.value ' + j.to_s
 
   # get dhcp clients and print out info for munin graph
   clients_xml = agent.get("#{router_path}/dhcp_clients.xml").body
   j = 0
   doc = Nokogiri::XML(clients_xml.to_s)
-  doc.xpath('//client').each do |client|
+  doc.xpath('//client').each do |_client|
     j += 1
   end
-  puts "dhcp_clients.value " + j.to_s
+  puts 'dhcp_clients.value ' + j.to_s
 
-  puts "multigraph uptime"
+  puts 'multigraph uptime'
   # get uptime of connection
   clients_xml = agent.get("#{router_path}/wan_connection_status.xml").body
   doc = Nokogiri::XML(clients_xml.to_s)
   uptime = doc.children.search('wan_interface_up_time_0')[0].text
-  puts "uptime.value " + sprintf("%.2f", (Float(uptime) / 86400))
+  puts 'uptime.value ' + format('%.2f', (Float(uptime) / 86_400))
 
   # graph overall interface packets transferred per interval
-  puts "multigraph if_packets"
-  for i in ["LAN", "WAN", "WLAN"] do
-    puts "#{i}_recv.value " + nics[i]["packets_received"]
-    puts "#{i}_send.value " + nics[i]["packets_sent"]
+  puts 'multigraph if_packets'
+  %w[LAN WAN WLAN].each do |i|
+    puts "#{i}_recv.value " + nics[i]['packets_received']
+    puts "#{i}_send.value " + nics[i]['packets_sent']
   end
 
   # graph overall interface dropped packets per interval
-  puts "multigraph if_drop"
-  for i in ["LAN", "WAN", "WLAN"] do
-    puts "#{i}_recv.value " + nics[i]["rx_dropped"]
-    puts "#{i}_send.value " + nics[i]["tx_dropped"]
+  puts 'multigraph if_drop'
+  %w[LAN WAN WLAN].each do |i|
+    puts "#{i}_recv.value " + nics[i]['rx_dropped']
+    puts "#{i}_send.value " + nics[i]['tx_dropped']
   end
 
   # graph overall interface collisions & errors per interval
-  puts "multigraph if_collerr"
-  for i in ["LAN", "WAN", "WLAN"] do
-    puts "#{i}_coll.value " + nics[i]["tx_collisions"]
-    puts "#{i}_err.value " + nics[i]["rx_errors"]
+  puts 'multigraph if_collerr'
+  %w[LAN WAN WLAN].each do |i|
+    puts "#{i}_coll.value " + nics[i]['tx_collisions']
+    puts "#{i}_err.value " + nics[i]['rx_errors']
   end
 
   # graph stats for each interface
-  for i in ["LAN", "WAN", "WLAN"] do
+  %w[LAN WAN WLAN].each do |i|
     puts "multigraph if_packets.#{i}"
-    puts "send.value " + nics[i]["packets_sent"]
-    puts "recv.value " + nics[i]["packets_received"]
+    puts 'send.value ' + nics[i]['packets_sent']
+    puts 'recv.value ' + nics[i]['packets_received']
     puts "multigraph if_drop.#{i}"
-    puts "send.value " + nics[i]["tx_dropped"]
-    puts "recv.value " + nics[i]["rx_dropped"]
+    puts 'send.value ' + nics[i]['tx_dropped']
+    puts 'recv.value ' + nics[i]['rx_dropped']
     puts "multigraph if_collerr.#{i}"
-    puts "coll.value " + nics[i]["tx_collisions"]
-    puts "err.value " + nics[i]["rx_errors"]
+    puts 'coll.value ' + nics[i]['tx_collisions']
+    puts 'err.value ' + nics[i]['rx_errors']
   end
 end
 
 def config
   # build the configuration for graphs
-  puts "multigraph if_packets"
+  puts 'multigraph if_packets'
   puts 'graph_title D-Link DIR-655 interface traffic'
   puts 'graph_category network'
   puts 'graph_order LAN_recv LAN_send WAN_recv WAN_send WLAN_recv WLAN_send'
   puts 'graph_vlabel packets in (-) / out (+) per ${graph_period}'
-  for i in ["LAN", "WAN", "WLAN"] do
+  %w[LAN WAN WLAN].each do |i|
     puts "#{i}_recv.type DERIVE"
     puts "#{i}_recv.graph no"
     puts "#{i}_recv.min 0"
@@ -151,12 +151,12 @@ def config
     puts "#{i}_send.min 0"
   end
 
-  puts "multigraph if_drop"
+  puts 'multigraph if_drop'
   puts 'graph_title D-Link DIR-655 interface drops'
   puts 'graph_category network'
   puts 'graph_order LAN_recv LAN_send WAN_recv WAN_send WLAN_recv WLAN_send'
   puts 'graph_vlabel packets / ${graph_period}'
-  for i in ["LAN", "WAN", "WLAN"] do
+  %w[LAN WAN WLAN].each do |i|
     puts "#{i}_recv.type DERIVE"
     puts "#{i}_recv.graph no"
     puts "#{i}_recv.min 0"
@@ -166,12 +166,12 @@ def config
     puts "#{i}_send.min 0"
   end
 
-  puts "multigraph if_collerr"
+  puts 'multigraph if_collerr'
   puts 'graph_title D-Link DIR-655 interface collisions & errors'
   puts 'graph_category network'
   puts 'graph_order LAN_coll LAN_err WAN_coll WAN_err WLAN_coll WLAN_coll'
   puts 'graph_vlabel packets / ${graph_period}'
-  for i in ["LAN", "WAN", "WLAN"] do
+  %w[LAN WAN WLAN].each do |i|
     puts "#{i}_coll.label #{i} collisions"
     puts "#{i}_coll.type DERIVE"
     puts "#{i}_coll.min 0"
@@ -180,26 +180,26 @@ def config
     puts "#{i}_err.min 0"
   end
 
-  puts "multigraph clients"
-  puts "graph_title D-Link DIR-655 client information"
-  puts "graph_category system"
-  puts "graph_order dhcp_clients wifi_assoc"
-  puts "graph_vlabel number of clients"
-  puts "dhcp_clients.label DHCP clients"
-  puts "dhcp_clients.type GAUGE"
-  puts "dhcp_clients.min 0"
-  puts "wifi_assoc.label wifi clients"
-  puts "wifi_assoc.type GAUGE"
-  puts "wifi_assoc.min 0"
+  puts 'multigraph clients'
+  puts 'graph_title D-Link DIR-655 client information'
+  puts 'graph_category system'
+  puts 'graph_order dhcp_clients wifi_assoc'
+  puts 'graph_vlabel number of clients'
+  puts 'dhcp_clients.label DHCP clients'
+  puts 'dhcp_clients.type GAUGE'
+  puts 'dhcp_clients.min 0'
+  puts 'wifi_assoc.label wifi clients'
+  puts 'wifi_assoc.type GAUGE'
+  puts 'wifi_assoc.min 0'
 
-  puts "multigraph uptime"
-  puts "graph_title Uptime"
+  puts 'multigraph uptime'
+  puts 'graph_title Uptime'
   puts 'graph_vlabel uptime in days'
   puts 'graph_category system'
   puts 'uptime.label uptime'
   puts 'uptime.draw AREA'
 
-  for i in ["LAN", "WAN", "WLAN"] do
+  %w[LAN WAN WLAN].each do |i|
     puts "multigraph if_packets.#{i}"
     puts "graph_title D-Link DIR-655 #{i} traffic"
     puts 'graph_category network'
@@ -243,8 +243,8 @@ def config
 end
 
 # main
-if ARGV.length == 1 and ARGV[0] == 'config'
-  config()
+if (ARGV.length == 1) && (ARGV[0] == 'config')
+  config
 else
-  output()
+  output
 end
diff --git a/plugins/router/snmp__linksys_poe b/plugins/router/snmp__linksys_poe
index a79a27ea..72d6e0bb 100755
--- a/plugins/router/snmp__linksys_poe
+++ b/plugins/router/snmp__linksys_poe
@@ -65,45 +65,43 @@ rights to this plugin are waived.  Do with it as you wish.
 
 require 'snmp'
 
-idx_oid = "enterprises.3955.89.108.1.1.2"
-max_oid = "enterprises.3955.89.108.1.1.6"
-cur_oid = "enterprises.3955.89.108.1.1.5"
+idx_oid = 'enterprises.3955.89.108.1.1.2'
+max_oid = 'enterprises.3955.89.108.1.1.6'
+cur_oid = 'enterprises.3955.89.108.1.1.5'
 
-community = ENV['community'] || "public"
+community = ENV['community'] || 'public'
 version = ENV['version'] == '1' ? :SNMPv1 : :SNMPv2c
 
 case ARGV[0]
-when "snmpconf"
-  puts "require 1.3.6.1.4.1.3955.89.108.1.1.2.1. [0-9]"
-  puts "require 1.3.6.1.4.1.3955.89.108.1.1.5.1. [0-9]"
-  puts "require 1.3.6.1.4.1.3955.89.108.1.1.6.1. [0-9]"
-  exit 0;
-when "config"
+when 'snmpconf'
+  puts 'require 1.3.6.1.4.1.3955.89.108.1.1.2.1. [0-9]'
+  puts 'require 1.3.6.1.4.1.3955.89.108.1.1.5.1. [0-9]'
+  puts 'require 1.3.6.1.4.1.3955.89.108.1.1.6.1. [0-9]'
+  exit 0
+when 'config'
   host = $0.match('^(?:|.*\/)snmp_([^_]+)')[1]
   puts "host_name #{host}"
-  puts "graph_title PoE Power Usage"
-  puts "graph_vlabel Watts"
-  puts "graph_category sensors"
+  puts 'graph_title PoE Power Usage'
+  puts 'graph_vlabel Watts'
+  puts 'graph_category sensors'
   max_current = 0
-  SNMP::Manager.open(:Host => host,
-                     :Community => community,
-                     :Version => version) do |manager|
+  SNMP::Manager.open(Host: host,
+                     Community: community,
+                     Version: version) do |manager|
     manager.walk([idx_oid, max_oid]) do |row|
       puts "iface_#{row[0].value}.label Port #{row[0].value}"
       puts "iface_#{row[0].value}.cdef iface_#{row[0].value},1000,/"
       puts "iface_#{row[0].value}.line #{row[1].value.to_f / 1000}"
-      if row[1].value > max_current
-        max_current = row[1].value
-      end
+      max_current = row[1].value if row[1].value > max_current
     end
   end
   puts "graph_args --upper-limit #{max_current.to_f / 1000}"
   exit 0
 else
   host = $0.match('^(?:|.*\/)snmp_([^_]+)')[1]
-  SNMP::Manager.open(:Host => host,
-                     :Community => community,
-                     :Version => version) do |manager|
+  SNMP::Manager.open(Host: host,
+                     Community: community,
+                     Version: version) do |manager|
     manager.walk([idx_oid, cur_oid]) do |row|
       puts "iface_#{row[0].value}.value #{row[1].value}"
     end
diff --git a/plugins/snmp/snmp_room_alert_ b/plugins/snmp/snmp_room_alert_
index d2e63dec..a1822add 100755
--- a/plugins/snmp/snmp_room_alert_
+++ b/plugins/snmp/snmp_room_alert_
@@ -1,6 +1,4 @@
 #!/usr/bin/env ruby
-# encoding: utf-8
-
 # Plugin to monitor Room Alert 11E environmental units.
 # Requires ruby and the ruby SNMP library.
 #
@@ -34,31 +32,31 @@
 
 require 'snmp'
 
-base_oid = "enterprises.20916.1.3.1"
+base_oid = 'enterprises.20916.1.3.1'
 
 case $0.match('[^_]+$')[0]
-when "temperature"
+when 'temperature'
   subchannel = 1
-  name = "temperature"
-  label = "°C"
-  letter = "t"
-when "humidity"
+  name = 'temperature'
+  label = '°C'
+  letter = 't'
+when 'humidity'
   subchannel = 3
-  name = "humidity"
-  label = "% Relative Humidity"
-  letter = "h"
+  name = 'humidity'
+  label = '% Relative Humidity'
+  letter = 'h'
 else
   exit 1
 end
 
 def is_vb_valid(vb, subchannel)
-  return (vb.name[-1] == 0 and vb.name[-2] == subchannel and vb.value > 1)
+  (vb.name[-1] == 0 and vb.name[-2] == subchannel and vb.value > 1)
 end
 
 def field_name(unit, vb, letter)
   clean_unit = unit.gsub(/[.-]/, '_')
   sensor = vb.name[-3].to_s
-  return "#{clean_unit}_#{letter}#{sensor}"
+  "#{clean_unit}_#{letter}#{sensor}"
 end
 
 def label(unit, vb)
@@ -67,28 +65,24 @@ def label(unit, vb)
   label = "#{unit} " + (ENV["label_#{clean_unit}_#{sensor}"] || "sensor #{sensor}")
 end
 
-units = (ENV['units'] || "").split(/\s+/)
-community = ENV['community'] || "public"
+units = (ENV['units'] || '').split(/\s+/)
+community = ENV['community'] || 'public'
 
 case ARGV[0]
-when "autoconf"
-  puts "no"
+when 'autoconf'
+  puts 'no'
   exit 0
-when "config"
+when 'config'
   puts "graph_title Room Alert 11E units (#{name} probes)"
   puts "graph_vlabel #{label}"
-  puts "graph_category sensors"
-  if name == "humidity"
-    puts "graph_args --lower-limit 0 --upper-limit 100"
-  end
+  puts 'graph_category sensors'
+  puts 'graph_args --lower-limit 0 --upper-limit 100' if name == 'humidity'
   units.each do |unit|
-    SNMP::Manager.open(:Host => unit,
-                       :Community => community,
-                       :Version => :SNMPv1) do |manager|
+    SNMP::Manager.open(Host: unit,
+                       Community: community,
+                       Version: :SNMPv1) do |manager|
       manager.walk(base_oid) do |vb|
-        if not is_vb_valid(vb, subchannel)
-          next
-        end
+        next unless is_vb_valid(vb, subchannel)
 
         puts "#{field_name(unit, vb, letter)}.label #{label(unit, vb)}"
       end
@@ -98,13 +92,11 @@ when "config"
 end
 
 units.each do |unit|
-  SNMP::Manager.open(:Host => unit,
-                     :Community => community,
-                     :Version => :SNMPv1) do |manager|
+  SNMP::Manager.open(Host: unit,
+                     Community: community,
+                     Version: :SNMPv1) do |manager|
     manager.walk(base_oid) do |vb|
-      if not is_vb_valid(vb, subchannel)
-        next
-      end
+      next unless is_vb_valid(vb, subchannel)
 
       puts "#{field_name(unit, vb, letter)}.value #{vb.value.to_f / 100}"
     end
diff --git a/plugins/ssh/sshd_invalid_countries_ruby b/plugins/ssh/sshd_invalid_countries_ruby
index 2664ec78..040a79d8 100755
--- a/plugins/ssh/sshd_invalid_countries_ruby
+++ b/plugins/ssh/sshd_invalid_countries_ruby
@@ -33,7 +33,6 @@ env.loadpath /usr/local/lib/ruby/gems/1.9.1/gems/geoip-0.8.8/lib/
 
 =end
 
-
 require (ENV['loadpath'] || '') + 'geoip'
 
 SYSLOG   = ENV['syslog'] || '/var/log/secure'
@@ -45,28 +44,26 @@ AWK_CMD  = 'awk \'/sshd\[.*Did not receive identification string/{print $12} ' +
 def getInvalids
   c = {}
   wholeips = `#{AWK_CMD}`.split("\n")
-  uniqueips = wholeips.inject({}) do |hash, key|
-    hash.include?(key) ? hash[key] += 1 : hash[key] = 1;
-    hash
+  uniqueips = wholeips.each_with_object({}) do |key, hash|
+    hash.include?(key) ? hash[key] += 1 : hash[key] = 1
   end
   geoip = GeoIP.new(GEOIP_DB)
   uniqueips.each do |ip, cnt|
     begin
       country = geoip.country(ip)[5]
       c[country] = c[country] ? c[country] + cnt : cnt
-    rescue
+    rescue StandardError
       c['Unknown'] = c['Unknown'] ? c['Unknown'] + cnt : cnt
     end
   end
-  c = c.to_a.sort { |a, b| a[0] <=> b[0] }
-  c
+  c.to_a.sort { |a, b| a[0] <=> b[0] }
 end
 
 case ARGV[0]
 when 'autoconf'
   begin
     fh = open(SYSLOG, 'r')
-  rescue
+  rescue StandardError
     puts 'no'
     exit 0
   else
@@ -79,7 +76,7 @@ when 'config'
   puts 'graph_vlabel number of invalid access per country'
   puts 'graph_category security'
   puts 'graph_info This graph shows the countries of invalid access to sshd.'
-  getInvalids.each { |k, v| puts k + '.label ' + k }
+  getInvalids.each { |k, _v| puts k + '.label ' + k }
   exit 0
 else
   getInvalids.each { |k, v| puts k + '.value ' + v.to_s }
diff --git a/plugins/thin/thin_memory b/plugins/thin/thin_memory
index 867e52ce..f1557fc4 100755
--- a/plugins/thin/thin_memory
+++ b/plugins/thin/thin_memory
@@ -35,21 +35,19 @@ You should have received a copy of the GNU General Public License along
 with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
-
 #%# family=auto
 #%# capabilities=autoconf
 
 =end
 
-
 module Munin
   class ThinProcessMemory
     # run main method
     def run
-      instances = get_pids()
+      instances = get_pids
       instances.each do |instance|
-        pid, port = instance.split("|")
-        rss = (pid_rss(pid).to_i) / 1024
+        pid, port = instance.split('|')
+        rss = pid_rss(pid).to_i / 1024
         puts "thin_#{port}.value #{rss}"
       end
     end
@@ -57,10 +55,10 @@ module Munin
     # only get the memory for each pid
     def pid_rss(pid)
       res = `grep "VmRSS" /proc/#{pid}/status`.split[1]
-      if res.match("cannot access")
-        return nil
+      if res.match('cannot access')
+        nil
       else
-        return res
+        res
       end
     end
 
@@ -70,7 +68,7 @@ module Munin
     end
 
     def autoconf
-      get_pids().length > 0
+      get_pids.length > 0
     end
   end
 end
@@ -78,26 +76,26 @@ end
 mpm = Munin::ThinProcessMemory.new
 
 case ARGV[0]
-when "config"
-  puts "graph_title Thin Memory"
-  puts "graph_vlabel RSS"
-  puts "graph_category webserver"
-  puts "graph_args --base 1024 -l 0"
-  puts "graph_scale yes"
-  puts "graph_info Tracks the size of individual thin processes"
+when 'config'
+  puts 'graph_title Thin Memory'
+  puts 'graph_vlabel RSS'
+  puts 'graph_category webserver'
+  puts 'graph_args --base 1024 -l 0'
+  puts 'graph_scale yes'
+  puts 'graph_info Tracks the size of individual thin processes'
   mpm.get_pids.each do |instance|
-    pid, port = instance.split("|")
+    pid, port = instance.split('|')
     puts "thin_#{port}.label thin_#{port}"
     puts "thin_#{port}.info Process memory"
     puts "thin_#{port}.type GAUGE"
     puts "thin_#{port}.min 0"
   end
-when "autoconf"
+when 'autoconf'
   if mpm.autoconf
-    puts "yes"
+    puts 'yes'
     exit 0
   end
-  puts "no"
+  puts 'no'
   exit 0
 else
   mpm.run
diff --git a/plugins/thin/thin_threads b/plugins/thin/thin_threads
index 6be71004..224edbf2 100755
--- a/plugins/thin/thin_threads
+++ b/plugins/thin/thin_threads
@@ -36,20 +36,18 @@ You should have received a copy of the GNU General Public License along
 with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
-
 #%# family=auto
 #%# capabilities=autoconf
 
 =end
 
-
 module Munin
   class ThinThreads
     def run
-      instances = get_pids()
+      instances = get_pids
       instances.each do |instance|
-        pid, port = instance.split("|")
-        rss = (get_threads(pid).to_i)
+        pid, port = instance.split('|')
+        rss = get_threads(pid).to_i
         puts "thin_#{port}.value #{rss}"
       end
     end
@@ -60,11 +58,11 @@ module Munin
     # TODO: make this work on OSX and Solaris,
     #        so the whole unix gang is happy ;)
     def get_threads(pid)
-      res = `grep "Threads" /proc/#{pid}/status | cut -d ":" -f2`.gsub(/\s+/, "")
-      if res.match("cannot access")
-        return nil
+      res = `grep "Threads" /proc/#{pid}/status | cut -d ":" -f2`.gsub(/\s+/, '')
+      if res.match('cannot access')
+        nil
       else
-        return res
+        res
       end
     end
 
@@ -74,7 +72,7 @@ module Munin
     end
 
     def autoconf
-      get_pids().length > 0
+      get_pids.length > 0
     end
   end
 end
@@ -82,26 +80,26 @@ end
 mpm = Munin::ThinThreads.new
 
 case ARGV[0]
-when "config"
-  puts "graph_title Thin Threads"
-  puts "graph_vlabel Threads"
-  puts "graph_category webserver"
-  puts "graph_args -l 0"
-  puts "graph_scale yes"
-  puts "graph_info Tracks how many threads per thin processes"
+when 'config'
+  puts 'graph_title Thin Threads'
+  puts 'graph_vlabel Threads'
+  puts 'graph_category webserver'
+  puts 'graph_args -l 0'
+  puts 'graph_scale yes'
+  puts 'graph_info Tracks how many threads per thin processes'
   mpm.get_pids.each do |instance|
-    pid, port = instance.split("|")
+    pid, port = instance.split('|')
     puts "thin_#{port}.label thin_#{port}"
     puts "thin_#{port}.info Threads per Thin process"
     puts "thin_#{port}.type GAUGE"
     puts "thin_#{port}.min 0"
   end
-when "autoconf"
+when 'autoconf'
   if mpm.autoconf
-    puts "yes"
+    puts 'yes'
     exit 0
   end
-  puts "no"
+  puts 'no'
   exit 0
 else
   mpm.run
diff --git a/plugins/thin/thins_peak_memory b/plugins/thin/thins_peak_memory
index 56d547a6..ee289d8a 100755
--- a/plugins/thin/thins_peak_memory
+++ b/plugins/thin/thins_peak_memory
@@ -33,20 +33,18 @@ You should have received a copy of the GNU General Public License along
 with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
-
 #%# family=auto
 #%# capabilities=autoconf
 
 =end
 
-
 module Munin
   class ThinPeakMemory
     def run
-      instances = get_pids()
+      instances = get_pids
       instances.each do |instance|
-        pid, port = instance.split("|")
-        hwm = (pid_hwm(pid).to_i) / 1024
+        pid, port = instance.split('|')
+        hwm = pid_hwm(pid).to_i / 1024
         puts "thin_#{port}.value #{hwm}"
       end
     end
@@ -59,10 +57,10 @@ module Munin
     #      so the whole unix gang is happy ;)
     def pid_hwm(pid)
       res = `grep "VmHWM" /proc/#{pid}/status`.split[1]
-      if res.match("cannot access")
-        return nil
+      if res.match('cannot access')
+        nil
       else
-        return res
+        res
       end
     end
 
@@ -72,7 +70,7 @@ module Munin
     end
 
     def autoconf
-      get_pids().length > 0
+      get_pids.length > 0
     end
   end
 end
@@ -80,26 +78,26 @@ end
 mpm = Munin::ThinPeakMemory.new
 
 case ARGV[0]
-when "config"
-  puts "graph_title Thin Peak Memory (High Water Mark)"
-  puts "graph_vlabel HWM"
-  puts "graph_category webserver"
-  puts "graph_args -l 0"
-  puts "graph_scale yes"
-  puts "graph_info Tracks the peak memory of thin processes, aka High Water Mark."
+when 'config'
+  puts 'graph_title Thin Peak Memory (High Water Mark)'
+  puts 'graph_vlabel HWM'
+  puts 'graph_category webserver'
+  puts 'graph_args -l 0'
+  puts 'graph_scale yes'
+  puts 'graph_info Tracks the peak memory of thin processes, aka High Water Mark.'
   mpm.get_pids.each do |instance|
-    pid, port = instance.split("|")
+    pid, port = instance.split('|')
     puts "thin_#{port}.label thin_#{port}"
     puts "thin_#{port}.info Peak Memory"
     puts "thin_#{port}.type GAUGE"
     puts "thin_#{port}.min 0"
   end
-when "autoconf"
+when 'autoconf'
   if mpm.autoconf
-    puts "yes"
+    puts 'yes'
     exit 0
   end
-  puts "no"
+  puts 'no'
   exit 0
 else
   mpm.run
diff --git a/plugins/unicorn/unicorn_memory_status b/plugins/unicorn/unicorn_memory_status
index a32543d9..01f08953 100755
--- a/plugins/unicorn/unicorn_memory_status
+++ b/plugins/unicorn/unicorn_memory_status
@@ -8,10 +8,10 @@
 #
 
 # set path to your rails app
-RAILS_ROOT = "/path/to/rails/app"
+RAILS_ROOT = '/path/to/rails/app'.freeze
 
 # set name to your unicorn.pid
-PID_NAME = "unicorn.pid"
+PID_NAME = 'unicorn.pid'.freeze
 
 module Munin
   class UnicornMemoryStatus
@@ -30,8 +30,9 @@ module Munin
       ps_output = `ps w --ppid #{master_pid}`
       ps_output.split("\n").each do |line|
         chunks = line.strip.split(/\s+/, 5)
-        pid, pcmd = chunks[0], chunks[4]
-        next if pid !~ /\A\d+\z/ or pcmd !~ /worker/
+        pid = chunks[0]
+        pcmd = chunks[4]
+        next if pid !~ /\A\d+\z/ || pcmd !~ /worker/
 
         result << pid.to_i
       end
@@ -42,18 +43,18 @@ module Munin
       result = 0
       memory = memory_usage
       result += memory[:master][master_pid]
-      memory[:worker].each do |pid, worker_memory|
+      memory[:worker].each do |_pid, worker_memory|
         result += worker_memory
       end
       result
     end
 
     def memory_usage
-      result = { :master => { master_pid => nil }, :worker => {} }
+      result = { master: { master_pid => nil }, worker: {} }
       ps_output = `ps auxw | grep unicorn`
       ps_output.split("\n").each do |line|
         chunks = line.strip.split(/\s+/, 11)
-        pid, pmem_rss, _ = chunks.values_at(1, 5, 10)
+        pid, pmem_rss, = chunks.values_at(1, 5, 10)
         pmem = pmem_rss.to_i * 1024
         pid = pid.to_i
 
@@ -69,15 +70,15 @@ module Munin
 end
 
 case ARGV[0]
-when "autoconf"
-  puts "yes"
-when "config"
+when 'autoconf'
+  puts 'yes'
+when 'config'
   puts "graph_title Unicorn [#{File.basename(__FILE__).gsub(/^unicorn_memory_status_/, '')}] - Memory usage"
-  puts "graph_args --base 1024 -l 0"
-  puts "graph_vlabel bytes"
-  puts "graph_category webserver"
-  puts "total_memory.label total_memory"
-  puts "total_memory.draw LINE2"
+  puts 'graph_args --base 1024 -l 0'
+  puts 'graph_vlabel bytes'
+  puts 'graph_category webserver'
+  puts 'total_memory.label total_memory'
+  puts 'total_memory.draw LINE2'
 else
   m = Munin::UnicornMemoryStatus.new(ENV['rails_root'] || RAILS_ROOT, ENV['pid_name'] || PID_NAME)
   puts "total_memory.value #{m.total_memory}"
diff --git a/plugins/unicorn/unicorn_status b/plugins/unicorn/unicorn_status
index 7db1911b..547116d3 100755
--- a/plugins/unicorn/unicorn_status
+++ b/plugins/unicorn/unicorn_status
@@ -8,10 +8,10 @@
 #
 
 # set path to your rails app
-RAILS_ROOT = "/path/to/rails/app"
+RAILS_ROOT = '/path/to/rails/app'.freeze
 
 # set name to your unicorn.pid
-PID_NAME = "unicorn.pid"
+PID_NAME = 'unicorn.pid'.freeze
 
 module Munin
   class UnicornStatus
@@ -30,8 +30,9 @@ module Munin
       ps_output = `ps w --ppid #{master_pid}`
       ps_output.split("\n").each do |line|
         chunks = line.strip.split(/\s+/, 5)
-        pid, pcmd = chunks[0], chunks[4]
-        next if pid !~ /\A\d+\z/ or pcmd !~ /worker/
+        pid = chunks[0]
+        pcmd = chunks[4]
+        next if pid !~ /\A\d+\z/ || pcmd !~ /worker/
 
         result << pid.to_i
       end
@@ -67,15 +68,15 @@ module Munin
 end
 
 case ARGV[0]
-when "autoconf"
-  puts "yes"
-when "config"
+when 'autoconf'
+  puts 'yes'
+when 'config'
   puts "graph_title Unicorn [#{File.basename(__FILE__).gsub(/^unicorn_status_/, '')}] - Status"
-  puts "graph_args -l 0"
-  puts "graph_vlabel number of workers"
-  puts "graph_category webserver"
-  puts "total_worker.label total_workers"
-  puts "idle_worker.label idle_workers"
+  puts 'graph_args -l 0'
+  puts 'graph_vlabel number of workers'
+  puts 'graph_category webserver'
+  puts 'total_worker.label total_workers'
+  puts 'idle_worker.label idle_workers'
 else
   m = Munin::UnicornStatus.new(ENV['rails_root'] || RAILS_ROOT, ENV['pid_name'] || PID_NAME)
   puts "total_worker.value #{m.worker_count}"
diff --git a/plugins/voldemort/voldemort b/plugins/voldemort/voldemort
index c11f9469..4c279f84 100755
--- a/plugins/voldemort/voldemort
+++ b/plugins/voldemort/voldemort
@@ -6,39 +6,37 @@
 require 'rubygems'
 require 'jmx4r'
 
-
-#%# family=auto
-#%# capabilities=autoconf
+# %# family=auto
+# %# capabilities=autoconf
 
 # friendly name => result of listPerfStatsKeys via JMX
 keys = {
-  "Throughput" => { "vlabel" => "rate",
-                    "type" => "ABSOLUTE",
-                    "values" => ["all_operation_throughput","delete_throughput", "get_all_throughput", "get_throughput", "put_throughput"]
-                  },
-  "Number of Calls" => { "vlabel" => "counts",
-                    "type" => "COUNTER",
-                    "values" => ["number_of_calls_to_delete","number_of_calls_to_get", "number_of_calls_to_get_all",
-                                 "number_of_calls_to_put", "number_of_exceptions"]
-                  }
+  'Throughput' => { 'vlabel' => 'rate',
+                    'type' => 'ABSOLUTE',
+                    'values' => %w[all_operation_throughput delete_throughput get_all_throughput get_throughput put_throughput] },
+  'Number of Calls' => { 'vlabel' => 'counts',
+                         'type' => 'COUNTER',
+                         'values' => %w[number_of_calls_to_delete number_of_calls_to_get number_of_calls_to_get_all
+                                        number_of_calls_to_put number_of_exceptions] }
 }
 
-if ARGV[0] == "config"
+case ARGV[0]
+when 'config'
   keys.each_key do |key|
-    puts "multigraph voldemort_#{key.gsub(" ", "_")}"
+    puts "multigraph voldemort_#{key.gsub(' ', '_')}"
     puts "graph_title #{key}"
-    puts "graph_scale no"
-    puts "graph_category search"
+    puts 'graph_scale no'
+    puts 'graph_category search'
     puts "graph_vlabel #{keys[key]['vlabel']}"
-    for data in keys[key]['values'] do
+    keys[key]['values'].each do |data|
       puts "#{data}.type #{keys[key]['type']}"
-      puts "#{data}.label #{data.gsub("_", " ")}"
+      puts "#{data}.label #{data.gsub('_', ' ')}"
     end
     puts
   end
   exit 0
-elsif ARGV[0] == "autoconf"
-  puts "yes"
+when 'autoconf'
+  puts 'yes'
   exit 0
 else
 
@@ -49,16 +47,18 @@ else
   # Make the platform MBean server able to work with JBossAS MBeans
   # JAVA_OPTS="$JAVA_OPTS -Djavax.management.builder.initial=org.jboss.system.server.jmx.MBeanServerBuilderImpl"
   # JBOSS_CLASSPATH="/opt/webtrends/jboss/bin/mbean"
-  JMX::MBean.establish_connection :port => 5400
-  vs = JMX::MBean.find_by_name "voldemort.store.stats.aggregate:type=aggregate-perf"
+  JMX::MBean.establish_connection port: 5400
+  vs = JMX::MBean.find_by_name 'voldemort.store.stats.aggregate:type=aggregate-perf'
 
   keys.each_key do |key|
-    puts "multigraph voldemort_#{key.gsub(" ", "_")}"
+    puts "multigraph voldemort_#{key.gsub(' ', '_')}"
     for data in keys[key]['values'] do
-      puts "#{data}.value #{begin vs.send("#{data}") rescue 0 end}"
+      puts "#{data}.value #{begin begin
+                                    vs.send(data.to_s)
+                                  rescue StandardError
+                                    0
+                                  end end}"
     end
     puts
   end
 end
-
-