Fix more rubocop detected style issues
authorTom Hughes <tom@compton.nu>
Mon, 2 Feb 2015 23:55:46 +0000 (23:55 +0000)
committerTom Hughes <tom@compton.nu>
Tue, 3 Feb 2015 00:57:18 +0000 (00:57 +0000)
71 files changed:
cookbooks/accounts/recipes/default.rb
cookbooks/apache/providers/conf.rb
cookbooks/apache/providers/module.rb
cookbooks/apache/recipes/default.rb
cookbooks/chef/libraries/compare_versions.rb
cookbooks/chef/libraries/git.rb
cookbooks/chef/libraries/subversion.rb
cookbooks/chef/recipes/default.rb
cookbooks/chef/recipes/server.rb
cookbooks/db/recipes/master.rb
cookbooks/dev/recipes/default.rb
cookbooks/dns/recipes/default.rb
cookbooks/exim/recipes/default.rb
cookbooks/git/recipes/server.rb
cookbooks/hardware/attributes/default.rb
cookbooks/hardware/recipes/default.rb
cookbooks/mediawiki/definitions/mediawiki_extension.rb
cookbooks/mediawiki/definitions/mediawiki_site.rb
cookbooks/mediawiki/recipes/default.rb
cookbooks/munin/files/default/plugins/api_calls_
cookbooks/munin/files/default/plugins/api_calls_num
cookbooks/munin/files/default/plugins/api_waits_
cookbooks/munin/files/default/plugins/passenger_memory
cookbooks/munin/files/default/plugins/passenger_processes
cookbooks/munin/files/default/plugins/passenger_queues
cookbooks/munin/libraries/expand.rb
cookbooks/munin/recipes/default.rb
cookbooks/mysql/libraries/mysql.rb
cookbooks/networking/libraries/interfaces.rb
cookbooks/networking/recipes/default.rb
cookbooks/nfs/recipes/server.rb
cookbooks/nginx/attributes/default.rb
cookbooks/nginx/recipes/default.rb
cookbooks/nominatim/recipes/default.rb
cookbooks/ntp/metadata.rb
cookbooks/ntp/recipes/default.rb
cookbooks/openssh/recipes/default.rb
cookbooks/openvpn/recipes/default.rb
cookbooks/piwik/attributes/default.rb
cookbooks/planet/files/default/replication-bin/replicate-changesets
cookbooks/planet/recipes/dump.rb
cookbooks/postgresql/definitions/postgresql_munin.rb
cookbooks/postgresql/libraries/postgresql.rb
cookbooks/rsyncd/recipes/default.rb
cookbooks/sysctl/metadata.rb
cookbooks/thinkup/recipes/default.rb
cookbooks/tile/files/default/ruby/expire.rb
cookbooks/tile/recipes/default.rb
cookbooks/tilecache/attributes/default.rb
cookbooks/tools/recipes/default.rb
cookbooks/trac/files/default/trac-authenticate
cookbooks/web/definitions/rails_port.rb
cookbooks/web/recipes/cgimap.rb
cookbooks/web/recipes/gpx.rb
cookbooks/web/recipes/rails.rb
cookbooks/wiki/attributes/default.rb
cookbooks/wiki/recipes/default.rb
cookbooks/wordpress/attributes/default.rb
cookbooks/wordpress/libraries/wordpress.rb
cookbooks/yournavigation/recipes/default.rb
roles/crm.rb
roles/fume.rb
roles/katie.rb
roles/konqi.rb
roles/nominatim.rb
roles/norbert.rb
roles/poldi.rb
roles/pummelzacken.rb
roles/smaug.rb
roles/tilecache.rb
roles/tyan-s7010.rb

index 545ef7c..bdb01eb 100644 (file)
@@ -71,7 +71,7 @@ search(:accounts, "*:*").each do |account|
         begin
           cookbook = run_context.cookbook_collection[cookbook_name]
           files = cookbook.relative_filenames_in_preferred_directory(node, :files, name.to_s)
-          not files.empty?
+          !files.empty?
         rescue Chef::Exceptions::FileNotFound
           false
         end
index 806a707..da339b9 100644 (file)
@@ -57,7 +57,7 @@ def create_conf
     group "root"
     mode 0644
     variables new_resource.variables
-    notifies :reload, "service[apache2]" if enabled? or available_name == enabled_name
+    notifies :reload, "service[apache2]" if enabled? || available_name == enabled_name
   end
 
   new_resource.updated_by_last_action(t.updated_by_last_action?)
index 166aa4a..d8072bc 100644 (file)
@@ -22,7 +22,7 @@ def whyrun_supported?
 end
 
 action :install do
-  if not installed?
+  if !installed?
     package package_name
 
     updated = true
@@ -47,7 +47,7 @@ action :install do
 end
 
 action :enable do
-  if not enabled?
+  unless enabled?
     link enabled_name("load") do
       to available_name("load")
       owner "root"
index be90671..30f7734 100644 (file)
@@ -25,7 +25,7 @@ if node[:lsb][:release].to_f < 14.04
     notifies :restart, "service[apache2]"
   end
 else
-  ["event", "itk", "prefork", "worker"].each do |mpm|
+  %w(event itk prefork worker).each do |mpm|
     if mpm == node[:apache][:mpm]
       apache_module "mpm_#{mpm}" do
         action [ :enable ]
index 2f5d60b..cf6ab1e 100644 (file)
@@ -1,11 +1,11 @@
 class Chef
   class Util
     def self.compare_versions(a, b)
-      if a.kind_of?(String)
+      if a.is_a?(String)
         a = a.split(".").map(&:to_i)
       end
 
-      if b.kind_of?(String)
+      if b.is_a?(String)
         b = b.split(".").map(&:to_i)
       end
 
index a5aa58c..775c2da 100644 (file)
@@ -6,7 +6,7 @@ class Chef
         command = git('ls-remote', @new_resource.repository, @new_resource.revision, "#{@new_resource.revision}^{}")
         @resolved_reference = shell_out!(command, run_options).stdout.split("\n").last
         if  @resolved_reference =~ /^([0-9a-f]{40})\s+(\S+)/
-          $1
+          Regexp.last_match[1]
         else
           nil
         end
index 50b207f..31b33c9 100644 (file)
@@ -23,14 +23,13 @@ class Chef
       def repo_attrs
         return {} unless ::File.exist?(::File.join(@new_resource.destination, ".svn"))
 
-        @repo_attrs ||= svn_info.lines.inject({}) do |attrs, line|
+        @repo_attrs ||= svn_info.lines.ech_with_object({}) do |attrs, line|
           if line =~ SVN_INFO_PATTERN
-            property, value = $1, $2
+            property, value = Regexp.last_match[1], Regexp.last_match[2]
             attrs[property] = value
           else
-            raise "Could not parse `svn info` data: #{line}"
+            fail "Could not parse `svn info` data: #{line}"
           end
-          attrs
         end
       end
 
index b440d85..600a21b 100644 (file)
@@ -22,9 +22,9 @@ chef_gem "pony"
 chef_package = "chef_#{node[:chef][:client][:version]}_amd64.deb"
 
 chef_platform = case node[:platform_version]
-                  when "12.10" then "12.04"
-                  when "14.04" then "13.04"
-                  else node[:platform_version]
+                when "12.10" then "12.04"
+                when "14.04" then "13.04"
+                else node[:platform_version]
                 end
 
 directory "/var/cache/chef" do
index 5cc28fa..18a0312 100644 (file)
@@ -20,9 +20,9 @@
 include_recipe "apache::ssl"
 
 chef_platform = case node[:platform_version]
-                  when "12.10" then "12.04"
-                  when "14.04" then "12.04"
-                  else node[:platform_version]
+                when "12.10" then "12.04"
+                when "14.04" then "12.04"
+                else node[:platform_version]
                 end
 
 chef_package = "chef-server_#{node[:chef][:server][:version]}_amd64.deb"
index 70ae0e6..4af5fdb 100644 (file)
@@ -80,7 +80,7 @@ end
 postgresql_extension "btree_gist" do
   cluster node[:db][:cluster]
   database "openstreetmap"
-  only_if { node[:postgresql][:clusters][node[:db][:cluster]] and node[:postgresql][:clusters][node[:db][:cluster]][:version] >= 9.0 }
+  only_if { node[:postgresql][:clusters][node[:db][:cluster]] && node[:postgresql][:clusters][node[:db][:cluster]][:version] >= 9.0 }
 end
 
 template "/etc/cron.daily/rails-db" do
index acea45e..a724d69 100644 (file)
@@ -105,7 +105,7 @@ search(:accounts, "*:*").each do |account|
   details = node[:accounts][:users][name] || {}
   port = 7000 + account["uid"].to_i
 
-  if ["user", "administrator"].include?(details[:status])
+  if %w(user administrator).include?(details[:status])
     user_home = details[:home] || account["home"] || "#{node[:accounts][:home]}/#{name}"
 
     if File.directory?("#{user_home}/public_html")
