Fix more rubocop detected style issues

diff --git a/cookbooks/accounts/recipes/default.rb b/cookbooks/accounts/recipes/default.rb
index 545ef7c9f5c81e08f4bca2676bb3f6b688703505..bdb01eb9e0e1b259515cb1fdbd226728cd314f56 100644 (file)
--- a/cookbooks/accounts/recipes/default.rb
+++ b/cookbooks/accounts/recipes/default.rb
@@ -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)
         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
         rescue Chef::Exceptions::FileNotFound
           false
         end

diff --git a/cookbooks/apache/providers/conf.rb b/cookbooks/apache/providers/conf.rb
index 806a7076adf37636d1080fc2a5159cb3631674f3..da339b955098f9d74b87e2e3c97c0e0322d10f17 100644 (file)
--- a/cookbooks/apache/providers/conf.rb
+++ b/cookbooks/apache/providers/conf.rb
@@ -57,7 +57,7 @@ def create_conf
     group "root"
     mode 0644
     variables new_resource.variables
     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?)
   end
 
   new_resource.updated_by_last_action(t.updated_by_last_action?)

diff --git a/cookbooks/apache/providers/module.rb b/cookbooks/apache/providers/module.rb
index 166aa4a4739adfced8ea204d0c4936ffc739d85c..d8072bc6ef84b1e28caaa75f1d80c7af389183aa 100644 (file)
--- a/cookbooks/apache/providers/module.rb
+++ b/cookbooks/apache/providers/module.rb
@@ -22,7 +22,7 @@ def whyrun_supported?
 end
 
 action :install do
 end
 
 action :install do

-  if not installed?
+  if !installed?

     package package_name
 
     updated = true
     package package_name
 
     updated = true


@@ -47,7 +47,7 @@ action :install do
 end
 
 action :enable do
 end
 
 action :enable do

-  if not enabled?
+  unless enabled?

     link enabled_name("load") do
       to available_name("load")
       owner "root"
     link enabled_name("load") do
       to available_name("load")
       owner "root"

diff --git a/cookbooks/apache/recipes/default.rb b/cookbooks/apache/recipes/default.rb
index be906716286830d34294cb3bb4db015f825651ee..30f77340dd4071aa9838685e9be9765de2f345c7 100644 (file)
--- a/cookbooks/apache/recipes/default.rb
+++ b/cookbooks/apache/recipes/default.rb
@@ -25,7 +25,7 @@ if node[:lsb][:release].to_f < 14.04
     notifies :restart, "service[apache2]"
   end
 else
     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 ]
     if mpm == node[:apache][:mpm]
       apache_module "mpm_#{mpm}" do
         action [ :enable ]

diff --git a/cookbooks/chef/libraries/compare_versions.rb b/cookbooks/chef/libraries/compare_versions.rb
index 2f5d60baff82b5f435397c50b1efdce8065c4060..cf6ab1ea23c84a3d0b5764b7d311d32889719efd 100644 (file)
--- a/cookbooks/chef/libraries/compare_versions.rb
+++ b/cookbooks/chef/libraries/compare_versions.rb
@@ -1,11 +1,11 @@
 class Chef
   class Util
     def self.compare_versions(a, b)
 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
 
         a = a.split(".").map(&:to_i)
       end
 

-      if b.kind_of?(String)
+      if b.is_a?(String)

         b = b.split(".").map(&:to_i)
       end
 
         b = b.split(".").map(&:to_i)
       end
 

diff --git a/cookbooks/chef/libraries/git.rb b/cookbooks/chef/libraries/git.rb
index a5aa58c96a5ecac93856818341e38759a1713d1e..775c2da1ab080976597e78a2d1e2561692e92cd7 100644 (file)
--- a/cookbooks/chef/libraries/git.rb
+++ b/cookbooks/chef/libraries/git.rb
@@ -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+)/
         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
         else
           nil
         end

diff --git a/cookbooks/chef/libraries/subversion.rb b/cookbooks/chef/libraries/subversion.rb
index 50b207ff8286e93e3e57bd3d769c7a94910ae787..31b33c92daca6449a994de0aa76ebde7b24ebea8 100644 (file)
--- a/cookbooks/chef/libraries/subversion.rb
+++ b/cookbooks/chef/libraries/subversion.rb
@@ -23,14 +23,13 @@ class Chef
       def repo_attrs
         return {} unless ::File.exist?(::File.join(@new_resource.destination, ".svn"))
 
       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
           if line =~ SVN_INFO_PATTERN

-            property, value = $1, $2
+            property, value = Regexp.last_match[1], Regexp.last_match[2]

             attrs[property] = value
           else
             attrs[property] = value
           else

-            raise "Could not parse `svn info` data: #{line}"
+            fail "Could not parse `svn info` data: #{line}"

           end
           end

-          attrs

         end
       end
 
         end
       end
 

diff --git a/cookbooks/chef/recipes/default.rb b/cookbooks/chef/recipes/default.rb
index b440d85e927645223aa5a6a71511067e1020f8f5..600a21b49a05ab4d5fb62d16a17285ade1a05fc6 100644 (file)
--- a/cookbooks/chef/recipes/default.rb
+++ b/cookbooks/chef/recipes/default.rb
@@ -22,9 +22,9 @@ chef_gem "pony"
 chef_package = "chef_#{node[:chef][:client][:version]}_amd64.deb"
 
 chef_platform = case node[:platform_version]
 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
                 end
 
 directory "/var/cache/chef" do

diff --git a/cookbooks/chef/recipes/server.rb b/cookbooks/chef/recipes/server.rb
index 5cc28fa69f09c2b6a8b8fc1d7dc33c55d728bd4d..18a0312e224a5e2a5b11dbfa66dad1f780021b4e 100644 (file)
--- a/cookbooks/chef/recipes/server.rb
+++ b/cookbooks/chef/recipes/server.rb
@@ -20,9 +20,9 @@
 include_recipe "apache::ssl"
 
 chef_platform = case node[:platform_version]
 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"
                 end
 
 chef_package = "chef-server_#{node[:chef][:server][:version]}_amd64.deb"

diff --git a/cookbooks/db/recipes/master.rb b/cookbooks/db/recipes/master.rb
index 70ae0e6736a8c7389b809b4517cffad26f638e48..4af5fdb5cd95dfdaf184a1d180c6b0a987c31624 100644 (file)
--- a/cookbooks/db/recipes/master.rb
+++ b/cookbooks/db/recipes/master.rb
@@ -80,7 +80,7 @@ end
 postgresql_extension "btree_gist" do
   cluster node[:db][:cluster]
   database "openstreetmap"
 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
 end
 
 template "/etc/cron.daily/rails-db" do