index 95e3957..3ea745d 100644 (file)
@@ -47,7 +47,7 @@ remote_directory "/srv/dns.openstreetmap.org/html" do
   files_mode 0644
 end
 
-zones = Array.new
+zones = []
 
 Dir.glob("/var/lib/dns/json/*.json").each do |kmlfile|
   zone = File.basename(kmlfile, ".json")
index 4956898..c334f9c 100644 (file)
@@ -39,7 +39,7 @@ template "/tmp/exim.ssl.cnf" do
   group "root"
   mode 0644
   not_if do
-    File.exist?("/etc/ssl/certs/exim.pem") and File.exist?("/etc/ssl/private/exim.key")
+    File.exist?("/etc/ssl/certs/exim.pem") && File.exist?("/etc/ssl/private/exim.key")
   end
 end
 
@@ -48,7 +48,7 @@ execute "/etc/ssl/certs/exim.pem" do
   user "root"
   group "ssl-cert"
   not_if do
-    File.exist?("/etc/ssl/certs/exim.pem") and File.exist?("/etc/ssl/private/exim.key")
+    File.exist?("/etc/ssl/certs/exim.pem") && File.exist?("/etc/ssl/private/exim.key")
   end
 end
 
@@ -86,7 +86,7 @@ search(:accounts, "*:*").each do |account|
   name = account["id"]
   details = node[:accounts][:users][name] || {}
 
-  if details[:status] and account["email"]
+  if details[:status] && account["email"]
     node.default[:exim][:aliases][name] = account["email"]
   end
 end
index 66efbb1..7361624 100644 (file)
@@ -61,7 +61,7 @@ Dir.new(git_directory).select { |name| name =~ /\.git$/ }.each do |repository|
     mode 0755
   end
 
-  if node[:recipes].include?("trace") and repository != "dns.git"
+  if node[:recipes].include?("trace") && repository != "dns.git"
     template "#{git_directory}/#{repository}/hooks/post-receive" do
       source "post-receive.erb"
       owner "root"
index 2a4e8e2..98c6f91 100644 (file)
@@ -1,14 +1,14 @@
-default[:hardware][:modules] = [ "loop", "lp", "rtc" ]
+default[:hardware][:modules] = %w(loop lp rtc)
 default[:hardware][:sensors] = {}
 
-if node[:dmi] and node[:dmi][:system]
+if node[:dmi] && node[:dmi][:system]
   case dmi.system.manufacturer
   when "HP"
     default[:apt][:sources] |= [ "management-component-pack" ]
 
     case dmi.system.product_name
-      when "ProLiant DL360 G6", "ProLiant DL360 G7"
-        default[:hardware][:sensors]["power_meter-*"][:power]["power1"] = { :ignore => true }
+    when "ProLiant DL360 G6", "ProLiant DL360 G7"
+      default[:hardware][:sensors]["power_meter-*"][:power]["power1"] = { :ignore => true }
     end
   end
 end
@@ -21,8 +21,8 @@ if Chef::Util.compare_versions(node[:kernel][:release], [3, 3]) < 0
   end
 end
 
-if node[:kernel] and node[:kernel][:modules]
-  raidmods = node[:kernel][:modules].keys & ["cciss", "hpsa", "mptsas", "mpt2sas", "megaraid_mm", "megaraid_sas", "aacraid"]
+if node[:kernel] && node[:kernel][:modules]
+  raidmods = node[:kernel][:modules].keys & %w(cciss hpsa mptsas mpt2sas megaraid_mm megaraid_sas aacraid)
 
   unless raidmods.empty?
     default[:apt][:sources] |= [ "hwraid" ]
index fec4b0c..4f0ec7d 100644 (file)
@@ -29,7 +29,7 @@ when "AuthenticAMD"
   end
 end
 
-if node[:dmi] and node[:dmi][:system]
+if node[:dmi] && node[:dmi][:system]
   case node[:dmi][:system][:manufacturer]
   when "empty"
     manufacturer = node[:dmi][:base_board][:manufacturer]
@@ -69,7 +69,7 @@ when "IBM"
   speed = "115200"
 end
 
-if manufacturer == "HP" and node[:lsb][:release].to_f > 11.10
+if manufacturer == "HP" && node[:lsb][:release].to_f > 11.10
   include_recipe "git"
 
   git "/opt/hp/hp-legacy" do
@@ -187,7 +187,7 @@ node[:kernel][:modules].each_key do |modname|
     status_packages["cciss-vol-status"] ||= []
   when "mptsas"
     tools_packages << "lsiutil"
-    #status_packages["mpt-status"] ||= []
+    # status_packages["mpt-status"] ||= []
   when "mpt2sas"
     tools_packages << "sas2ircu"
     status_packages["sas2ircu-status"] ||= []
@@ -204,9 +204,9 @@ node[:kernel][:modules].each_key do |modname|
 end
 
 node[:block_device].each do |name, attributes|
-  if attributes[:vendor] == "HP" and attributes[:model] == "LOGICAL VOLUME"
+  if attributes[:vendor] == "HP" && attributes[:model] == "LOGICAL VOLUME"
     if name =~ /^cciss!(c[0-9]+)d[0-9]+$/
-      status_packages["cciss-vol-status"] |= [ "cciss/#{$1}d0" ]
+      status_packages["cciss-vol-status"] |= [ "cciss/#{Regexp.last_match[1]}d0" ]
     else
       Dir.glob("/sys/block/#{name}/device/scsi_generic/*").each do |sg|
         status_packages["cciss-vol-status"] |= [ File.basename(sg) ]
@@ -215,7 +215,7 @@ node[:block_device].each do |name, attributes|
   end
 end
 
-["hpacucli", "lsiutil", "sas2ircu", "megactl", "megacli", "arcconf"].each do |tools_package|
+%w(hpacucli lsiutil sas2ircu megactl megacli arcconf).each do |tools_package|
   if tools_packages.include?(tools_package)
     package tools_package
   else
index f3622dc..1d507c6 100644 (file)
@@ -51,7 +51,7 @@ define :mediawiki_extension, :action => [ :enable ], :variables => {} do
       action :sync
       repository repository
       reference reference
-      #depth 1
+      # depth 1
       enable_submodules true
       user node[:mediawiki][:user]
       group node[:mediawiki][:group]
@@ -78,7 +78,7 @@ define :mediawiki_extension, :action => [ :enable ], :variables => {} do
     group node[:mediawiki][:group]
     mode 0664
     content "<?php require_once('#{extension_directory}/#{name}.php');\n"
-    only_if do File.exist?("#{extension_directory}/#{name}.php") end
+    only_if { File.exist?("#{extension_directory}/#{name}.php") }
     notifies :create, resources(:template => "#{mediawiki_directory}/LocalSettings.php")
   end
 end
index 5b08a34..94441b3 100644 (file)
@@ -20,7 +20,7 @@
 define :mediawiki_site, :action => [ :create, :enable ] do
   name = params[:name]
 
-  #/etc/cron.d names cannot contain a dot
+  # /etc/cron.d names cannot contain a dot
   cron_name = name.tr(".", "_")
 
   aliases = Array(params[:aliases])
@@ -60,7 +60,7 @@ define :mediawiki_site, :action => [ :create, :enable ] do
     :recaptcha_private => params[:recaptcha_private_key]
   }
 
-#----------------
+  #----------------
 
   node.set_unless[:mediawiki][:sites][name] = {}
   node.set[:mediawiki][:sites][name][:site_directory] = site_directory
@@ -68,7 +68,7 @@ define :mediawiki_site, :action => [ :create, :enable ] do
   node.set[:mediawiki][:sites][name][:version] = mediawiki_version
   node.set_unless[:mediawiki][:sites][name][:wgSecretKey] = random_password(64)
 
-#----------------
+  #----------------
 
   mysql_user "#{database_params[:username]}@localhost" do
     password database_params[:password]
@@ -87,7 +87,7 @@ define :mediawiki_site, :action => [ :create, :enable ] do
 
   execute "#{mediawiki[:directory]}/maintenance/install.php" do
     action :nothing
-    #Use metanamespace as Site Name to ensure correct set namespace
+    # Use metanamespace as Site Name to ensure correct set namespace
     command "php maintenance/install.php --server '#{name}' --dbtype 'mysql' --dbname '#{database_params[:name]}' --dbuser '#{database_params[:username]}' --dbpass '#{database_params[:password]}' --dbserver '#{database_params[:host]}' --scriptpath /w --pass '#{mediawiki[:site_admin_pw]}' '#{mediawiki[:metanamespace]}' '#{mediawiki[:site_admin_user]}'"
     cwd mediawiki[:directory]
     user node[:mediawiki][:user]
@@ -122,14 +122,14 @@ define :mediawiki_site, :action => [ :create, :enable ] do
     action :sync
     repository mediawiki_repository
     reference mediawiki_reference
-    #depth 1
+    # depth 1
     user node[:mediawiki][:user]
     group node[:mediawiki][:group]
     notifies :run, resources(:execute => "#{mediawiki[:directory]}/maintenance/install.php"), :immediately
     notifies :run, resources(:execute => "#{mediawiki[:directory]}/maintenance/update.php")
   end
 
-  #Safety catch if git doesn't update but install.php hasn't run
+  # Safety catch if git doesn't update but install.php hasn't run
   ruby_block "catch-installer-localsettings-run" do
     block do
       #
@@ -165,11 +165,7 @@ define :mediawiki_site, :action => [ :create, :enable ] do
     owner node[:mediawiki][:user]
     group node[:mediawiki][:group]
     mode 0664
-    variables({
-      :name => name,
-      :database_params => database_params,
-      :mediawiki => mediawiki
-    })
+    variables :name => name, :database_params => database_params, :mediawiki => mediawiki
     notifies :run, resources(:execute => "#{mediawiki[:directory]}/maintenance/update.php")
   end
 
@@ -179,11 +175,7 @@ define :mediawiki_site, :action => [ :create, :enable ] do
     owner "root"
     group "root"
     mode 0644
-    variables({
-      :name => name,
-      :directory => site_directory,
-      :user => node[:mediawiki][:user]
-    })
+    variables :name => name, :directory => site_directory, :user => node[:mediawiki][:user]
   end
 
   template "/etc/cron.daily/mediawiki-#{cron_name}-backup" do
@@ -192,14 +184,10 @@ define :mediawiki_site, :action => [ :create, :enable ] do
     owner "root"
     group "root"
     mode 0700
-    variables({
-      :name => name,
-      :directory => site_directory,
-      :database_params => database_params
-    })
+    variables :name => name, :directory => site_directory, :database_params => database_params
   end
 
-  #MediaWiki Default Extension
+  # MediaWiki Default Extension
 
   mediawiki_extension "Cite" do
     site name
@@ -276,7 +264,7 @@ define :mediawiki_site, :action => [ :create, :enable ] do
   end
 
   # MediaWiki Language Extension Bundle
-  #fixme should automatically resolve tag
+  # FIXME: should automatically resolve tag
   mw_lang_ext_bundle_tag = "2014.09"
 
   mediawiki_extension "Babel" do
@@ -302,25 +290,21 @@ define :mediawiki_site, :action => [ :create, :enable ] do
     tag mw_lang_ext_bundle_tag
   end
 
-  #LocalisationUpdate Update Cron
-  #template "/etc/cron.d/mediawiki-#{name}-LocalisationUpdate" do
-  #  cookbook "mediawiki"
-  #  source "mediawiki-LocalisationUpdate.cron.erb"
-  #  owner "root"
-  #  group "root"
-  #  mode 0755
-  #  variables({
-  #    :name => name,
-  #    :directory => site_directory,
-  #    :user => node[:mediawiki][:user]
-  #  })
-  #end
-
-  #mediawiki_extension "Translate" do
-  #  site name
-  #  template "mw-ext-Translate.inc.php.erb"
-  #  tag mw_lang_ext_bundle_tag
-  #end
+  # LocalisationUpdate Update Cron
+  # template "/etc/cron.d/mediawiki-#{name}-LocalisationUpdate" do
+  #   cookbook "mediawiki"
+  #   source "mediawiki-LocalisationUpdate.cron.erb"
+  #   owner "root"
+  #   group "root"
+  #   mode 0755
+  #   variables :name => name, :directory => site_directory, :user => node[:mediawiki][:user]
+  # end
+
+  # mediawiki_extension "Translate" do
+  #   site name
+  #   template "mw-ext-Translate.inc.php.erb"
+  #   tag mw_lang_ext_bundle_tag
+  # end
 
   mediawiki_extension "UniversalLanguageSelector" do
     site name
@@ -356,7 +340,7 @@ define :mediawiki_site, :action => [ :create, :enable ] do
     template "mw-ext-CirrusSearch.inc.php.erb"
   end
 
-  #OSM specifc extensions
+  # OSM specifc extensions
 
   mediawiki_extension "osmtaginfo" do
     site name
@@ -403,14 +387,11 @@ define :mediawiki_site, :action => [ :create, :enable ] do
     cookbook "mediawiki"
     template "apache.erb"
     directory site_directory
-    variables({
-      :aliases => aliases,
-      :mediawiki => mediawiki
-    })
+    variables :aliases => aliases, :mediawiki => mediawiki
     notifies :reload, "service[apache2]"
   end
 
-  #Fixme - Needs to run once
+  # FIXME: needs to run once
   execute "#{mediawiki[:directory]}/extensions/CirrusSearch/maintenance/updateSearchIndexConfig.php" do
     action :nothing
     command "php extensions/CirrusSearch/maintenance/updateSearchIndexConfig.php"
@@ -418,5 +399,4 @@ define :mediawiki_site, :action => [ :create, :enable ] do
     user node[:mediawiki][:user]
     group node[:mediawiki][:group]
   end
-
 end
index 5bc5b6a..267c4f4 100644 (file)
@@ -22,7 +22,7 @@ include_recipe "apache::ssl"
 include_recipe "mysql"
 include_recipe "git"
 
-#Mediawiki Base Requirements
+# Mediawiki Base Requirements
 package "php5"
 package "php5-cli"
 package "php5-curl"
@@ -34,15 +34,15 @@ package "libapache2-mod-php5"
 
 package "php-wikidiff2"
 
-#Mediawiki Image + SVG support
+# Mediawiki Image + SVG support
 package "imagemagick"
 package "librsvg2-bin"
 
-#Mediawiki PDF support via Extension:PdfHandler
+# Mediawiki PDF support via Extension:PdfHandler
 package "ghostscript"
 package "poppler-utils"
 
-#Mediawiki backup
+# Mediawiki backup
 package "xz-utils"
 
 link "/etc/php5/apache2/conf.d/20-wikidiff2.ini" do
index 73e2615..cc2fd8a 100755 (executable)
@@ -52,15 +52,14 @@ if ARGV[0] == 'config'
   CALL_TYPES.each { |k, v| puts "#{k}.label #{v}" }
 
 else
-  counts = uris_from_status(server).
-      collect { |x| categorise_uri(x) }.
-      inject(Hash.new) do |h, e|
-    if h.has_key? e
+  counts = uris_from_status(server)
+           .collect { |x| categorise_uri(x) }
+           .each_with_object(Hash.new) do |h, e|
+    if h.key? e
       h[e] += 1
     else
       h[e] = 1
     end
-    h
   end
 
   CALL_TYPES.keys.each do |type|
index c65ae0a..0e5034a 100755 (executable)
@@ -8,7 +8,7 @@ require 'apache_log_regex'
 NUM_LINES = 10000
 
 def uris_from_logs
-  lines = Array.new
+  lines = []
   max_time = nil
   min_time = nil
   parser = ApacheLogRegex.new('%h %l %u %t \"%r\" %>s %b \"%{Referer}i\" \"%{User-Agent}i\" %x')
@@ -20,7 +20,7 @@ def uris_from_logs
       min_time = [min_time, t].compact.min
       max_time = [max_time, t].compact.max
       lines << uri
-    rescue ApacheLogRegex::ParseError => e
+    rescue ApacheLogRegex::ParseError
       # nil
     end
   end
@@ -62,15 +62,14 @@ if ARGV[0] == 'config'
 else
   min_time, max_time, lines = uris_from_logs
   delta_t = (max_time - min_time).to_f * 24 * 60
-  counts = lines.
-      collect { |x| categorise_uri(x) }.
-      inject(Hash.new) do |h, e|
-    if h.has_key? e
+  counts = lines
+           .collect { |x| categorise_uri(x) }
+           .each_with_object(Hash.new) do |h, e|
+    if h.key? e
       h[e] += 1
     else
       h[e] = 1
     end
-    h
   end
 
   CALL_TYPES.keys.each do |type|
index f74e3a5..fcf956c 100755 (executable)
@@ -52,21 +52,20 @@ if ARGV[0] == 'config'
   CALL_TYPES.each { |k, v| puts "#{k}.label #{v}" }
 
 else
-  counts = uri_and_times_from_status(server).
-      collect { |x, y| [categorise_uri(x), y] }.
-      inject(Hash.new) do |h, e|
+  counts = uri_and_times_from_status(server)
+           .collect { |x, y| [categorise_uri(x), y] }
+           .each_with_object(Hash.new) do |h, e|
     category, time = e
-    if h.has_key? category
+    if h.key? category
       h[category] += [time]
     else
       h[category] = [time]
     end
-    h
   end
 
   CALL_TYPES.keys.each do |type|
     count = counts[type] || [0]
-    avg = count.inject(0) { |x, y|x + y } / (1.0 * count.length)
+    avg = count.inject(0) { |a, e| a + e } / (1.0 * count.length)
     puts "#{type}.value #{avg}"
   end
 end