diff --git a/cookbooks/dev/recipes/default.rb b/cookbooks/dev/recipes/default.rb
index acea45e176c727c74f35da82789d94abcf820ef3..a724d69c53cd3f0a1eecb54c9d982adaf0c2fc53 100644 (file)
--- a/cookbooks/dev/recipes/default.rb
+++ b/cookbooks/dev/recipes/default.rb
@@ -105,7 +105,7 @@ search(:accounts, "*:*").each do |account|
   details = node[:accounts][:users][name] || {}
   port = 7000 + account["uid"].to_i
 
   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")
     user_home = details[:home] || account["home"] || "#{node[:accounts][:home]}/#{name}"
 
     if File.directory?("#{user_home}/public_html")

diff --git a/cookbooks/dns/recipes/default.rb b/cookbooks/dns/recipes/default.rb
index 95e3957faa00b96d783aac529b4ff3cc141253e4..3ea745d3c71aacfb66c9b5111d40ce9f748fc64d 100644 (file)
--- a/cookbooks/dns/recipes/default.rb
+++ b/cookbooks/dns/recipes/default.rb
@@ -47,7 +47,7 @@ remote_directory "/srv/dns.openstreetmap.org/html" do
   files_mode 0644
 end
 
   files_mode 0644
 end
 

-zones = Array.new
+zones = []

 
 Dir.glob("/var/lib/dns/json/*.json").each do |kmlfile|
   zone = File.basename(kmlfile, ".json")
 
 Dir.glob("/var/lib/dns/json/*.json").each do |kmlfile|
   zone = File.basename(kmlfile, ".json")

diff --git a/cookbooks/exim/recipes/default.rb b/cookbooks/exim/recipes/default.rb
index 49568984183f31661b27a0e66c47190a367960b8..c334f9cedfbfc43726c46d4d76284e4439214050 100644 (file)
--- a/cookbooks/exim/recipes/default.rb
+++ b/cookbooks/exim/recipes/default.rb
@@ -39,7 +39,7 @@ template "/tmp/exim.ssl.cnf" do
   group "root"
   mode 0644
   not_if 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
 
   end
 end
 


@@ -48,7 +48,7 @@ execute "/etc/ssl/certs/exim.pem" do
   user "root"
   group "ssl-cert"
   not_if 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
 
   end
 end
 


@@ -86,7 +86,7 @@ search(:accounts, "*:*").each do |account|
   name = account["id"]
   details = node[:accounts][:users][name] || {}
 
   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
     node.default[:exim][:aliases][name] = account["email"]
   end
 end

diff --git a/cookbooks/git/recipes/server.rb b/cookbooks/git/recipes/server.rb
index 66efbb1465627b6cf1dfd3fcb2dfcec16c46ece2..7361624c4d4a75ca374b0f1a952c8e64de18da47 100644 (file)
--- a/cookbooks/git/recipes/server.rb
+++ b/cookbooks/git/recipes/server.rb
@@ -61,7 +61,7 @@ Dir.new(git_directory).select { |name| name =~ /\.git$/ }.each do |repository|
     mode 0755
   end
 
     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"
     template "#{git_directory}/#{repository}/hooks/post-receive" do
       source "post-receive.erb"
       owner "root"

diff --git a/cookbooks/hardware/attributes/default.rb b/cookbooks/hardware/attributes/default.rb
index 2a4e8e2724eef198c7d9b45df0c651b372ea72ae..98c6f91862db4007b1586f15078f5b707abffee1 100644 (file)
--- a/cookbooks/hardware/attributes/default.rb
+++ b/cookbooks/hardware/attributes/default.rb
@@ -1,14 +1,14 @@
-default[:hardware][:modules] = [ "loop", "lp", "rtc" ]
+default[:hardware][:modules] = %w(loop lp rtc)

 default[:hardware][:sensors] = {}
 
 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
   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
     end
   end
 end


@@ -21,8 +21,8 @@ if Chef::Util.compare_versions(node[:kernel][:release], [3, 3]) < 0
   end
 end
 
   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" ]
 
   unless raidmods.empty?
     default[:apt][:sources] |= [ "hwraid" ]

diff --git a/cookbooks/hardware/recipes/default.rb b/cookbooks/hardware/recipes/default.rb
index fec4b0c98a3d4c397263120fc6476e9fce8d9416..4f0ec7df629e9b5458f35d3569b15a3f63a13980 100644 (file)
--- a/cookbooks/hardware/recipes/default.rb
+++ b/cookbooks/hardware/recipes/default.rb
@@ -29,7 +29,7 @@ when "AuthenticAMD"
   end
 end
 
   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]
   case node[:dmi][:system][:manufacturer]
   when "empty"
     manufacturer = node[:dmi][:base_board][:manufacturer]


@@ -69,7 +69,7 @@ when "IBM"
   speed = "115200"
 end
 
   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
   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["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"] ||= []
   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|
 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]+$/
     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) ]
     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
 
   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
   if tools_packages.include?(tools_package)
     package tools_package
   else

diff --git a/cookbooks/mediawiki/definitions/mediawiki_extension.rb b/cookbooks/mediawiki/definitions/mediawiki_extension.rb
index f3622dc3759d4e9e6a95c0a08514346df9db879d..1d507c69dcd241d7b68f01d86abc2287c7601ef7 100644 (file)
--- a/cookbooks/mediawiki/definitions/mediawiki_extension.rb
+++ b/cookbooks/mediawiki/definitions/mediawiki_extension.rb
@@ -51,7 +51,7 @@ define :mediawiki_extension, :action => [ :enable ], :variables => {} do
       action :sync
       repository repository
       reference reference
       action :sync
       repository repository
       reference reference

-      #depth 1
+      # depth 1

       enable_submodules true
       user node[:mediawiki][:user]
       group node[:mediawiki][:group]
       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"
     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
     notifies :create, resources(:template => "#{mediawiki_directory}/LocalSettings.php")
   end
 end

diff --git a/cookbooks/mediawiki/definitions/mediawiki_site.rb b/cookbooks/mediawiki/definitions/mediawiki_site.rb
index 5b08a34921838b3ef9ab7814ab1d66ded995c129..94441b30af88bc3dafc879a01eb9171b668e760f 100644 (file)
--- a/cookbooks/mediawiki/definitions/mediawiki_site.rb
+++ b/cookbooks/mediawiki/definitions/mediawiki_site.rb
@@ -20,7 +20,7 @@
 define :mediawiki_site, :action => [ :create, :enable ] do
   name = params[:name]
 
 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])
   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]
   }
 
     :recaptcha_private => params[:recaptcha_private_key]
   }
 

-#----------------
+  #----------------

 
   node.set_unless[:mediawiki][:sites][name] = {}
   node.set[:mediawiki][:sites][name][:site_directory] = site_directory
 
   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)
 
   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]
 
   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
 
   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]
     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
     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
 
     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
       #
   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
     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
 
     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
     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
   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
     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
 
   end
 