index 2ce1a30..8c8a60b 100755 (executable)
@@ -21,7 +21,7 @@ def output_values
     exit 1
   end
   status =~ /(\d+\.\d+)/
-  puts "memory.value #{($1.to_f * 1024 * 1024).round}"
+  puts "memory.value #{(Regexp.last_match[1].to_f * 1024 * 1024).round}"
 end
 
 if ARGV[0] == "config"
index d89d041..75a3692 100755 (executable)
@@ -23,10 +23,10 @@ def output_values
     exit 1
   end
   status =~ /active\s+=\s+(\d+)/
-  puts "active.value #{$1}"
+  puts "active.value #{Regexp.last_match[1]}"
 
   status =~ /inactive\s+=\s+(\d+)/
-  puts "inactive.value #{$1}"
+  puts "inactive.value #{Regexp.last_match[1]}"
 end
 
 if ARGV[0] == "config"
index 9a67faf..506afb8 100755 (executable)
@@ -18,7 +18,7 @@ def output_values
     exit 1
   end
   status =~ /Waiting on global queue:\s+(\d+)/
-  puts "global.value #{$1}"
+  puts "global.value #{Regexp.last_match[1]}"
 end
 
 if ARGV[0] == "config"
index 081dfc5..dcd73cb 100644 (file)
@@ -2,8 +2,8 @@ class Chef
   class Munin
     def self.expand(template, nodes)
       nodes.map do |node|
-        if node.kind_of?(Hash)
-          template.gsub(/%%([^%]+)%%/) { node[$1.to_sym] }
+        if node.is_a?(Hash)
+          template.gsub(/%%([^%]+)%%/) { node[Regexp.last_match[1].to_sym] }
         else
           template.gsub("%%", node)
         end
index 302a601..59dc6e4 100644 (file)
@@ -173,7 +173,7 @@ munin_plugin "http_loadtime" do
 end
 
 node[:network][:interfaces].each do |ifname, ifattr|
-  if ifattr[:encapsulation] == "Ethernet" and ifattr[:state] == "up"
+  if ifattr[:encapsulation] == "Ethernet" && ifattr[:state] == "up"
     munin_plugin "if_err_#{ifname}" do
       target "if_err_"
     end
index bc5dd33..9ddc238 100644 (file)
@@ -87,33 +87,28 @@ class Chef
     end
 
     def users
-      @users ||= query("SELECT * FROM user").inject({}) do |users, user|
+      @users ||= query("SELECT * FROM user").each_with_object({}) do |users, user|
         name = "'#{user[:user]}'@'#{user[:host]}'"
 
-        users[name] = USER_PRIVILEGES.inject({}) do |privileges, privilege|
+        users[name] = USER_PRIVILEGES.each_with_object({}) do |privileges, privilege|
           privileges[privilege] = user["#{privilege}_priv".to_sym] == "Y"
-          privileges
         end
-
-        users
       end
     end
 
     def databases
-      @databases ||= query("SHOW databases").inject({}) do |databases, database|
+      @databases ||= query("SHOW databases").each_with_object({}) do |databases, database|
         databases[database[:database]] = {
           :permissions => {}
         }
-        databases
       end
 
       query("SELECT * FROM db").each do |record|
         if database = @databases[record[:db]]
           user = "'#{record[:user]}'@'#{record[:host]}'"
 
-          database[:permissions][user] = DATABASE_PRIVILEGES.inject([]) do |privileges, privilege|
+          database[:permissions][user] = DATABASE_PRIVILEGES.each_with_object([]) do |privileges, privilege|
             privileges << privilege if record["#{privilege}_priv".to_sym] == "Y"
-            privileges
           end
         end
       end
index 67b2bd2..fa0a93c 100644 (file)
@@ -7,8 +7,8 @@ class Chef
       networking_interfaces = networking[:interfaces] || []
 
       networking_interfaces.each_value  do |interface|
-        if options[:role].nil? or interface[:role].to_s == options[:role].to_s
-          if options[:family].nil? or interface[:family].to_s == options[:family].to_s
+        if options[:role].nil? || interface[:role].to_s == options[:role].to_s
+          if options[:family].nil? || interface[:family].to_s == options[:family].to_s
             if block.nil?
               interfaces << interface
             else
index 74fb004..ec6749f 100644 (file)
@@ -23,7 +23,7 @@
 require "ipaddr"
 
 node[:networking][:interfaces].each do |name, interface|
-  if interface[:role] and role = node[:networking][:roles][interface[:role]]
+  if interface[:role] && role = node[:networking][:roles][interface[:role]]
     if role[interface[:family]]
       node.default[:networking][:interfaces][name][:prefix] = role[interface[:family]][:prefix]
       node.default[:networking][:interfaces][name][:gateway] = role[interface[:family]][:gateway]
@@ -81,7 +81,7 @@ template "/etc/resolv.conf" do
 end
 
 node.interfaces(:role => :internal) do |interface|
-  if interface[:gateway] and interface[:gateway] != interface[:address]
+  if interface[:gateway] && interface[:gateway] != interface[:address]
     search(:node, "networking_interfaces*address:#{interface[:gateway]}") do |gateway|
       if gateway[:openvpn]
         gateway[:openvpn][:tunnels].each_value do |tunnel|
@@ -108,12 +108,12 @@ node.interfaces(:role => :internal) do |interface|
   end
 end
 
-zones = Hash.new
+zones = {}
 
 search(:node, "networking:interfaces").collect do |n|
   if n[:fqdn] != node[:fqdn]
     n.interfaces.each do |interface|
-      if interface[:role] == "external" and interface[:zone]
+      if interface[:role] == "external" && interface[:zone]
         zones[interface[:zone]] ||= Hash.new
         zones[interface[:zone]][interface[:family]] ||= Array.new
         zones[interface[:zone]][interface[:family]] << interface[:address]
@@ -199,7 +199,7 @@ firewall_rule "limit-icmp-echo" do
   rate_limit "s:1/sec:5"
 end
 
-[ "ucl", "ic", "bm" ].each do |zone|
+%w(ucl ic bm).each do |zone|
   firewall_rule "accept-openvpn-#{zone}" do
     action :accept
     family :inet
@@ -226,7 +226,7 @@ else
   end
 end
 
-if not node.interfaces(:family => :inet6).empty?
+unless node.interfaces(:family => :inet6).empty?
   package "shorewall6"
 
   service "shorewall6" do
index 8b04b84..1e4b00a 100644 (file)
@@ -25,10 +25,10 @@ service "rpcbind" do
   supports :status => true, :restart => true, :reload => true
 end
 
-#service "nfs-kernel-server" do
-#  action [ :enable, :start ]
-#  supports :status => true, :restart => true, :reload => true
-#end
+# service "nfs-kernel-server" do
+#   action [ :enable, :start ]
+#   supports :status => true, :restart => true, :reload => true
+# end
 
 exports = {}
 
index 7e054bd..f28efd2 100644 (file)
@@ -1 +1 @@
-#default[:nginx][:mpm] = "worker"
+# default[:nginx][:mpm] = "worker"
index 540e97c..b92d157 100644 (file)
@@ -19,7 +19,7 @@
 
 package "nginx"
 
-#admins = data_bag_item("nginx", "admins")
+# admins = data_bag_item("nginx", "admins")
 
 template "/etc/nginx/nginx.conf" do
   source "nginx.conf.erb"
index 2259c9d..ae1b794 100644 (file)
@@ -111,7 +111,6 @@ template "/etc/logrotate.d/nominatim" do
   mode 0644
 end
 
-
 package "osmosis"
 package "gcc"
 package "proj-bin"
@@ -161,7 +160,6 @@ directory "#{source_directory}/log" do
   mode 0755
 end
 
-
 template "#{source_directory}/.git/hooks/post-merge" do
   source "update_source.erb"
   owner  "nominatim"
@@ -215,7 +213,6 @@ template "/etc/cron.d/nominatim" do
   variables :bin_directory => "#{source_directory}/utils", :mailto => email_errors
 end
 
-
 template "#{source_directory}/utils/nominatim-update" do
   source "updater.erb"
   user   "nominatim"
index 6c5ce11..adcfc75 100644 (file)
@@ -8,7 +8,7 @@ version           "0.8.2"
 
 recipe "ntp", "Installs and configures ntp either as a server or client"
 
-%w{ ubuntu debian redhat centos fedora }.each do |os|
+%w(ubuntu debian redhat centos fedora).each do |os|
   supports os
 end
 
index 12447a2..077fd7a 100644 (file)
@@ -51,7 +51,7 @@ template "/etc/ntp.conf" do
   notifies :restart, "service[ntp]"
 end
 
-munin_plugins = [ "ntp_kernel_err", "ntp_kernel_pll_freq", "ntp_kernel_pll_off", "ntp_offset" ]
+munin_plugins = %w(ntp_kernel_err ntp_kernel_pll_freq ntp_kernel_pll_off ntp_offset)
 
 munin_plugin "ntp_kernel_err"
 munin_plugin "ntp_kernel_pll_freq"
@@ -60,7 +60,7 @@ munin_plugin "ntp_offset"
 
 if File.directory?("/etc/munin/plugins")
   Dir.new("/etc/munin/plugins").each do |plugin|
-    if plugin.match(/^ntp_/) and not munin_plugins.include?(plugin)
+    if plugin.match(/^ntp_/) && !munin_plugins.include?(plugin)
       munin_plugin plugin do
         action :delete
       end
index 0bae591..90ac311 100644 (file)
@@ -31,9 +31,7 @@ service "ssh" do
   supports :status => true, :restart => true, :reload => true
 end
 
-hosts = search(:node, "networking:interfaces").sort_by do |node|
-  node[:hostname]
-end.collect do |node|
+hosts = search(:node, "networking:interfaces").sort_by { |n| n[:hostname] }.collect do |node|
   names = [ node[:hostname] ]
 
   node.interfaces(:role => :external).each do |interface|
index b1955aa..3a0bccc 100644 (file)
@@ -27,7 +27,7 @@ end
 
 node[:openvpn][:tunnels].each do |name, details|
   if peer = search(:node, "fqdn:#{details[:peer][:host]}").first
-    if peer[:openvpn] and not details[:peer][:address]
+    if peer[:openvpn] && !details[:peer][:address]
       node.default[:openvpn][:tunnels][name][:peer][:address] = peer[:openvpn][:address]
     end
 
@@ -49,7 +49,7 @@ node[:openvpn][:tunnels].each do |name, details|
     if File.exist?("/etc/openvpn/#{name}.key")
       node.set[:openvpn][:keys][name] = IO.read("/etc/openvpn/#{name}.key")
     end
-  elsif peer and peer[:openvpn]
+  elsif peer && peer[:openvpn]
     file "/etc/openvpn/#{name}.key" do
       owner "root"
       group "root"
@@ -65,10 +65,10 @@ node[:openvpn][:tunnels].each do |name, details|
       group "root"
       mode 0644
       variables :name => name,
-      :address => node[:openvpn][:address],
-      :port => node[:openvpn][:tunnels][name][:port],
-      :mode => node[:openvpn][:tunnels][name][:mode],
-      :peer => node[:openvpn][:tunnels][name][:peer]
+                :address => node[:openvpn][:address],
+                :port => node[:openvpn][:tunnels][name][:port],
+                :mode => node[:openvpn][:tunnels][name][:mode],
+                :peer => node[:openvpn][:tunnels][name][:peer]
       notifies :restart, "service[openvpn]"
     end
   else
index b044d41..9e7365d 100644 (file)
@@ -1,13 +1,10 @@
 default[:piwik][:version] = "2.10.0"
-default[:piwik][:plugins] = [
-  "Actions", "API", "BulkTracking", "Contents", "CoreAdminHome",
-  "CoreConsole", "CoreHome", "CorePluginsAdmin", "CoreUpdater",
-  "CoreVisualizations", "CustomVariables", "Dashboard",
-  "DevicesDetection", "DevicePlugins", "DoNotTrack", "Feedback",
-  "Goals", "ImageGraph", "Installation", "LanguagesManager", "Live",
-  "Login", "Morpheus", "MultiSites", "PrivacyManager", "Provider",
-  "Proxy", "Referrers", "Resolution", "SEO", "SitesManager",
-  "UserCountry", "UserCountryMap", "UserSettings", "UsersManager",
-  "Widgetize", "VisitFrequency", "VisitorInterest", "VisitsSummary",
-  "VisitTime"
-]
+default[:piwik][:plugins] = %w(
+  Actions API BulkTracking Contents CoreAdminHome CoreConsole CoreHome
+  CorePluginsAdmin CoreUpdater CoreVisualizations CustomVariables
+  Dashboard DevicesDetection DevicePlugins DoNotTrack Feedback Goals
+  ImageGraph Installation LanguagesManager Live Login Morpheus MultiSites
+  PrivacyManager Provider Proxy Referrers Resolution SEO SitesManager
+  UserCountry UserCountryMap UserSettings UsersManager Widgetize
+  VisitFrequency VisitorInterest VisitsSummary VisitTime
+)
index daa7ab0..b25f97a 100644 (file)
@@ -33,7 +33,7 @@ class Changeset
   end
 
   def open?(t)
-    not closed?(t)
+    !closed?(t)
   end
 
   def activity_between?(t1, t2)
@@ -60,10 +60,10 @@ class Replicator
     # time (see rails_port's changeset model). so it is probably enough
     # for us to look at anything that was closed recently, and filter from
     # there.
-    @conn.
-      exec("select id, created_at, closed_at, num_changes from changesets where closed_at > ((now() at time zone 'utc') - '1 hour'::interval)").
-      map { |row| Changeset.new(row) }.
-      select { |cs| cs.activity_between?(last_run, @now) }
+    @conn
+      .exec("select id, created_at, closed_at, num_changes from changesets where closed_at > ((now() at time zone 'utc') - '1 hour'::interval)")
+      .map { |row| Changeset.new(row) }
+      .select { |cs| cs.activity_between?(last_run, @now) }
   end
 
   # creates an XML file containing the changeset information from the
@@ -75,8 +75,8 @@ class Replicator
       'generator' => 'replicate_changesets.rb',
       'copyright' => "OpenStreetMap and contributors",
       'attribution' => "http://www.openstreetmap.org/copyright",
-      'license' => "http://opendatacommons.org/licenses/odbl/1-0/" }.
-      each { |k, v| doc.root[k] = v }
+      'license' => "http://opendatacommons.org/licenses/odbl/1-0/" }
+      .each { |k, v| doc.root[k] = v }
 
     changesets.each do |cs|
       xml = XML::Node.new("changeset")
@@ -90,10 +90,10 @@ class Replicator
       xml['user'] = res[0]['display_name']
       xml['uid'] = res[0]['id']
 
-      unless (res[0]['min_lat'].nil? ||
-              res[0]['max_lat'].nil? ||
-              res[0]['min_lon'].nil? ||
-              res[0]['max_lon'].nil?)
+      unless res[0]['min_lat'].nil? ||
+             res[0]['max_lat'].nil? ||
+             res[0]['min_lon'].nil? ||
+             res[0]['max_lon'].nil?
         xml['min_lat'] = (res[0]['min_lat'].to_f / GEO_SCALE).to_s
         xml['max_lat'] = (res[0]['max_lat'].to_f / GEO_SCALE).to_s
         xml['min_lon'] = (res[0]['min_lon'].to_f / GEO_SCALE).to_s
@@ -119,7 +119,7 @@ class Replicator
     File.open(@config['state_file'], "r") do |fl|
       fl.flock(File::LOCK_EX)
 
-      sequence = (@state.has_key?('sequence') ? @state['sequence'] + 1 : 0)
+      sequence = (@state.key?('sequence') ? @state['sequence'] + 1 : 0)
       data_file = @config['data_dir'] + sprintf("/%03d/%03d/%03d.osm.gz", sequence / 1000000, (sequence / 1000) % 1000, (sequence % 1000))
       tmp_state = @config['state_file'] + ".tmp"
       tmp_data = "/tmp/changeset_data.osm.tmp"
index 6309256..edffb0e 100644 (file)
@@ -20,7 +20,7 @@
 node.default[:incron][:planetdump] = {
   :user => "www-data",
   :path => "/store/backup",
-  :events => [ "IN_CREATE", "IN_MOVED_TO" ],
+  :events => %w(IN_CREATE IN_MOVED_TO),
   :command => "/usr/local/bin/planetdump $#"
 }
 
index 8062044..3b60563 100644 (file)
@@ -22,7 +22,7 @@ define :postgresql_munin, :action => :create do
   suffix = cluster.tr("/", ":")
   database = params[:database]
 
-  if node[:postgresql][:clusters] and node[:postgresql][:clusters][cluster]
+  if node[:postgresql][:clusters] && node[:postgresql][:clusters][cluster]
     munin_plugin "postgres_cache_#{database}:#{suffix}" do
       action params[:action]
       target "postgres_cache_"
index 316f8ec..c121747 100644 (file)
@@ -63,58 +63,53 @@ class Chef
     end
 
     def users
-      @users ||= query("SELECT * FROM pg_user").inject({}) do |users, user|
+      @users ||= query("SELECT * FROM pg_user").each_with_oject({}) do |users, user|
         users[user[:usename]] = {
           :superuser => user[:usesuper] == "t",
           :createdb => user[:usercreatedb] == "t",
           :createrole => user[:usecatupd] == "t",
           :replication => user[:userepl] == "t"
         }
-        users
       end
     end
 
     def databases