-  #MediaWiki Default Extension
+  # MediaWiki Default Extension

 
   mediawiki_extension "Cite" do
     site name
 
   mediawiki_extension "Cite" do
     site name


@@ -276,7 +264,7 @@ define :mediawiki_site, :action => [ :create, :enable ] do
   end
 
   # MediaWiki Language Extension Bundle
   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
   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
 
     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
 
   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
 
     template "mw-ext-CirrusSearch.inc.php.erb"
   end
 

-  #OSM specifc extensions
+  # OSM specifc extensions

 
   mediawiki_extension "osmtaginfo" do
     site name
 
   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
     cookbook "mediawiki"
     template "apache.erb"
     directory site_directory

-    variables({
-      :aliases => aliases,
-      :mediawiki => mediawiki
-    })
+    variables :aliases => aliases, :mediawiki => mediawiki

     notifies :reload, "service[apache2]"
   end
 
     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"
   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
     user node[:mediawiki][:user]
     group node[:mediawiki][:group]
   end

-

 end
 end

diff --git a/cookbooks/mediawiki/recipes/default.rb b/cookbooks/mediawiki/recipes/default.rb
index 5bc5b6ac169a9165666cc634503a5aa0f37d525f..267c4f4b152b73382918f439ae065b5ca6a685cb 100644 (file)
--- a/cookbooks/mediawiki/recipes/default.rb
+++ b/cookbooks/mediawiki/recipes/default.rb
@@ -22,7 +22,7 @@ include_recipe "apache::ssl"
 include_recipe "mysql"
 include_recipe "git"
 
 include_recipe "mysql"
 include_recipe "git"
 

-#Mediawiki Base Requirements
+# Mediawiki Base Requirements

 package "php5"
 package "php5-cli"
 package "php5-curl"
 package "php5"
 package "php5-cli"
 package "php5-curl"


@@ -34,15 +34,15 @@ package "libapache2-mod-php5"
 
 package "php-wikidiff2"
 
 
 package "php-wikidiff2"
 

-#Mediawiki Image + SVG support
+# Mediawiki Image + SVG support

 package "imagemagick"
 package "librsvg2-bin"
 
 package "imagemagick"
 package "librsvg2-bin"
 

-#Mediawiki PDF support via Extension:PdfHandler
+# Mediawiki PDF support via Extension:PdfHandler

 package "ghostscript"
 package "poppler-utils"
 
 package "ghostscript"
 package "poppler-utils"
 

-#Mediawiki backup
+# Mediawiki backup

 package "xz-utils"
 
 link "/etc/php5/apache2/conf.d/20-wikidiff2.ini" do
 package "xz-utils"
 
 link "/etc/php5/apache2/conf.d/20-wikidiff2.ini" do

diff --git a/cookbooks/munin/files/default/plugins/api_calls_ b/cookbooks/munin/files/default/plugins/api_calls_
index 73e2615d0bf356a5f8a1f5617a589a6bc5801b61..cc2fd8a9da23b25422587e4460fe6359fbb7100f 100755 (executable)
--- a/cookbooks/munin/files/default/plugins/api_calls_
+++ b/cookbooks/munin/files/default/plugins/api_calls_
@@ -52,15 +52,14 @@ if ARGV[0] == 'config'
   CALL_TYPES.each { |k, v| puts "#{k}.label #{v}" }
 
 else
   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[e] += 1
     else
       h[e] = 1
     end

-    h

   end
 
   CALL_TYPES.keys.each do |type|
   end
 
   CALL_TYPES.keys.each do |type|

diff --git a/cookbooks/munin/files/default/plugins/api_calls_num b/cookbooks/munin/files/default/plugins/api_calls_num
index c65ae0a7ad48c207e06df7f00046511b4a7699c6..0e5034a6316778e18f4bbb4c794e2b7094cf2d12 100755 (executable)
--- a/cookbooks/munin/files/default/plugins/api_calls_num
+++ b/cookbooks/munin/files/default/plugins/api_calls_num
@@ -8,7 +8,7 @@ require 'apache_log_regex'
 NUM_LINES = 10000
 
 def uris_from_logs
 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')
   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
       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
       # 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
 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[e] += 1
     else
       h[e] = 1
     end

-    h

   end
 
   CALL_TYPES.keys.each do |type|
   end
 
   CALL_TYPES.keys.each do |type|

diff --git a/cookbooks/munin/files/default/plugins/api_waits_ b/cookbooks/munin/files/default/plugins/api_waits_
index f74e3a5cdf2c7119ac02fbd562213c7fbf73a3d4..fcf956ccb5b25e53e334b9b7e157a9c1ff5ae9d9 100755 (executable)
--- a/cookbooks/munin/files/default/plugins/api_waits_
+++ b/cookbooks/munin/files/default/plugins/api_waits_
@@ -52,21 +52,20 @@ if ARGV[0] == 'config'
   CALL_TYPES.each { |k, v| puts "#{k}.label #{v}" }
 
 else
   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
     category, time = e

-    if h.has_key? category
+    if h.key? category

       h[category] += [time]
     else
       h[category] = [time]
     end
       h[category] += [time]
     else
       h[category] = [time]
     end

-    h

   end
 
   CALL_TYPES.keys.each do |type|
     count = counts[type] || [0]
   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
     puts "#{type}.value #{avg}"
   end
 end

diff --git a/cookbooks/munin/files/default/plugins/passenger_memory b/cookbooks/munin/files/default/plugins/passenger_memory
index 2ce1a305f3a4efc33b88b27c1a9fc70e52d62e04..8c8a60bb22f4e7ff9dcc531af29e8be91b8f8197 100755 (executable)
--- a/cookbooks/munin/files/default/plugins/passenger_memory
+++ b/cookbooks/munin/files/default/plugins/passenger_memory
@@ -21,7 +21,7 @@ def output_values
     exit 1
   end
   status =~ /(\d+\.\d+)/
     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"
 end
 
 if ARGV[0] == "config"

diff --git a/cookbooks/munin/files/default/plugins/passenger_processes b/cookbooks/munin/files/default/plugins/passenger_processes
index d89d0412e9139d2fac1a0ec39f908ef4b6ffa413..75a36923393255f0e482a950997a2f8784291436 100755 (executable)
--- a/cookbooks/munin/files/default/plugins/passenger_processes
+++ b/cookbooks/munin/files/default/plugins/passenger_processes
@@ -23,10 +23,10 @@ def output_values
     exit 1
   end
   status =~ /active\s+=\s+(\d+)/
     exit 1
   end
   status =~ /active\s+=\s+(\d+)/