-      @databases ||= query("SELECT d.datname, u.usename, d.encoding, d.datcollate, d.datctype FROM pg_database AS d INNER JOIN pg_user AS u ON d.datdba = u.usesysid").inject({}) do |databases, database|
+      @databases ||= query("SELECT d.datname, u.usename, d.encoding, d.datcollate, d.datctype FROM pg_database AS d INNER JOIN pg_user AS u ON d.datdba = u.usesysid").each_with_object({}) do |databases, database|
         databases[database[:datname]] = {
           :owner => database[:usename],
           :encoding => database[:encoding],
           :collate => database[:datcollate],
           :ctype => database[:datctype]
         }
-        databases
       end
     end
 
     def extensions(database)
       @extensions ||= {}
-      @extensions[database] ||= query("SELECT extname, extversion FROM pg_extension", :database => database).inject({}) do |extensions, extension|
+      @extensions[database] ||= query("SELECT extname, extversion FROM pg_extension", :database => database).each_with_object({}) do |extensions, extension|
         extensions[extension[:extname]] = {
           :version => extension[:extversion]
         }
-        databases
       end
     end
 
     def tables(database)
       @tables ||= {}
-      @tables[database] ||= query("SELECT n.nspname, c.relname, u.usename, c.relacl FROM pg_class AS c INNER JOIN pg_user AS u ON c.relowner = u.usesysid INNER JOIN pg_namespace AS n ON c.relnamespace = n.oid", :database => database).inject({}) do |tables, table|
+      @tables[database] ||= query("SELECT n.nspname, c.relname, u.usename, c.relacl FROM pg_class AS c INNER JOIN pg_user AS u ON c.relowner = u.usesysid INNER JOIN pg_namespace AS n ON c.relnamespace = n.oid", :database => database).each_with_object({}) do |tables, table|
         name = "#{table[:nspname]}.#{table[:relname]}"
 
         tables[name] = {
           :owner => table[:usename],
           :permissions => parse_acl(table[:relacl] || "{}")
         }
-
-        tables
       end
     end
 
   private
 
     def parse_acl(acl)
-      acl.sub(/^\{(.*)\}$/, "\\1").split(",").inject({}) do |permissions, entry|
-        entry = entry.sub(/^"(.*)"$/) { $1.gsub(/\\"/, '"') }.sub(/\/.*$/, "")
+      acl.sub(/^\{(.*)\}$/, "\\1").split(",").each_with_object({}) do |permissions, entry|
+        entry = entry.sub(/^"(.*)"$/) { Regexp.last_match[1].gsub(/\\"/, '"') }.sub(/\/.*$/, "")
         user, privileges = entry.split("=")
 
         user = user.sub(/^"(.*)"$/, "\\1")
@@ -124,8 +119,6 @@ class Chef
           "a" => :insert, "r" => :select, "w" => :update, "d" => :delete,
           "D" => :truncate, "x" => :references, "t" => :trigger
         }.values_at(*(privileges.chars)).compact
-
-        permissions
       end
     end
   end
index 41eaee8..fc0c4a5 100644 (file)
@@ -19,8 +19,8 @@
 
 include_recipe "networking"
 
-hosts_allow = Hash.new
-hosts_deny = Hash.new
+hosts_allow = {}
+hosts_deny = {}
 
 node[:rsyncd][:modules].each do |name, details|
   hosts_allow[name] = details[:hosts_allow] || []
index e5a5287..6d0db32 100644 (file)
@@ -5,7 +5,7 @@ license          "Apache 2.0"
 description      "Configures kernel parameters"
 long_description IO.read(File.join(File.dirname(__FILE__), 'README.md'))
 version          "0.1"
-%w{redhat centos debian ubuntu}.each do |os|
+%w(redhat centos debian ubuntu).each do |os|
   supports os
 end
 recipe           "sysctl", "Configure kernel parameters"
index c7b6ed2..c06feb7 100644 (file)
@@ -81,7 +81,7 @@ thinkup_config = edit_file "/srv/thinkup.openstreetmap.org/webapp/config.sample.
   line.gsub!(/^(\$THINKUP_CFG\['site_root_path'\] *=) '[^']*';$/, "\\1 '/';")
   line.gsub!(/^(\$THINKUP_CFG\['timezone'\] *=) '[^']*';$/, "\\1 'Europe/London';")
   line.gsub!(/^(\$THINKUP_CFG\['db_user'\] *=) '[^']*';$/, "\\1 'thinkup';")
-  line.gsub!(/^(\$THINKUP_CFG\['db_password'\] *=) '[^']*';$/, "\\1 '#{passwords["database"]}';")
+  line.gsub!(/^(\$THINKUP_CFG\['db_password'\] *=) '[^']*';$/, "\\1 '#{passwords['database']}';")
   line.gsub!(/^(\$THINKUP_CFG\['db_name'\] *=) '[^']*';$/, "\\1 'thinkup';")
 
   line
@@ -98,7 +98,7 @@ end
 thinkup_cron = edit_file "/srv/thinkup.openstreetmap.org/extras/cron/config.sample" do |line|
   line.gsub!(/^thinkup="[^"]*"$/, "thinkup=\"/srv/thinkup.openstreetmap.org\"")
   line.gsub!(/^thinkup_username="[^"]*"$/, "thinkup_username=\"openstreetmap@jonno.cix.co.uk\"")
-  line.gsub!(/^thinkup_password="[^"]*"$/, "thinkup_password=\"#{passwords["admin"]}\"")
+  line.gsub!(/^thinkup_password="[^"]*"$/, "thinkup_password=\"#{passwords['admin']}\"")
   line.gsub!(/^php="[^"]*"$/, "php=\"/usr/bin/php\"")
   line.gsub!(/^#crawl_interval=[0-9]+$/, "crawl_interval=30")
 
index 4c9d1a3..c691694 100755 (executable)
@@ -24,7 +24,7 @@ module Expire
   NODE_CACHE_FILE = "/store/database/nodes"
 
   # turns a spherical mercator coord into a tile coord
-  def Expire.tile_from_merc(point, zoom)
+  def self.tile_from_merc(point, zoom)
     # renormalise into unit space [0,1]
     point.x = 0.5 + point.x / SIZE
     point.y = 0.5 - point.y / SIZE
@@ -36,21 +36,21 @@ module Expire
   end
 
   # turns a latlon -> tile x,y given a zoom level
-  def Expire.tile_from_latlon(latlon, zoom)
+  def self.tile_from_latlon(latlon, zoom)
     # first convert to spherical mercator
     point = PROJ.forward(latlon)
     tile_from_merc(point, zoom)
   end
 
   # this must match the definition of xyz_to_meta in mod_tile
-  def Expire.xyz_to_meta(x, y, z)
+  def self.xyz_to_meta(x, y, z)
     # mask off the final few bits
     x &= ~(METATILE - 1)
     y &= ~(METATILE - 1)
     # generate the path
-    hash_path = (0..4).collect { |i|
+    hash_path = (0..4).collect do |i|
       (((x >> 4 * i) & 0xf) << 4) | ((y >> 4 * i) & 0xf)
-    }.reverse.join('/')
+    end.reverse.join('/')
     z.to_s + '/' + hash_path + ".meta"
   end
 
@@ -58,12 +58,12 @@ module Expire
   EXPIRY_TIME = Time.parse("2000-01-01 00:00:00")
 
   # expire the meta tile by setting the modified time back
-  def Expire.expire_meta(meta)
+  def self.expire_meta(meta)
     puts "Expiring #{meta}"
     File.utime(EXPIRY_TIME, EXPIRY_TIME, meta)
   end
 
-  def Expire.expire(change_file, min_zoom, max_zoom, tile_dirs)
+  def self.expire(change_file, min_zoom, max_zoom, tile_dirs)
     do_expire(change_file, min_zoom, max_zoom) do |set|
       new_set = Set.new
       meta_set = Set.new
@@ -93,12 +93,12 @@ module Expire
     end
   end
 
-  def Expire.do_expire(change_file, min_zoom, max_zoom, &_)
+  def self.do_expire(change_file, min_zoom, max_zoom, &_)
     # read in the osm change file
     doc = XML::Document.file(change_file)
 
     # hash map to contain all the nodes
-    nodes = Hash.new
+    nodes = {}
 
     # we put all the nodes into the hash, as it doesn't matter whether the node was
     # added, deleted or modified - the tile will need updating anyway.
index fc4feb0..675f8e7 100644 (file)
@@ -351,13 +351,7 @@ postgresql_extension "postgis" do
   database "gis"
 end
 
-[ "geography_columns",
-  "planet_osm_nodes",
-  "planet_osm_rels",
-  "planet_osm_ways",
-  "raster_columns",
-  "raster_overviews",
-  "spatial_ref_sys" ].each do |table|
+%w(geography_columns planet_osm_nodes planet_osm_rels planet_osm_ways raster_columns raster_overviews spatial_ref_sys).each do |table|
   postgresql_table table do
     cluster node[:tile][:database][:cluster]
     database "gis"
@@ -366,11 +360,7 @@ end
   end
 end
 
-[ "geometry_columns",
-  "planet_osm_line",
-  "planet_osm_point",
-  "planet_osm_polygon",
-  "planet_osm_roads" ].each do |table|
+%w(geometry_columns planet_osm_line planet_osm_point planet_osm_polygon planet_osm_roads).each do |table|
   postgresql_table table do
     cluster node[:tile][:database][:cluster]
     database "gis"
index a5a25db..6487cd9 100644 (file)
@@ -2,13 +2,13 @@
 default[:tilecache][:tile_parent] = "parent.tile.openstreetmap.org"
 default[:tilecache][:tile_siblings] = []
 
-#Per IP bucket refill rate
+# Per IP bucket refill rate
 default[:tilecache][:ip_bucket_refill] = "4096"
-#Per IP bucket size
+# Per IP bucket size
 default[:tilecache][:ip_bucket_size] = "67108864"
-#Per Class C refill rate
+# Per Class C refill rate
 default[:tilecache][:net_bucket_refill] = "8192"
-#Per Class C bucket size
+# Per Class C bucket size
 default[:tilecache][:net_bucket_size] = "134217728"
 
 default[:tilecache][:ssl][:certificate] = "tile.openstreetmap"
index a2245a4..8424642 100644 (file)
@@ -45,12 +45,12 @@ service "rsyslog" do
   supports :status => true, :restart => true, :reload => true
 end
 
-#Remove unused base package
+# Remove unused base package
 package "mlocate" do
   action :purge
 end
 
-#Remove ubuntu "desktop" vestigal package
+# Remove ubuntu "desktop" vestigal package
 package "whoopsie" do
   action :purge
 end
index cfd33e7..489a12c 100755 (executable)
@@ -11,5 +11,5 @@ request.basic_auth user, pass
 
 response = Net::HTTP.new("api.openstreetmap.org").request(request)
 
-exit!(0) if response.kind_of?(Net::HTTPSuccess)
+exit!(0) if response.is_a?(Net::HTTPSuccess)
 exit!(1)
index 6abec93..9fa88c5 100644 (file)
@@ -242,7 +242,7 @@ define :rails_port, :action => [ :create, :enable ] do
     cwd "#{rails_directory}/lib/quad_tile"
     user rails_user
     group rails_group
-    not_if { File.exist?("#{rails_directory}/lib/quad_tile/Makefile") and File.mtime("#{rails_directory}/lib/quad_tile/Makefile") >= File.mtime("#{rails_directory}/lib/quad_tile/extconf.rb") }
+    not_if { File.exist?("#{rails_directory}/lib/quad_tile/Makefile") && File.mtime("#{rails_directory}/lib/quad_tile/Makefile") >= File.mtime("#{rails_directory}/lib/quad_tile/extconf.rb") }
   end
 
   execute "#{rails_directory}/lib/quad_tile/Makefile" do
@@ -251,10 +251,10 @@ define :rails_port, :action => [ :create, :enable ] do
     user rails_user
     group rails_group
     not_if do
-      File.exist?("#{rails_directory}/lib/quad_tile/quad_tile_so.so") and
-      File.mtime("#{rails_directory}/lib/quad_tile/quad_tile_so.so") >= File.mtime("#{rails_directory}/lib/quad_tile/Makefile") and
-      File.mtime("#{rails_directory}/lib/quad_tile/quad_tile_so.so") >= File.mtime("#{rails_directory}/lib/quad_tile/quad_tile.c") and
-      File.mtime("#{rails_directory}/lib/quad_tile/quad_tile_so.so") >= File.mtime("#{rails_directory}/lib/quad_tile/quad_tile.h")
+      File.exist?("#{rails_directory}/lib/quad_tile/quad_tile_so.so") &&
+        File.mtime("#{rails_directory}/lib/quad_tile/quad_tile_so.so") >= File.mtime("#{rails_directory}/lib/quad_tile/Makefile") &&
+        File.mtime("#{rails_directory}/lib/quad_tile/quad_tile_so.so") >= File.mtime("#{rails_directory}/lib/quad_tile/quad_tile.c") &&
+        File.mtime("#{rails_directory}/lib/quad_tile/quad_tile_so.so") >= File.mtime("#{rails_directory}/lib/quad_tile/quad_tile.h")
     end
     notifies :touch, "file[#{rails_directory}/tmp/restart.txt]"
   end
index 4a8ab12..09a3f78 100644 (file)
@@ -108,7 +108,7 @@ file "/etc/init.d/cgimap" do
   content cgimap_init
 end
 
-if ["database_offline", "api_offline"].include?(node[:web][:status])
+if %w(database_offline api_offline).include?(node[:web][:status])
   service "cgimap" do
     action :stop
   end
index b2117dd..10463da 100644 (file)
@@ -67,7 +67,7 @@ template "/etc/init.d/gpx-import" do
             :database_password => db_passwords["gpximport"]
 end
 
-if ["database_offline", "database_readonly", "gpx_offline"].include?(node[:web][:status])
+if %w(database_offline database_readonly gpx_offline).include?(node[:web][:status])
   service "gpx-import" do
     action :stop
   end
index bc10fac..6616c49 100644 (file)
@@ -38,7 +38,7 @@ end
 rails_directory = "#{node[:web][:base_directory]}/rails"
 
 piwik_configuration = data_bag_item("web", "piwik").to_hash.reject do |k, _|
-  ["chef_type", "data_bag", "id"].include?(k)
+  %w(chef_type data_bag id).include?(k)
 end
 
 rails_port "www.openstreetmap.org" do
@@ -62,7 +62,7 @@ rails_port "www.openstreetmap.org" do
   gpx_dir "/store/rails/gpx"
   attachments_dir "/store/rails/attachments"
   log_path "#{node[:web][:log_directory]}/rails.log"
-  memcache_servers [ "rails1", "rails2", "rails3" ]
+  memcache_servers %w(rails1 rails2 rails3)
   potlatch2_key web_passwords["potlatch2_key"]
   id_key web_passwords["id_key"]
   oauth_key web_passwords["oauth_key"]
index 2d4ae94..1d73d9f 100644 (file)
@@ -1,3 +1,3 @@
 
-#Force apache to listen only on localhost
-#default[:apache][:listen_address] = "127.0.0.1"
+# Force apache to listen only on localhost
+# default[:apache][:listen_address] = "127.0.0.1"
index 29fa60d..85bb746 100644 (file)
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-#include_recipe "squid"
+# include_recipe "squid"
 
 include_recipe "mediawiki"
 
@@ -50,8 +50,7 @@ mediawiki_site  "wiki.openstreetmap.org" do
   recaptcha_public_key "6LdFIQATAAAAAMwtHeI8KDgPqvRbXeNYSq1gujKz"
   recaptcha_private_key passwords["recaptcha"]
 
-  #site_readonly "MAINTENANCE UPDATE: WIKI READ-ONLY. ETA: Tuesday 8:00am UTC/GMT."
-
+  # site_readonly "MAINTENANCE UPDATE: WIKI READ-ONLY. ETA: Tuesday 8:00am UTC/GMT."
 end
 
 cookbook_file "/srv/wiki.openstreetmap.org/osm_logo_wiki.png" do
@@ -82,9 +81,7 @@ end
 apache_site "dump.wiki.openstreetmap.org" do
   template  "apache_wiki_dump.erb"
   directory "/srv/dump.wiki.openstreetmap.org"
-  variables({
-    :aliases => "dump.wiki.osm.org"
-  })
+  variables :aliases => "dump.wiki.osm.org"
 end
 
 template "/etc/cron.d/wiki-osm-org-dump" do
index 615e089..09845b9 100644 (file)
@@ -5,7 +5,7 @@ default[:accounts][:users][:wordpress][:status] = :role
 default[:apache][:mpm] = "prefork"
 
 # Make sure httpclient and php_serialize are installed
-default[:chef][:gems] |= [ "httpclient", "php_serialize" ]
+default[:chef][:gems] |= %w(httpclient php_serialize)
 
 # Set wordpress defaults
 default[:wordpress][:user] = "wordpress"
index 8ebb146..f8046aa 100644 (file)
@@ -13,7 +13,7 @@ class Chef
 
     def self.current_plugin_version(name)
       if svn_cat("http://plugins.svn.wordpress.org/#{name}/trunk/readme.txt") =~ /Stable tag:\s*([^\s\r]*)[\s\r]*/
-        $1
+        Regexp.last_match[1]
       else
         "trunk"
       end
index ce2c81b..2accede 100644 (file)
@@ -24,10 +24,10 @@ package "php5-cli"
 
 package "php-apc"
 
-#Required for osmosis
+# Required for osmosis
 package "default-jre-headless"
 
-#Required for building gosmore
+# Required for building gosmore
 package "build-essential"
 package "libxml2-dev"
 package "libgtk2.0-dev"
index 032bc40..ee6a58f 100644 (file)
@@ -1,7 +1,6 @@
 name "crm"
 description "Role applied to CRM server"
 
-
 default_attributes(
   :exim => {
     :local_domains => [ "crm.osmfoundation.org" ],
index e4c7391..a6aa6f1 100644 (file)
@@ -31,8 +31,8 @@ default_attributes(
       :parameters => {
         "block/cciss\!c0d0/queue/nr_requests" => "512",
         "block/cciss\!c0d1/queue/nr_requests" => "512",
-       "block/cciss\!c0d0/queue/scheduler" => "noop",
-       "block/cciss\!c0d1/queue/scheduler" => "noop"
+        "block/cciss\!c0d0/queue/scheduler" => "noop",
+        "block/cciss\!c0d1/queue/scheduler" => "noop"
       }
     }
   },
index ff9c9ab..1cd4ece 100644 (file)
@@ -29,14 +29,14 @@ default_attributes(
   :tilecache => {
     :tile_parent => "germany.render.openstreetmap.org",
     :tile_siblings => [
-       "tabaluga.openstreetmap.org",
-       "konqi.openstreetmap.org",
-       "trogdor.openstreetmap.org",
-       "nepomuk.openstreetmap.org",
-       "ridgeback.openstreetmap.org",
-       "fume.openstreetmap.org",
-       "gorynych.openstreetmap.org",
-       "simurgh.openstreetmap.org"
+      "tabaluga.openstreetmap.org",
+      "konqi.openstreetmap.org",
+      "trogdor.openstreetmap.org",
+      "nepomuk.openstreetmap.org",
+      "ridgeback.openstreetmap.org",
+      "fume.openstreetmap.org",
+      "gorynych.openstreetmap.org",
+      "simurgh.openstreetmap.org"
     ]
   }
 )
index 3b3fe2d..4dd68f0 100644 (file)
@@ -29,14 +29,14 @@ default_attributes(
   :tilecache => {
     :tile_parent => "germany.render.openstreetmap.org",
     :tile_siblings => [
-       "tabaluga.openstreetmap.org",
-       "katie.openstreetmap.org",
-       "trogdor.openstreetmap.org",
-       "nepomuk.openstreetmap.org",
-       "ridgeback.openstreetmap.org",
-       "fume.openstreetmap.org",
-       "gorynych.openstreetmap.org",
-       "simurgh.openstreetmap.org"
+      "tabaluga.openstreetmap.org",
+      "katie.openstreetmap.org",
+      "trogdor.openstreetmap.org",
+      "nepomuk.openstreetmap.org",
+      "ridgeback.openstreetmap.org",
+      "fume.openstreetmap.org",
+      "gorynych.openstreetmap.org",
+      "simurgh.openstreetmap.org"
     ]
   }
 )
index 0aa56aa..dff1acc 100644 (file)
@@ -64,7 +64,7 @@ default_attributes(
     :network_conntrack_max => {
       :comment => "Increase max number of connections tracked",
       :parameters => {
-         "net.netfilter.nf_conntrack_max" => "131072"
+        "net.netfilter.nf_conntrack_max" => "131072"
       }
     }
   },
index 28fc3aa..a968122 100644 (file)
@@ -35,10 +35,10 @@ default_attributes(
       :parameters => {
         "block/cciss\!c0d0/queue/nr_requests" => "512",
         "block/cciss\!c0d1/queue/nr_requests" => "512",
-       "block/cciss\!c0d0/queue/scheduler" => "noop",
-       "block/cciss\!c0d1/queue/scheduler" => "noop",
-       "block/sda/queue/nr_requests" => "512",
-       "block/sda/queue/scheduler" => "deadline"
+        "block/cciss\!c0d0/queue/scheduler" => "noop",
+        "block/cciss\!c0d1/queue/scheduler" => "noop",
+        "block/sda/queue/nr_requests" => "512",
+        "block/sda/queue/scheduler" => "deadline"
       }
     }
   }
index ad8988a..127a526 100644 (file)
@@ -67,33 +67,33 @@ default_attributes(
     :enabled => false,
     :flatnode_file => "/ssd-old/nominatim/nodes.store",
     :database => {
-        :cluster => "9.3/main",
-        :dbname => "nominatim",
-        :postgis => "2.1"
+      :cluster => "9.3/main",
+      :dbname => "nominatim",
+      :postgis => "2.1"
     },
     :fpm_pools => {
-        :www => {
-            :port => "8000",
-            :pm => "dynamic",
-            :max_children => "60"
-        },
-        :bulk => {
-            :port => "8001",
-            :pm => "static",
-            :max_children => "10"
-        }
+      :www => {
+        :port => "8000",
+        :pm => "dynamic",
+        :max_children => "60"
+      },
+      :bulk => {
+        :port => "8001",
+        :pm => "static",
+        :max_children => "10"
+      }
     },
     :tablespaces => {
-        "Osm2pgsql_Data" => "aux",
-        "Osm2pgsql_Index" => "data",
-        "Place_Data" => "ssd2",
-        "Place_Index" => "ssd1",
-        "Address_Data" => "ssd2",
-        "Address_Index" => "ssd1",
-        "Search_Data" => "ssd1",
-        "Search_Index" => "ssd1",
-        "Aux_Data" => "aux",
-        "Aux_Index" => "aux"
+      "Osm2pgsql_Data" => "aux",
+      "Osm2pgsql_Index" => "data",
+      "Place_Data" => "ssd2",
+      "Place_Index" => "ssd1",
+      "Address_Data" => "ssd2",
+      "Address_Index" => "ssd1",
+      "Search_Data" => "ssd1",
+      "Search_Index" => "ssd1",
+      "Aux_Data" => "aux",
+      "Aux_Index" => "aux"
     }
   }
 )