-  puts "active.value #{$1}"
+  puts "active.value #{Regexp.last_match[1]}"

 
   status =~ /inactive\s+=\s+(\d+)/
 
   status =~ /inactive\s+=\s+(\d+)/

-  puts "inactive.value #{$1}"
+  puts "inactive.value #{Regexp.last_match[1]}"

 end
 
 if ARGV[0] == "config"
 end
 
 if ARGV[0] == "config"

diff --git a/cookbooks/munin/files/default/plugins/passenger_queues b/cookbooks/munin/files/default/plugins/passenger_queues
index 9a67fafc7a528076c48ef9bdbc0a3f85efded50b..506afb8510ed8645a73f93ccad36856d23f37d65 100755 (executable)
--- a/cookbooks/munin/files/default/plugins/passenger_queues
+++ b/cookbooks/munin/files/default/plugins/passenger_queues
@@ -18,7 +18,7 @@ def output_values
     exit 1
   end
   status =~ /Waiting on global queue:\s+(\d+)/
     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"
 end
 
 if ARGV[0] == "config"

diff --git a/cookbooks/munin/libraries/expand.rb b/cookbooks/munin/libraries/expand.rb
index 081dfc5c12299a2bd8ee7544b97e765059363794..dcd73cb69d77afb8b5dbbab1cf152a90992952b9 100644 (file)
--- a/cookbooks/munin/libraries/expand.rb
+++ b/cookbooks/munin/libraries/expand.rb
@@ -2,8 +2,8 @@ class Chef
   class Munin
     def self.expand(template, nodes)
       nodes.map do |node|
   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
         else
           template.gsub("%%", node)
         end

diff --git a/cookbooks/munin/recipes/default.rb b/cookbooks/munin/recipes/default.rb
index 302a601f79fd0618367f5426c53f11a1d1c5b6fa..59dc6e45639973d04d0e8278f02a9ec6aee04a7c 100644 (file)
--- a/cookbooks/munin/recipes/default.rb
+++ b/cookbooks/munin/recipes/default.rb
@@ -173,7 +173,7 @@ munin_plugin "http_loadtime" do
 end
 
 node[:network][:interfaces].each do |ifname, ifattr|
 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
     munin_plugin "if_err_#{ifname}" do
       target "if_err_"
     end

diff --git a/cookbooks/mysql/libraries/mysql.rb b/cookbooks/mysql/libraries/mysql.rb
index bc5dd33509491e32933baa5d8846abb257fdafd4..9ddc238c7525c1b21fc506d8bef14bdcf6e8d544 100644 (file)
--- a/cookbooks/mysql/libraries/mysql.rb
+++ b/cookbooks/mysql/libraries/mysql.rb
@@ -87,33 +87,28 @@ class Chef
     end
 
     def users
     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]}'"
 
         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[privilege] = user["#{privilege}_priv".to_sym] == "Y"

-          privileges

         end
         end

-
-        users

       end
     end
 
     def databases
       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[database[:database]] = {
           :permissions => {}
         }

-        databases

       end
 
       query("SELECT * FROM db").each do |record|
         if database = @databases[record[:db]]
           user = "'#{record[:user]}'@'#{record[:host]}'"
 
       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 << privilege if record["#{privilege}_priv".to_sym] == "Y"

-            privileges

           end
         end
       end
           end
         end
       end

diff --git a/cookbooks/networking/libraries/interfaces.rb b/cookbooks/networking/libraries/interfaces.rb
index 67b2bd2373176e2fd69ab93697170d2de6025f77..fa0a93c7d7347ed63f53b1adb08bf5aa849134ee 100644 (file)
--- a/cookbooks/networking/libraries/interfaces.rb
+++ b/cookbooks/networking/libraries/interfaces.rb
@@ -7,8 +7,8 @@ class Chef
       networking_interfaces = networking[:interfaces] || []
 
       networking_interfaces.each_value  do |interface|
       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
             if block.nil?
               interfaces << interface
             else

diff --git a/cookbooks/networking/recipes/default.rb b/cookbooks/networking/recipes/default.rb
index 74fb0048365dda9a68e1ef398e9f4017264c521a..ec6749fe7fbe334cde93869872df016a8a0cb282 100644 (file)
--- a/cookbooks/networking/recipes/default.rb
+++ b/cookbooks/networking/recipes/default.rb
@@ -23,7 +23,7 @@
 require "ipaddr"
 
 node[:networking][:interfaces].each do |name, interface|
 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]
     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|
 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|
     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
 
   end
 end
 

-zones = Hash.new
+zones = {}

 
 search(:node, "networking:interfaces").collect do |n|
   if n[:fqdn] != node[:fqdn]
     n.interfaces.each do |interface|
 
 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]
         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
 
   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
   firewall_rule "accept-openvpn-#{zone}" do
     action :accept
     family :inet


@@ -226,7 +226,7 @@ else
   end
 end
 
   end
 end
 

-if not node.interfaces(:family => :inet6).empty?
+unless node.interfaces(:family => :inet6).empty?

   package "shorewall6"
 
   service "shorewall6" do
   package "shorewall6"
 
   service "shorewall6" do

diff --git a/cookbooks/nfs/recipes/server.rb b/cookbooks/nfs/recipes/server.rb
index 8b04b840b034b290d23381de376d1dba79c53a5c..1e4b00a1c058f3de1a69414fe8d071fac0fd0d93 100644 (file)
--- a/cookbooks/nfs/recipes/server.rb
+++ b/cookbooks/nfs/recipes/server.rb
@@ -25,10 +25,10 @@ service "rpcbind" do
   supports :status => true, :restart => true, :reload => true
 end
 
   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 = {}
 
 
 exports = {}
 

diff --git a/cookbooks/nginx/attributes/default.rb b/cookbooks/nginx/attributes/default.rb
index 7e054bd64a2c4a994c6c1f20208242182f0af777..f28efd253f9934d544bb9721756fad5002b7745e 100644 (file)
--- a/cookbooks/nginx/attributes/default.rb
+++ b/cookbooks/nginx/attributes/default.rb
@@ -1 +1 @@
-#default[:nginx][:mpm] = "worker"
+# default[:nginx][:mpm] = "worker"

diff --git a/cookbooks/nginx/recipes/default.rb b/cookbooks/nginx/recipes/default.rb
index 540e97c241ee8e58cb2fca16a1411184bddceabc..b92d15749fffef25500da8a806c409c580fec096 100644 (file)
--- a/cookbooks/nginx/recipes/default.rb
+++ b/cookbooks/nginx/recipes/default.rb
@@ -19,7 +19,7 @@
 
 package "nginx"
 
 
 package "nginx"
 

-#admins = data_bag_item("nginx", "admins")
+# admins = data_bag_item("nginx", "admins")

 
 template "/etc/nginx/nginx.conf" do
   source "nginx.conf.erb"
 
 template "/etc/nginx/nginx.conf" do
   source "nginx.conf.erb"