index 0c03f3a..559cb61 100644 (file)
@@ -35,33 +35,33 @@ default_attributes(
     :enabled => true,
     :flatnode_file => "/ssd/nominatim/nodes.store",
     :database => {
-        :cluster => "9.3/main",
-        :dbname => "nominatim",
-        :postgis => "2.1"
+      :cluster => "9.3/main",
+      :dbname => "nominatim",
+      :postgis => "2.1"
     },
     :fpm_pools => {
-        :www => {
-            :port => "8000",
-            :pm => "dynamic",
-            :max_children => "70"
-        },
-        :bulk => {
-            :port => "8001",
-            :pm => "static",
-            :max_children => "10"
-        }
+      :www => {
+        :port => "8000",
+        :pm => "dynamic",
+        :max_children => "70"
+      },
+      :bulk => {
+        :port => "8001",
+        :pm => "static",
+        :max_children => "10"
+      }
     },
     :tablespaces => {
-        "Osm2pgsql_Data" => "ssd",
-        "Osm2pgsql_Index" => "ssd",
-        "Place_Data" => "ssd",
-        "Place_Index" => "ssd",
-        "Address_Data" => "ssd",
-        "Address_Index" => "ssd",
-        "Search_Data" => "ssd",
-        "Search_Index" => "ssd",
-        "Aux_Data" => "data",
-        "Aux_Index" => "ssd"
+      "Osm2pgsql_Data" => "ssd",
+      "Osm2pgsql_Index" => "ssd",
+      "Place_Data" => "ssd",
+      "Place_Index" => "ssd",
+      "Address_Data" => "ssd",
+      "Address_Index" => "ssd",
+      "Search_Data" => "ssd",
+      "Search_Index" => "ssd",
+      "Aux_Data" => "data",
+      "Aux_Index" => "ssd"
     }
   }
 )
index 9937714..2554b86 100644 (file)
@@ -64,8 +64,8 @@ default_attributes(
       :parameters => {
         "block/sda/queue/nr_requests" => "512",
         "block/sdb/queue/nr_requests" => "512",
-       "block/sda/queue/scheduler" => "noop",
-       "block/sdb/queue/scheduler" => "noop"
+        "block/sda/queue/scheduler" => "noop",
+        "block/sdb/queue/scheduler" => "noop"
       }
     }
   }
index 3a6d7fa..db78d4d 100644 (file)
@@ -15,7 +15,7 @@ default_attributes(
     :network_conntrack_max => {
       :comment => "Increase max number of connections tracked",
       :parameters => {
-         "net.netfilter.nf_conntrack_max" => "131072"
+        "net.netfilter.nf_conntrack_max" => "131072"
       }
     },
     :squid_swappiness => {
index 6cd45eb..30d13b7 100644 (file)
@@ -3,9 +3,7 @@ description "Role applied to machines using the Tyan S710 motherboard"
 
 default_attributes(
   :hardware => {
-    :modules => [
-      "i2c_i801", "jc42", "w83793"
-    ],
+    :modules => %w(i2c_i801 jc42 w83793),
     :sensors => {
       "jc42-*" => {
         :temps => {