diff --git a/cookbooks/nominatim/recipes/default.rb b/cookbooks/nominatim/recipes/default.rb
index 2259c9da79f5114f7142fe3d031c16656ec0d921..ae1b7945603596c379ad65f60de13ac15e4dedc6 100644 (file)
--- a/cookbooks/nominatim/recipes/default.rb
+++ b/cookbooks/nominatim/recipes/default.rb
@@ -111,7 +111,6 @@ template "/etc/logrotate.d/nominatim" do
   mode 0644
 end
 
   mode 0644
 end
 

-

 package "osmosis"
 package "gcc"
 package "proj-bin"
 package "osmosis"
 package "gcc"
 package "proj-bin"


@@ -161,7 +160,6 @@ directory "#{source_directory}/log" do
   mode 0755
 end
 
   mode 0755
 end
 

-

 template "#{source_directory}/.git/hooks/post-merge" do
   source "update_source.erb"
   owner  "nominatim"
 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
 
   variables :bin_directory => "#{source_directory}/utils", :mailto => email_errors
 end
 

-

 template "#{source_directory}/utils/nominatim-update" do
   source "updater.erb"
   user   "nominatim"
 template "#{source_directory}/utils/nominatim-update" do
   source "updater.erb"
   user   "nominatim"

diff --git a/cookbooks/ntp/metadata.rb b/cookbooks/ntp/metadata.rb
index 6c5ce11e90e2b2548a1a6b6be50e05ebee8989e3..adcfc75ca36a05409d68a33a0a5a91502da1d0df 100644 (file)
--- a/cookbooks/ntp/metadata.rb
+++ b/cookbooks/ntp/metadata.rb
@@ -8,7 +8,7 @@ version           "0.8.2"
 
 recipe "ntp", "Installs and configures ntp either as a server or client"
 
 
 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
 
   supports os
 end
 

diff --git a/cookbooks/ntp/recipes/default.rb b/cookbooks/ntp/recipes/default.rb
index 12447a28d6442c9626c1586cafa551953fb27457..077fd7ac2d2274bd5b59d049a639e22c1c38492d 100644 (file)
--- a/cookbooks/ntp/recipes/default.rb
+++ b/cookbooks/ntp/recipes/default.rb
@@ -51,7 +51,7 @@ template "/etc/ntp.conf" do
   notifies :restart, "service[ntp]"
 end
 
   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"
 
 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 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
       munin_plugin plugin do
         action :delete
       end

diff --git a/cookbooks/openssh/recipes/default.rb b/cookbooks/openssh/recipes/default.rb
index 0bae591ceb4d2ea4c973b202fda626fb517074ad..90ac311b6b87fc5cc706fe6d98356e6388a4e634 100644 (file)
--- a/cookbooks/openssh/recipes/default.rb
+++ b/cookbooks/openssh/recipes/default.rb
@@ -31,9 +31,7 @@ service "ssh" do
   supports :status => true, :restart => true, :reload => true
 end
 
   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|
   names = [ node[:hostname] ]
 
   node.interfaces(:role => :external).each do |interface|

diff --git a/cookbooks/openvpn/recipes/default.rb b/cookbooks/openvpn/recipes/default.rb
index b1955aa8c633c2d42c4fa5c56a4edc52b289a41c..3a0bcccf5d7db1bed38dd71c2c5c4a8a7acd8313 100644 (file)
--- a/cookbooks/openvpn/recipes/default.rb
+++ b/cookbooks/openvpn/recipes/default.rb
@@ -27,7 +27,7 @@ end
 
 node[:openvpn][:tunnels].each do |name, details|
   if peer = search(:node, "fqdn:#{details[:peer][:host]}").first
 
 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
 
       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
     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"
     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,
       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
       notifies :restart, "service[openvpn]"
     end
   else

diff --git a/cookbooks/piwik/attributes/default.rb b/cookbooks/piwik/attributes/default.rb
index b044d41db56172cfd215cdac5a69714132f9cec6..9e7365d4a05e76116642ecce68201aee1d6c478e 100644 (file)
--- a/cookbooks/piwik/attributes/default.rb
+++ b/cookbooks/piwik/attributes/default.rb
@@ -1,13 +1,10 @@
 default[:piwik][:version] = "2.10.0"
 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
+)

diff --git a/cookbooks/planet/files/default/replication-bin/replicate-changesets b/cookbooks/planet/files/default/replication-bin/replicate-changesets
index daa7ab0b6319e48b93bd275af35e35a447ec8d95..b25f97aab6bd7c64420ed4b302141c8418dea9c8 100644 (file)
--- a/cookbooks/planet/files/default/replication-bin/replicate-changesets
+++ b/cookbooks/planet/files/default/replication-bin/replicate-changesets
@@ -33,7 +33,7 @@ class Changeset
   end
 
   def open?(t)
   end
 
   def open?(t)

-    not closed?(t)
+    !closed?(t)

   end
 
   def activity_between?(t1, t2)
   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.
     # 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
   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",
       '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")
 
     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']
 
       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
         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)
 
     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"
       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"

diff --git a/cookbooks/planet/recipes/dump.rb b/cookbooks/planet/recipes/dump.rb
index 63092565adc49fa3c2ed68645f24e403ad64f796..edffb0ef49a604da28934e860eb7f4069d5171ed 100644 (file)
--- a/cookbooks/planet/recipes/dump.rb
+++ b/cookbooks/planet/recipes/dump.rb
@@ -20,7 +20,7 @@
 node.default[:incron][:planetdump] = {
   :user => "www-data",
   :path => "/store/backup",
 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 $#"
 }
 
   :command => "/usr/local/bin/planetdump $#"
 }
 

diff --git a/cookbooks/postgresql/definitions/postgresql_munin.rb b/cookbooks/postgresql/definitions/postgresql_munin.rb
index 8062044f78b46010d0df05aebda44f32f808ece8..3b6056394785226d67651ae1d37a7f74b7d6cb4a 100644 (file)
--- a/cookbooks/postgresql/definitions/postgresql_munin.rb
+++ b/cookbooks/postgresql/definitions/postgresql_munin.rb
@@ -22,7 +22,7 @@ define :postgresql_munin, :action => :create do
   suffix = cluster.tr("/", ":")
   database = params[:database]
 
   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_"
     munin_plugin "postgres_cache_#{database}:#{suffix}" do
       action params[:action]
       target "postgres_cache_"

diff --git a/cookbooks/postgresql/libraries/postgresql.rb b/cookbooks/postgresql/libraries/postgresql.rb
index 316f8eca37f4f200de7b0e6af5ec8b5f1e516a29..c12174753b14ecbb8fb867b2e27b47cd3ffcb985 100644 (file)
--- a/cookbooks/postgresql/libraries/postgresql.rb
+++ b/cookbooks/postgresql/libraries/postgresql.rb
@@ -63,58 +63,53 @@ class Chef
     end
 
     def users
     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[user[:usename]] = {
           :superuser => user[:usesuper] == "t",
           :createdb => user[:usercreatedb] == "t",
           :createrole => user[:usecatupd] == "t",
           :replication => user[:userepl] == "t"
         }

-        users

       end
     end
 
     def databases
       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[database[:datname]] = {
           :owner => database[:usename],
           :encoding => database[:encoding],
           :collate => database[:datcollate],
           :ctype => database[:datctype]
         }

-        databases

       end
     end
 
     def extensions(database)
       @extensions ||= {}
       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]
         }
         extensions[extension[:extname]] = {
           :version => extension[:extversion]
         }

-        databases

       end
     end
 
     def tables(database)
       @tables ||= {}
       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] || "{}")
         }
         name = "#{table[:nspname]}.#{table[:relname]}"
 
         tables[name] = {
           :owner => table[:usename],
           :permissions => parse_acl(table[:relacl] || "{}")
         }

-
-        tables

       end
     end
 
   private
 
     def parse_acl(acl)
       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")
         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
           "a" => :insert, "r" => :select, "w" => :update, "d" => :delete,
           "D" => :truncate, "x" => :references, "t" => :trigger
         }.values_at(*(privileges.chars)).compact

-
-        permissions

       end
     end
   end
       end
     end
   end

diff --git a/cookbooks/rsyncd/recipes/default.rb b/cookbooks/rsyncd/recipes/default.rb
index 41eaee80a4549af1be7d6cd116337e2bb7826366..fc0c4a5041a6acefd4cb8604e148ee00b7f768d4 100644 (file)
--- a/cookbooks/rsyncd/recipes/default.rb
+++ b/cookbooks/rsyncd/recipes/default.rb
@@ -19,8 +19,8 @@
 
 include_recipe "networking"
 
 
 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] || []
 
 node[:rsyncd][:modules].each do |name, details|
   hosts_allow[name] = details[:hosts_allow] || []

diff --git a/cookbooks/sysctl/metadata.rb b/cookbooks/sysctl/metadata.rb
index e5a5287edff7f6caaf405e7a8b81a089b814503b..6d0db322955ec790cc31721583d97b285fe9afc5 100644 (file)
--- a/cookbooks/sysctl/metadata.rb
+++ b/cookbooks/sysctl/metadata.rb
@@ -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"
 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"
   supports os
 end
 recipe           "sysctl", "Configure kernel parameters"

diff --git a/cookbooks/thinkup/recipes/default.rb b/cookbooks/thinkup/recipes/default.rb
index c7b6ed23f71c39f3370e4698162c597197bd3653..c06feb7f925105457198e8e71a5447eeb55ef864 100644 (file)
--- a/cookbooks/thinkup/recipes/default.rb
+++ b/cookbooks/thinkup/recipes/default.rb
@@ -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\['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
   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\"")
 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")
 
   line.gsub!(/^php="[^"]*"$/, "php=\"/usr/bin/php\"")
   line.gsub!(/^#crawl_interval=[0-9]+$/, "crawl_interval=30")
 

diff --git a/cookbooks/tile/files/default/ruby/expire.rb b/cookbooks/tile/files/default/ruby/expire.rb
index 4c9d1a31becd76967c0b2727ce403772699fbbc6..c6916943ade718c8c0840be36a0549aeb88e348b 100755 (executable)
--- a/cookbooks/tile/files/default/ruby/expire.rb
+++ b/cookbooks/tile/files/default/ruby/expire.rb
@@ -24,7 +24,7 @@ module Expire
   NODE_CACHE_FILE = "/store/database/nodes"
 
   # turns a spherical mercator coord into a tile coord
   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
     # 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
   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
     # 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
     # 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)
       (((x >> 4 * i) & 0xf) << 4) | ((y >> 4 * i) & 0xf)

-    }.reverse.join('/')
+    end.reverse.join('/')

     z.to_s + '/' + hash_path + ".meta"
   end
 
     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
   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
 
     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
     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
 
     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
     # 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.
 
     # 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.

diff --git a/cookbooks/tile/recipes/default.rb b/cookbooks/tile/recipes/default.rb
index fc4feb0b63fe19eeefffb68fbb6d5685f9d5b588..675f8e7df0a17cb92ed7567787d1e89753cb4f31 100644 (file)
--- a/cookbooks/tile/recipes/default.rb
+++ b/cookbooks/tile/recipes/default.rb
@@ -351,13 +351,7 @@ postgresql_extension "postgis" do
   database "gis"
 end
 
   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"
   postgresql_table table do
     cluster node[:tile][:database][:cluster]
     database "gis"


@@ -366,11 +360,7 @@ end
   end
 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"
   postgresql_table table do
     cluster node[:tile][:database][:cluster]
     database "gis"

diff --git a/cookbooks/tilecache/attributes/default.rb b/cookbooks/tilecache/attributes/default.rb
index a5a25db1160f65fba1718eceec06c94a9b768e6c..6487cd9071a6bc8b8a1dd5a8bcb70c24fb93f388 100644 (file)
--- a/cookbooks/tilecache/attributes/default.rb
+++ b/cookbooks/tilecache/attributes/default.rb
@@ -2,13 +2,13 @@
 default[:tilecache][:tile_parent] = "parent.tile.openstreetmap.org"
 default[:tilecache][:tile_siblings] = []
 
 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"
 default[:tilecache][:ip_bucket_refill] = "4096"

-#Per IP bucket size
+# Per IP bucket size

 default[:tilecache][:ip_bucket_size] = "67108864"
 default[:tilecache][:ip_bucket_size] = "67108864"

-#Per Class C refill rate
+# Per Class C refill rate

 default[:tilecache][:net_bucket_refill] = "8192"
 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"
 default[:tilecache][:net_bucket_size] = "134217728"
 
 default[:tilecache][:ssl][:certificate] = "tile.openstreetmap"

diff --git a/cookbooks/tools/recipes/default.rb b/cookbooks/tools/recipes/default.rb
index a2245a407ab684b71c15c601d5675687ef6b8a13..8424642bfe0df7f6c79ff2eebc3c3447d82ef835 100644 (file)
--- a/cookbooks/tools/recipes/default.rb
+++ b/cookbooks/tools/recipes/default.rb
@@ -45,12 +45,12 @@ service "rsyslog" do
   supports :status => true, :restart => true, :reload => true
 end
 
   supports :status => true, :restart => true, :reload => true
 end
 

-#Remove unused base package
+# Remove unused base package

 package "mlocate" do
   action :purge
 end
 
 package "mlocate" do
   action :purge
 end
 

-#Remove ubuntu "desktop" vestigal package
+# Remove ubuntu "desktop" vestigal package

 package "whoopsie" do
   action :purge
 end
 package "whoopsie" do
   action :purge
 end

diff --git a/cookbooks/trac/files/default/trac-authenticate b/cookbooks/trac/files/default/trac-authenticate
index cfd33e756f8c75a8d56e572d68a4870707d0f85b..489a12ca06825b06d234fa9422a4f4119d8a7bb0 100755 (executable)
--- a/cookbooks/trac/files/default/trac-authenticate
+++ b/cookbooks/trac/files/default/trac-authenticate
@@ -11,5 +11,5 @@ request.basic_auth user, pass
 
 response = Net::HTTP.new("api.openstreetmap.org").request(request)
 
 
 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)
 exit!(1)

diff --git a/cookbooks/web/definitions/rails_port.rb b/cookbooks/web/definitions/rails_port.rb
index 6abec935710c6078d6bf99f4a938f52444d44cbb..9fa88c52e686354b60ad86a1548c181396e30185 100644 (file)
--- a/cookbooks/web/definitions/rails_port.rb
+++ b/cookbooks/web/definitions/rails_port.rb
@@ -242,7 +242,7 @@ define :rails_port, :action => [ :create, :enable ] do
     cwd "#{rails_directory}/lib/quad_tile"
     user rails_user
     group rails_group
     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
   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
     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
     end
     notifies :touch, "file[#{rails_directory}/tmp/restart.txt]"
   end

diff --git a/cookbooks/web/recipes/cgimap.rb b/cookbooks/web/recipes/cgimap.rb
index 4a8ab12c1ce0d0379e27f2de42844458677a47c4..09a3f787e3455ecb2857af0aba5d9b1e75f51bbc 100644 (file)
--- a/cookbooks/web/recipes/cgimap.rb
+++ b/cookbooks/web/recipes/cgimap.rb
@@ -108,7 +108,7 @@ file "/etc/init.d/cgimap" do
   content cgimap_init
 end
 
   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
   service "cgimap" do
     action :stop
   end

diff --git a/cookbooks/web/recipes/gpx.rb b/cookbooks/web/recipes/gpx.rb
index b2117dde9cc466bb2176f9adb51f5e3a35cb5be9..10463da89f966ad98f7c88602fa08ae100f1b867 100644 (file)
--- a/cookbooks/web/recipes/gpx.rb
+++ b/cookbooks/web/recipes/gpx.rb
@@ -67,7 +67,7 @@ template "/etc/init.d/gpx-import" do
             :database_password => db_passwords["gpximport"]
 end
 
             :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
   service "gpx-import" do
     action :stop
   end

diff --git a/cookbooks/web/recipes/rails.rb b/cookbooks/web/recipes/rails.rb
index bc10fac3980a681730d1655a6a12b24cf84ee673..6616c49f0980434733d86e26d2968f2f40254110 100644 (file)
--- a/cookbooks/web/recipes/rails.rb
+++ b/cookbooks/web/recipes/rails.rb
@@ -38,7 +38,7 @@ end
 rails_directory = "#{node[:web][:base_directory]}/rails"
 
 piwik_configuration = data_bag_item("web", "piwik").to_hash.reject do |k, _|
 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
 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"
   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"]
   potlatch2_key web_passwords["potlatch2_key"]
   id_key web_passwords["id_key"]
   oauth_key web_passwords["oauth_key"]

diff --git a/cookbooks/wiki/attributes/default.rb b/cookbooks/wiki/attributes/default.rb
index 2d4ae94742b7eb3fb29aa2f2ca759ac97f42eec5..1d73d9ff0d2a366dc4f341aebbbf507789cc2c5f 100644 (file)
--- a/cookbooks/wiki/attributes/default.rb
+++ b/cookbooks/wiki/attributes/default.rb
@@ -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"

diff --git a/cookbooks/wiki/recipes/default.rb b/cookbooks/wiki/recipes/default.rb
index 29fa60db5f841676fae9dddbedf8ea9823cb6c5b..85bb7466f7cc5d93afdc66a21b2a3b5d32329f81 100644 (file)
--- a/cookbooks/wiki/recipes/default.rb
+++ b/cookbooks/wiki/recipes/default.rb
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
 # limitations under the License.
 #
 

-#include_recipe "squid"
+# include_recipe "squid"

 
 include_recipe "mediawiki"
 
 
 include_recipe "mediawiki"
 


@@ -50,8 +50,7 @@ mediawiki_site  "wiki.openstreetmap.org" do
   recaptcha_public_key "6LdFIQATAAAAAMwtHeI8KDgPqvRbXeNYSq1gujKz"
   recaptcha_private_key passwords["recaptcha"]
 
   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
 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"
 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
 end
 
 template "/etc/cron.d/wiki-osm-org-dump" do

diff --git a/cookbooks/wordpress/attributes/default.rb b/cookbooks/wordpress/attributes/default.rb
index 615e08925429e9a357f455c72f418ef8e85a0a3f..09845b949d9e1a0adbbfa126fa9c00ad150b674c 100644 (file)
--- a/cookbooks/wordpress/attributes/default.rb
+++ b/cookbooks/wordpress/attributes/default.rb
@@ -5,7 +5,7 @@ default[:accounts][:users][:wordpress][:status] = :role
 default[:apache][:mpm] = "prefork"
 
 # Make sure httpclient and php_serialize are installed
 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"
 
 # Set wordpress defaults
 default[:wordpress][:user] = "wordpress"

diff --git a/cookbooks/wordpress/libraries/wordpress.rb b/cookbooks/wordpress/libraries/wordpress.rb
index 8ebb1460c621dc953fae8e1d41462239dfa4e567..f8046aaaa5786aa3b71d49054c90863b18c96757 100644 (file)
--- a/cookbooks/wordpress/libraries/wordpress.rb
+++ b/cookbooks/wordpress/libraries/wordpress.rb
@@ -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]*/
 
     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
       else
         "trunk"
       end

diff --git a/cookbooks/yournavigation/recipes/default.rb b/cookbooks/yournavigation/recipes/default.rb
index ce2c81b7ce0b2db36e5e3011a61935b3e191ea82..2accedee2e496c4cce78e5040bf8e41ecc60a73d 100644 (file)
--- a/cookbooks/yournavigation/recipes/default.rb
+++ b/cookbooks/yournavigation/recipes/default.rb
@@ -24,10 +24,10 @@ package "php5-cli"
 
 package "php-apc"
 
 
 package "php-apc"
 

-#Required for osmosis
+# Required for osmosis

 package "default-jre-headless"
 
 package "default-jre-headless"
 

-#Required for building gosmore
+# Required for building gosmore

 package "build-essential"
 package "libxml2-dev"
 package "libgtk2.0-dev"
 package "build-essential"
 package "libxml2-dev"
 package "libgtk2.0-dev"

diff --git a/roles/crm.rb b/roles/crm.rb
index 032bc40a3765c9d8697ec6553b20885826b5d468..ee6a58fff1abaad197cf043cef96e28b01f5ae18 100644 (file)
--- a/roles/crm.rb
+++ b/roles/crm.rb
@@ -1,7 +1,6 @@
 name "crm"
 description "Role applied to CRM server"
 
 name "crm"
 description "Role applied to CRM server"
 

-

 default_attributes(
   :exim => {
     :local_domains => [ "crm.osmfoundation.org" ],
 default_attributes(
   :exim => {
     :local_domains => [ "crm.osmfoundation.org" ],

diff --git a/roles/fume.rb b/roles/fume.rb
index e4c739136bfc60530b75ad48e7b4c873bdb45f1e..a6aa6f1d2801f4dae3d2a16e2f416e38a9a90e73 100644 (file)
--- a/roles/fume.rb
+++ b/roles/fume.rb
@@ -31,8 +31,8 @@ default_attributes(
       :parameters => {
         "block/cciss\!c0d0/queue/nr_requests" => "512",
         "block/cciss\!c0d1/queue/nr_requests" => "512",
       :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"

       }
     }
   },
       }
     }
   },

diff --git a/roles/katie.rb b/roles/katie.rb
index ff9c9ab53a0aafc4c295ba90ecd832a93361dced..1cd4ece16e34978eacad1e1677ca2952d5bdac91 100644 (file)
--- a/roles/katie.rb
+++ b/roles/katie.rb
@@ -29,14 +29,14 @@ default_attributes(
   :tilecache => {
     :tile_parent => "germany.render.openstreetmap.org",
     :tile_siblings => [
   :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"

     ]
   }
 )
     ]
   }
 )

diff --git a/roles/konqi.rb b/roles/konqi.rb
index 3b3fe2dc32880ec4fb8167808164326ca5198c9a..4dd68f0255eff20166bba4af5ea05890f70388ed 100644 (file)
--- a/roles/konqi.rb
+++ b/roles/konqi.rb
@@ -29,14 +29,14 @@ default_attributes(
   :tilecache => {
     :tile_parent => "germany.render.openstreetmap.org",
     :tile_siblings => [
   :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"

     ]
   }
 )
     ]
   }
 )

diff --git a/roles/nominatim.rb b/roles/nominatim.rb
index 0aa56aaf40c74e8ed77089ddcad8dfd58bfb7af3..dff1acca745ddde2df3c147fd84197d5ea8f28b0 100644 (file)
--- a/roles/nominatim.rb
+++ b/roles/nominatim.rb
@@ -64,7 +64,7 @@ default_attributes(
     :network_conntrack_max => {
       :comment => "Increase max number of connections tracked",
       :parameters => {
     :network_conntrack_max => {
       :comment => "Increase max number of connections tracked",
       :parameters => {

-         "net.netfilter.nf_conntrack_max" => "131072"
+        "net.netfilter.nf_conntrack_max" => "131072"

       }
     }
   },
       }
     }
   },

diff --git a/roles/norbert.rb b/roles/norbert.rb
index 28fc3aaa1f79cf1c7801f290100bf767391923e8..a96812200bf7b62d861b377266a0d14349059aab 100644 (file)
--- a/roles/norbert.rb
+++ b/roles/norbert.rb
@@ -35,10 +35,10 @@ default_attributes(
       :parameters => {
         "block/cciss\!c0d0/queue/nr_requests" => "512",
         "block/cciss\!c0d1/queue/nr_requests" => "512",
       :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"

       }
     }
   }
       }
     }
   }

diff --git a/roles/poldi.rb b/roles/poldi.rb
index ad8988a579ed5a2a7cbc91fb8da3004ba07374a7..127a5265b0d9fade6e5211a4302f61698efcd858 100644 (file)
--- a/roles/poldi.rb
+++ b/roles/poldi.rb
@@ -67,33 +67,33 @@ default_attributes(
     :enabled => false,
     :flatnode_file => "/ssd-old/nominatim/nodes.store",
     :database => {
     :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 => {
     },
     :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 => {
     },
     :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"

     }
   }
 )
     }
   }
 )

diff --git a/roles/pummelzacken.rb b/roles/pummelzacken.rb
index 0c03f3a1b4fffb17af7019858bc5d2d4fd9630f3..559cb61da41be41ea7e5c52397ee3a16526eac88 100644 (file)
--- a/roles/pummelzacken.rb
+++ b/roles/pummelzacken.rb
@@ -35,33 +35,33 @@ default_attributes(
     :enabled => true,
     :flatnode_file => "/ssd/nominatim/nodes.store",
     :database => {
     :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 => {
     },
     :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 => {
     },
     :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"

     }
   }
 )
     }
   }
 )

diff --git a/roles/smaug.rb b/roles/smaug.rb
index 9937714f86d3790da17444a0e55d2a05fc43002d..2554b865b94e2b680b3f1b71724d514b6be4959a 100644 (file)
--- a/roles/smaug.rb
+++ b/roles/smaug.rb
@@ -64,8 +64,8 @@ default_attributes(
       :parameters => {
         "block/sda/queue/nr_requests" => "512",
         "block/sdb/queue/nr_requests" => "512",
       :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"

       }
     }
   }
       }
     }
   }

diff --git a/roles/tilecache.rb b/roles/tilecache.rb
index 3a6d7fa5522c713da022fbe71e2d9308b2a78852..db78d4d76c6371d02eb18b70c11fea45344057cb 100644 (file)
--- a/roles/tilecache.rb
+++ b/roles/tilecache.rb
@@ -15,7 +15,7 @@ default_attributes(
     :network_conntrack_max => {
       :comment => "Increase max number of connections tracked",
       :parameters => {
     :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 => {
       }
     },
     :squid_swappiness => {

diff --git a/roles/tyan-s7010.rb b/roles/tyan-s7010.rb
index 6cd45ebb4536e8a95f996c00b664c3ad9f9699a3..30d13b774a2599d434fd8db89bde77210089fb87 100644 (file)
--- a/roles/tyan-s7010.rb
+++ b/roles/tyan-s7010.rb
@@ -3,9 +3,7 @@ description "Role applied to machines using the Tyan S710 motherboard"
 
 default_attributes(
   :hardware => {
 
 default_attributes(
   :hardware => {

-    :modules => [
-      "i2c_i801", "jc42", "w83793"
-    ],
+    :modules => %w(i2c_i801 jc42 w83793),

     :sensors => {
       "jc42-*" => {
         :temps => {
     :sensors => {
       "jc42-*" => {
         :temps => {