Yet more rubucop cleanups
authorTom Hughes <tom@compton.nu>
Tue, 3 Feb 2015 17:16:43 +0000 (17:16 +0000)
committerTom Hughes <tom@compton.nu>
Tue, 3 Feb 2015 17:22:54 +0000 (17:22 +0000)
129 files changed:
.rubocop_todo.yml
cookbooks/apache/recipes/default.rb
cookbooks/apache/recipes/ssl.rb
cookbooks/bind/recipes/default.rb
cookbooks/chef/attributes/default.rb
cookbooks/chef/definitions/ohai_plugin.rb
cookbooks/chef/files/default/knife.rb
cookbooks/chef/recipes/default.rb
cookbooks/chef/recipes/server.rb
cookbooks/civicrm/recipes/default.rb
cookbooks/clamav/recipes/default.rb
cookbooks/dev/recipes/default.rb
cookbooks/dhcpd/recipes/default.rb
cookbooks/elasticsearch/attributes/default.rb
cookbooks/elasticsearch/recipes/default.rb
cookbooks/exim/attributes/default.rb
cookbooks/exim/metadata.rb
cookbooks/exim/recipes/default.rb
cookbooks/fail2ban/recipes/default.rb
cookbooks/forum/recipes/default.rb
cookbooks/foundation/recipes/board.rb
cookbooks/foundation/recipes/wiki.rb
cookbooks/geodns/recipes/default.rb
cookbooks/gps-tile/recipes/default.rb
cookbooks/hardware/attributes/default.rb
cookbooks/hardware/recipes/default.rb
cookbooks/incron/recipes/default.rb
cookbooks/mailman/recipes/default.rb
cookbooks/mediawiki/definitions/mediawiki_extension.rb
cookbooks/mediawiki/definitions/mediawiki_site.rb
cookbooks/memcached/recipes/default.rb
cookbooks/munin/files/default/plugins/api_calls_
cookbooks/munin/files/default/plugins/api_waits_
cookbooks/munin/files/default/plugins/passenger_memory
cookbooks/munin/files/default/plugins/passenger_processes
cookbooks/munin/files/default/plugins/passenger_queues
cookbooks/munin/files/default/plugins/passenger_requests
cookbooks/munin/recipes/default.rb
cookbooks/munin/recipes/server.rb
cookbooks/mysql/recipes/default.rb
cookbooks/networking/attributes/default.rb
cookbooks/networking/recipes/default.rb
cookbooks/nfs/recipes/default.rb
cookbooks/nfs/recipes/server.rb
cookbooks/nginx/definitions/nginx_site.rb
cookbooks/nginx/recipes/default.rb
cookbooks/nominatim/recipes/default.rb
cookbooks/ntp/recipes/default.rb
cookbooks/openssh/recipes/default.rb
cookbooks/openvpn/recipes/default.rb
cookbooks/owl/recipes/default.rb
cookbooks/passenger/attributes/default.rb
cookbooks/planet/recipes/replication.rb
cookbooks/postgresql/recipes/default.rb
cookbooks/rsyncd/attributes/default.rb
cookbooks/rsyncd/recipes/default.rb
cookbooks/snmpd/attributes/default.rb
cookbooks/snmpd/recipes/default.rb
cookbooks/spamassassin/recipes/default.rb
cookbooks/squid/definitions/squid_fragment.rb
cookbooks/squid/recipes/default.rb
cookbooks/ssl/recipes/default.rb
cookbooks/supybot/recipes/default.rb
cookbooks/tile/recipes/default.rb
cookbooks/tilecache/recipes/default.rb
cookbooks/tools/recipes/default.rb
cookbooks/web/definitions/rails_port.rb
cookbooks/web/recipes/backend.rb
cookbooks/web/recipes/cgimap.rb
cookbooks/web/recipes/frontend.rb
cookbooks/web/recipes/gpx.rb
cookbooks/wiki/recipes/default.rb
cookbooks/wordpress/definitions/wordpress_plugin.rb
cookbooks/wordpress/definitions/wordpress_site.rb
cookbooks/wordpress/definitions/wordpress_theme.rb
cookbooks/xinetd/recipes/default.rb
roles/base.rb
roles/blix-nl.rb
roles/blix-no.rb
roles/blix.rb
roles/bluehost.rb
roles/bytemark.rb
roles/carnet.rb
roles/crm.rb
roles/db-master.rb
roles/db.rb
roles/delta.rb
roles/dev.rb
roles/euserv.rb
roles/faffy.rb
roles/firefishynet.rb
roles/forum.rb
roles/foundation.rb
roles/gateway.rb
roles/git.rb
roles/gps-tile.rb
roles/hetzner.rb
roles/ic.rb
roles/ironbelly.rb
roles/jump.rb
roles/lyonix.rb
roles/mail.rb
roles/nchc.rb
roles/nominatim.rb
roles/orm.rb
roles/osuosl.rb
roles/otrs.rb
roles/ovh-ca.rb
roles/ovh.rb
roles/owl.rb
roles/paulla.rb
roles/planet.rb
roles/poldi.rb
roles/prgmr.rb
roles/pummelzacken.rb
roles/ridley.rb
roles/smaug.rb
roles/taginfo.rb
roles/teleservice.rb
roles/tile.rb
roles/tilecache.rb
roles/ucl-wates.rb
roles/ucl-wolfson.rb
roles/ucl.rb
roles/web-frontend.rb
roles/web.rb
roles/wiki.rb
roles/yandex.rb
roles/yevaud.rb

index 6c5c982..a30c810 100644 (file)
@@ -101,21 +101,6 @@ Style/RegexpLiteral:
 Style/SelfAssignment:
   Enabled: false
 
-# Offense count: 27
-# Cop supports --auto-correct.
-Style/SingleSpaceBeforeFirstArg:
-  Enabled: false
-
-# Offense count: 410
-# Cop supports --auto-correct.
-Style/SpaceInsideBrackets:
-  Enabled: false
-
-# Offense count: 6
-# Cop supports --auto-correct.
-Style/SpecialGlobalVars:
-  Enabled: false
-
 # Offense count: 6548
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles.
index 30f7734..6c2ee79 100644 (file)
@@ -28,11 +28,11 @@ else
   %w(event itk prefork worker).each do |mpm|
     if mpm == node[:apache][:mpm]
       apache_module "mpm_#{mpm}" do
-        action [ :enable ]
+        action [:enable]
       end
     else
       apache_module "mpm_#{mpm}" do
-        action [ :disable ]
+        action [:disable]
       end
     end
   end
@@ -63,7 +63,7 @@ template "/etc/apache2/ports.conf" do
 end
 
 service "apache2" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
@@ -78,7 +78,7 @@ apache_module "status" do
 end
 
 apache_module "reqtimeout" do
-  action [ :disable ]
+  action [:disable]
 end
 
 munin_plugin "apache_accesses"
index 700e10e..c857704 100644 (file)
@@ -19,7 +19,7 @@
 
 certificate = node[:apache][:ssl][:certificate]
 
-node.default[:ssl][:certificates] = node[:ssl][:certificates] | [ certificate ]
+node.default[:ssl][:certificates] = node[:ssl][:certificates] | [certificate]
 
 include_recipe "apache"
 include_recipe "ssl"
index 1d2a8d8..519b9c8 100644 (file)
@@ -22,7 +22,7 @@ include_recipe "networking"
 package "bind9"
 
 service "bind9" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
index d6fe448..4b0e6ea 100644 (file)
@@ -1,5 +1,5 @@
 # Add the opscode APT source for chef
-default[:apt][:sources] = node[:apt][:sources] | [ "opscode" ]
+default[:apt][:sources] = node[:apt][:sources] | ["opscode"]
 
 # Set the default server version
 default[:chef][:server][:version] = "11.1.3-1"
index 50be323..be7c18e 100644 (file)
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-define :ohai_plugin, :action => [ :create, :delete ] do
+define :ohai_plugin, :action => [:create, :delete] do
   plugin_name = params[:name]
   plugin_action = params[:action]
 
index 1e191ce..a65cba9 100644 (file)
@@ -5,7 +5,7 @@ validation_key '/etc/chef/validation.pem'
 chef_server_url 'https://chef.openstreetmap.org'
 cache_type 'BasicFile'
 cache_options :path => '.chef/checksums'
-cookbook_path [ 'cookbooks' ]
+cookbook_path ['cookbooks']
 cookbook_copyright 'OpenStreetMap Administrators'
 cookbook_email 'admins@openstreetmap.org'
 cookbook_license 'apachev2'
index 600a21b..a9fa67b 100644 (file)
@@ -123,7 +123,7 @@ end
 
 service "chef-client" do
   provider Chef::Provider::Service::Upstart
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
   subscribes :restart, "dpkg_package[chef]"
   subscribes :restart, "template[/etc/init/chef-client.conf]"
index 18a0312..f314515 100644 (file)
@@ -85,7 +85,7 @@ end
 
 service "chef-server-runsvdir" do
   provider Chef::Provider::Service::Upstart
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
index 6c2d5ec..7865752 100644 (file)
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-node.default[:ssl][:certificates] = node[:ssl][:certificates] | [ "crm.osmfoundation" ]
+node.default[:ssl][:certificates] = node[:ssl][:certificates] | ["crm.osmfoundation"]
 
 include_recipe "wordpress"
 include_recipe "mysql"
index ff34b85..08c873e 100644 (file)
@@ -21,11 +21,11 @@ package "clamav-daemon"
 package "clamav-freshclam"
 
 service "clamav-daemon" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true
 end
 
 service "clamav-freshclam" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true
 end
index a724d69..75dcefc 100644 (file)
@@ -66,7 +66,7 @@ end
 
 service "php5-fpm" do
   provider Chef::Provider::Service::Upstart
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
@@ -182,7 +182,7 @@ if node[:postgresql][:clusters][:"9.1/main"]
       end
     else
       apache_site site_name do
-        action [ :delete ]
+        action [:delete]
       end
 
       directory rails_directory do
index bcf49aa..35f327d 100644 (file)
@@ -32,7 +32,7 @@ template "/etc/dhcp/dhcpd.conf" do
 end
 
 service "isc-dhcp-server" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true
   subscribes :restart, "template[/etc/dhcp/dhcpd.conf]"
 end
index d9afa93..e5e5c35 100644 (file)
@@ -1 +1 @@
-default[:apt][:sources] |= [ "elasticsearch" ]
+default[:apt][:sources] |= ["elasticsearch"]
index 41ea68a..aaabe4c 100644 (file)
@@ -23,6 +23,6 @@ package "elasticsearch" do
 end
 
 service "elasticsearch" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true
 end
index 9575f51..77b8d87 100644 (file)
@@ -1,10 +1,10 @@
-default[:exim][:local_domains] = [ "localhost", "@" ]
-default[:exim][:relay_to_domains] = [ ]
-default[:exim][:relay_from_hosts] = [ "127.0.0.1", "::1" ]
-default[:exim][:daemon_smtp_ports] = [ 25 ]
-default[:exim][:trusted_users] = [ ]
+default[:exim][:local_domains] = ["localhost", "@"]
+default[:exim][:relay_to_domains] = []
+default[:exim][:relay_from_hosts] = ["127.0.0.1", "::1"]
+default[:exim][:daemon_smtp_ports] = [25]
+default[:exim][:trusted_users] = []
 default[:exim][:smarthost_name] = nil
 default[:exim][:smarthost_via] = "mail.openstreetmap.org:26"
 default[:exim][:routes] = {}
 default[:exim][:aliases][:root] = "tomh"
-default[:exim][:rewrites] = [ ]
+default[:exim][:rewrites] = []
index 98a7c73..df68d45 100644 (file)
@@ -15,27 +15,27 @@ attribute "exim",
 attribute "exim/local_domains",
   :display_name => "Domains to Handle Locally",
   :description => "List of domains we are prepared to accept mail for",
-  :default => [ "@" ]
+  :default => ["@"]
 
 attribute "exim/relay_to_domains",
   :display_name => "Domains to Relay To",
   :description => "List of domains we are prepared to relay to",
-  :default => [ ]
+  :default => []
 
 attribute "exim/relay_from_hosts",
   :display_name => "Hosts to Relay From",
   :description => "List of hosts we are prepared to relay from",
-  :default => [ "127.0.0.1", "::1" ]
+  :default => ["127.0.0.1", "::1"]
 
 attribute "exim/daemon_smtp_ports",
   :display_name => "Ports to Listen On",
   :description => "List of ports we will listen on",
-  :default => [ 25 ]
+  :default => [25]
 
 attribute "exim/trusted_users",
   :display_name => "Trusted Users",
   :description => "List of users we will trust",
-  :default => [ ]
+  :default => []
 
 attribute "exim/smarthost_name",
   :display_name => "Smarthost Name",
index c334f9c..9c687a3 100644 (file)
@@ -53,7 +53,7 @@ execute "/etc/ssl/certs/exim.pem" do
 end
 
 service "exim4" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
   subscribes :restart, "execute[/etc/ssl/certs/exim.pem]"
 end
index 1fe3840..0fdc6c9 100644 (file)
@@ -28,7 +28,7 @@ template "/etc/fail2ban/jail.local" do
 end
 
 service "fail2ban" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :reload => true, :restart => true
   subscribes :reload, "template[/etc/fail2ban/jail.local]"
 end
index 25de3e8..7369863 100644 (file)
@@ -30,7 +30,7 @@ package "php-apc"
 apache_module "php5"
 
 apache_site "default" do
-  action [ :disable ]
+  action [:disable]
 end
 
 apache_site "forum.openstreetmap.org" do
index 8c27088..06d9b8a 100644 (file)
@@ -21,7 +21,7 @@ include_recipe "mediawiki"
 
 passwords = data_bag_item("foundation", "passwords")
 
-mediawiki_site  "board.osmfoundation.org" do
+mediawiki_site "board.osmfoundation.org" do
   directory "/srv/board.osmfoundation.org"
   database_name "board-wiki"
   database_username "board-wikiuser"
index 5d54b54..19a4b48 100644 (file)
@@ -21,7 +21,7 @@ include_recipe "mediawiki"
 
 passwords = data_bag_item("foundation", "passwords")
 
-mediawiki_site  "wiki.osmfoundation.org" do
+mediawiki_site "wiki.osmfoundation.org" do
   aliases "www.osmfoundation.org", "osmfoundation.org"
   directory "/srv/wiki.osmfoundation.org"
   database_name "osmf-wiki"
index 2308a57..98e91a8 100644 (file)
@@ -22,7 +22,7 @@ package "geoip-database-contrib"
 package "gdnsd"
 
 service "gdnsd" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
index a243250..e4ac47d 100644 (file)
@@ -86,7 +86,7 @@ template "/etc/init.d/gps-update" do
 end
 
 service "gps-update" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :restart => true, :status => true
   subscribes :restart, "git[/srv/gps-tile.openstreetmap.org/updater]"
 end
index 98c6f91..c0a805b 100644 (file)
@@ -4,7 +4,7 @@ default[:hardware][:sensors] = {}
 if node[:dmi] && node[:dmi][:system]
   case dmi.system.manufacturer
   when "HP"
-    default[:apt][:sources] |= [ "management-component-pack" ]
+    default[:apt][:sources] |= ["management-component-pack"]
 
     case dmi.system.product_name
     when "ProLiant DL360 G6", "ProLiant DL360 G7"
@@ -14,10 +14,10 @@ if node[:dmi] && node[:dmi][:system]
 end
 
 if Chef::Util.compare_versions(node[:kernel][:release], [3, 3]) < 0
-  default[:hardware][:modules] |= [ "microcode" ]
+  default[:hardware][:modules] |= ["microcode"]
 
   if node[:cpu]["0"][:vendor_id] == "GenuineIntel"
-    default[:hardware][:modules] |= [ "coretemp" ]
+    default[:hardware][:modules] |= ["coretemp"]
   end
 end
 
@@ -25,12 +25,12 @@ 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" ]
+    default[:apt][:sources] |= ["hwraid"]
   end
 end
 
 if node[:kernel][:modules].include?("ipmi_si")
-  default[:hardware][:modules] |= [ "ipmi_devintf" ]
+  default[:hardware][:modules] |= ["ipmi_devintf"]
 end
 
 if File.exist?("/proc/xen")
index 4f0ec7d..a0ee584 100644 (file)
@@ -111,7 +111,7 @@ unless unit.nil?
 
   service "ttyS#{unit}" do
     provider Chef::Provider::Service::Upstart
-    action [ :enable, :start ]
+    action [:enable, :start]
     supports :status => true, :restart => true, :reload => false
     subscribes :restart, "template[/etc/init/ttyS#{unit}.conf]"
   end
@@ -168,7 +168,7 @@ if node[:lsb][:release].to_f >= 12.10
   end
 
   service "irqbalance" do
-    action [ :start, :enable ]
+    action [:start, :enable]
     supports :status => false, :restart => true, :reload => false
     subscribes :restart, "template[/etc/default/irqbalance]"
   end
@@ -206,10 +206,10 @@ end
 node[:block_device].each do |name, attributes|
   if attributes[:vendor] == "HP" && attributes[:model] == "LOGICAL VOLUME"
     if name =~ /^cciss!(c[0-9]+)d[0-9]+$/
-      status_packages["cciss-vol-status"] |= [ "cciss/#{Regexp.last_match[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) ]
+        status_packages["cciss-vol-status"] |= [File.basename(sg)]
       end
     end
   end
@@ -238,7 +238,7 @@ end
     end
 
     service "#{status_package}d" do
-      action [ :start, :enable ]
+      action [:start, :enable]
       supports :status => false, :restart => true, :reload => false
       subscribes :restart, "template[/etc/default/#{status_package}d]"
     end
@@ -306,7 +306,7 @@ if node[:hardware][:watchdog]
   end
 
   service "watchdog" do
-    action [ :enable, :start ]
+    action [:enable, :start]
   end
 end
 
index ef44a69..5423bc7 100644 (file)
@@ -20,7 +20,7 @@
 package "incron"
 
 service "incron" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :reload => true, :restart => true
 end
 
index 94de382..1719792 100644 (file)
@@ -22,7 +22,7 @@ include_recipe "apache::ssl"
 package "mailman"
 
 service "mailman" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :restart => true, :reload => true
 end
 
index 1d507c6..c5d1ec8 100644 (file)
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-define :mediawiki_extension, :action => [ :enable ], :variables => {} do
+define :mediawiki_extension, :action => [:enable], :variables => {} do
   name = params[:name]
   site = params[:site]
   mediawiki_directory = node[:mediawiki][:sites][site][:directory]
index 94441b3..d565bed 100644 (file)
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-define :mediawiki_site, :action => [ :create, :enable ] do
+define :mediawiki_site, :action => [:create, :enable] do
   name = params[:name]
 
   # /etc/cron.d names cannot contain a dot
index 0ed312b..8ef4cec 100644 (file)
@@ -20,7 +20,7 @@
 package "memcached"
 
 service "memcached" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true
 end
 
index cc2fd8a..f56fa48 100755 (executable)
@@ -43,7 +43,7 @@ def categorise_uri(line)
   end
 end
 
-server = $0.match("api_calls_(.*)")[1]
+server = $PROGRAM_NAME.match("api_calls_(.*)")[1]
 
 if ARGV[0] == 'config'
   puts "graph_title Active requests"
index fcf956c..f781526 100755 (executable)
@@ -43,7 +43,7 @@ def categorise_uri(line)
   end
 end
 
-server = $0.match("api_waits_(.*)")[1]
+server = $PROGRAM_NAME.match("api_waits_(.*)")[1]
 
 if ARGV[0] == 'config'
   puts "graph_title Wait times for active requests"
index 8c8a60b..7e6e36a 100755 (executable)
@@ -16,7 +16,7 @@ end
 
 def output_values
   status = `/usr/sbin/passenger-memory-stats | tail -1`
-  unless $?.success?
+  unless $CHILD_STATUS.success?
     $stderr.puts "failed executing passenger-memory-stats"
     exit 1
   end
index 75a3692..59f9cda 100755 (executable)
@@ -18,7 +18,7 @@ end
 
 def output_values
   status = `/usr/sbin/passenger-status`
-  unless $?.success?
+  unless $CHILD_STATUS.success?
     $stderr.puts "failed executing passenger-status"
     exit 1
   end
index 506afb8..a7060fd 100755 (executable)
@@ -13,7 +13,7 @@ end
 
 def output_values
   status = `/usr/sbin/passenger-status`
-  unless $?.success?
+  unless $CHILD_STATUS.success?
     $stderr.puts "failed executing passenger-status"
     exit 1
   end
index f783be9..39c87c8 100755 (executable)
@@ -17,7 +17,7 @@ end
 
 def output_values
   status = `/usr/sbin/passenger-status`
-  unless $?.success?
+  unless $CHILD_STATUS.success?
     $stderr.puts "failed executing passenger-status"
     exit 1
   end
index 59dc6e4..79c58df 100644 (file)
@@ -25,7 +25,7 @@ service "munin-node" do
   if node[:lsb][:release].to_f >= 14.04
     provider Chef::Provider::Service::Upstart
   end
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
@@ -128,7 +128,7 @@ else
 end
 
 disks = node[:block_device].select do |_, attributes|
-  [ "ATA", "FUJITSU", "SEAGATE", "DELL", "COMPAQ", "IBM-ESXS" ].include?(attributes[:vendor])
+  ["ATA", "FUJITSU", "SEAGATE", "DELL", "COMPAQ", "IBM-ESXS"].include?(attributes[:vendor])
 end
 
 if disks.empty?
index 0c74cd0..3e72ea1 100644 (file)
@@ -36,7 +36,7 @@ directory "/var/lib/munin/rrdcached" do
 end
 
 service "rrdcached" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   subscribes :restart, "template[/etc/default/rrdcached]"
 end
 
index f9d2e2d..063526f 100644 (file)
@@ -21,7 +21,7 @@ package "mysql-server"
 package "mysql-client"
 
 service "mysql" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
index 49bd27a..0caaa96 100644 (file)
@@ -1,3 +1,3 @@
 default[:networking][:interfaces] = {}
-default[:networking][:nameservers] = [ ]
-default[:networking][:search] = [ ]
+default[:networking][:nameservers] = []
+default[:networking][:search] = []
index ec6749f..dc5c692 100644 (file)
@@ -125,7 +125,7 @@ end
 package "shorewall"
 
 service "shorewall" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :restart => true
   status_command "shorewall status"
 end
@@ -230,7 +230,7 @@ unless node.interfaces(:family => :inet6).empty?
   package "shorewall6"
 
   service "shorewall6" do
-    action [ :enable, :start ]
+    action [:enable, :start]
     supports :restart => true
     status_command "shorewall6 status"
   end
index e5e88ba..8d44e2b 100644 (file)
@@ -35,7 +35,7 @@ node[:nfs].each do |mountpoint, details|
   end
 
   mount mountpoint do
-    action [ :mount, :enable ]
+    action [:mount, :enable]
     device "#{details[:host]}:#{details[:path]}"
     fstype "nfs"
     options mount_options
index 1e4b00a..1142adb 100644 (file)
@@ -21,7 +21,7 @@ package "nfs-kernel-server"
 
 service "rpcbind" do
   provider Chef::Provider::Service::Upstart
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
index f6e1f3a..dc63df5 100644 (file)
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-define :nginx_site, :action => [ :create ], :variables => {} do
+define :nginx_site, :action => [:create], :variables => {} do
   name = params[:name]
   directory = params[:directory] || "/var/www/#{name}"
   site_action = params[:action]
index b92d157..2c8b00a 100644 (file)
@@ -29,7 +29,7 @@ template "/etc/nginx/nginx.conf" do
 end
 
 service "nginx" do
-  action [ :enable ]
+  action [:enable]
   supports :status => true, :restart => true, :reload => true
   subscribes :restart, "template[/etc/nginx/nginx.conf]"
 end
index ae1b794..6134c1d 100644 (file)
@@ -44,7 +44,7 @@ postgis_version = node[:nominatim][:database][:postgis]
 
 service "php5-fpm" do
   provider Chef::Provider::Service::Upstart
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
@@ -55,7 +55,7 @@ apache_site "nominatim.openstreetmap.org" do
 end
 
 apache_site "default" do
-  action [ :disable ]
+  action [:disable]
 end
 
 node[:nominatim][:fpm_pools].each do |name, data|
@@ -162,9 +162,9 @@ end
 
 template "#{source_directory}/.git/hooks/post-merge" do
   source "update_source.erb"
-  owner  "nominatim"
-  group  "nominatim"
-  mode   0755
+  owner "nominatim"
+  group "nominatim"
+  mode 0755
   variables :source_directory => source_directory
 end
 
@@ -215,16 +215,16 @@ end
 
 template "#{source_directory}/utils/nominatim-update" do
   source "updater.erb"
-  user   "nominatim"
-  group  "nominatim"
-  mode   0755
+  user "nominatim"
+  group "nominatim"
+  mode 0755
 end
 
 template "/etc/init.d/nominatim-update" do
   source "updater.init.erb"
-  user   "nominatim"
-  group  "nominatim"
-  mode   0755
+  user "nominatim"
+  group "nominatim"
+  mode 0755
   variables :source_directory => source_directory
 end
 
@@ -251,25 +251,25 @@ end
 remote_file "#{source_directory}/data/wikipedia_article.sql.bin" do
   action :create_if_missing
   source "http://www.nominatim.org/data/wikipedia_article.sql.bin"
-  owner  "nominatim"
-  group  "nominatim"
-  mode   0644
+  owner "nominatim"
+  group "nominatim"
+  mode 0644
 end
 
 remote_file "#{source_directory}/data/wikipedia_redirect.sql.bin" do
   action :create_if_missing
   source "http://www.nominatim.org/data/wikipedia_redirect.sql.bin"
-  owner  "nominatim"
-  group  "nominatim"
-  mode   0644
+  owner "nominatim"
+  group "nominatim"
+  mode 0644
 end
 
 remote_file "#{source_directory}/data/gb_postcode_data.sql.gz" do
   action :create_if_missing
   source "http://www.nominatim.org/data/gb_postcode_data.sql.gz"
-  owner  "nominatim"
-  group  "nominatim"
-  mode   0644
+  owner "nominatim"
+  group "nominatim"
+  mode 0644
 end
 
 template "/usr/local/bin/backup-nominatim" do
index 077fd7a..c2aba25 100644 (file)
@@ -39,7 +39,7 @@ file "/etc/timezone" do
 end
 
 service "ntp" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true
 end
 
index 90ac311..fce68db 100644 (file)
@@ -27,20 +27,20 @@ service "ssh" do
   if node[:lsb][:release].to_f >= 14.04
     provider Chef::Provider::Service::Upstart
   end
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
 hosts = search(:node, "networking:interfaces").sort_by { |n| n[:hostname] }.collect do |node|
-  names = [ node[:hostname] ]
+  names = [node[:hostname]]
 
   node.interfaces(:role => :external).each do |interface|
-    names |= [ "#{node[:hostname]}.openstreetmap.org" ]
-    names |= [ "#{node[:hostname]}.#{interface[:zone]}.openstreetmap.org" ]
+    names |= ["#{node[:hostname]}.openstreetmap.org"]
+    names |= ["#{node[:hostname]}.#{interface[:zone]}.openstreetmap.org"]
   end
 
   unless node.interfaces(:role => :internal).empty?
-    names |= [ "#{node[:hostname]}.#{node[:networking][:roles][:external][:zone]}.openstreetmap.org" ]
+    names |= ["#{node[:hostname]}.#{node[:networking][:roles][:external][:zone]}.openstreetmap.org"]
   end
 
   keys = {
index 3a0bccc..9b01bfd 100644 (file)
@@ -20,7 +20,7 @@
 package "openvpn"
 
 service "openvpn" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
   ignore_failure true
 end
index 455a1ca..5776fea 100644 (file)
@@ -130,5 +130,5 @@ end
 
 apache_site "owl.openstreetmap.org" do
   template "apache.erb"
-  variables :aliases => [ "owl.osm.org" ]
+  variables :aliases => ["owl.osm.org"]
 end
index a67ac6a..e58c16d 100644 (file)
@@ -1,4 +1,4 @@
-default[:apt][:sources] = node[:apt][:sources] |  [ "brightbox-ruby-ng", "passenger" ]
+default[:apt][:sources] = node[:apt][:sources] |  ["brightbox-ruby-ng", "passenger"]
 
 default[:passenger][:ruby_version] = "1.9.1"
 default[:passenger][:max_pool_size] = 6
index 42a6dbe..1ae670b 100644 (file)
@@ -165,7 +165,7 @@ directory "/var/log/replication" do
   mode 0755
 end
 
-[ "streaming-replicator", "streaming-server" ].each do |name|
+["streaming-replicator", "streaming-server"].each do |name|
   template "/etc/init.d/#{name}" do
     source "streaming.init.erb"
     owner "root"
@@ -175,7 +175,7 @@ end
   end
 
   service name do
-    action [ :enable, :start ]
+    action [:enable, :start]
     supports :restart => true, :status => true
     subscribes :restart, "template[/etc/init.d/#{name}]"
   end
index a8b37c5..9fe6434 100644 (file)
@@ -20,7 +20,7 @@
 package "postgresql-common"
 
 service "postgresql" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
index 55bd9a7..0cba387 100644 (file)
@@ -1 +1 @@
-default[:rsyncd][:modules] = [ ]
+default[:rsyncd][:modules] = []
index fc0c4a5..6d4d6b7 100644 (file)
@@ -43,7 +43,7 @@ end
 package "rsync"
 
 service "rsync" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true
 end
 
index 55bd9a7..0cba387 100644 (file)
@@ -1 +1 @@
-default[:rsyncd][:modules] = [ ]
+default[:rsyncd][:modules] = []
index ef6094b..ecc8170 100644 (file)
@@ -24,7 +24,7 @@ communities = data_bag_item("snmpd", "communities")
 package "snmpd"
 
 service "snmpd" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true
 end
 
index 6c64083..2c7e49d 100644 (file)
@@ -20,7 +20,7 @@
 package "spamassassin"
 
 service "spamassassin" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
@@ -46,7 +46,7 @@ if node[:exim][:smarthost_name]
   end
 end
 
-trusted_networks = trusted_networks - [ "127.0.0.1", "::1" ]
+trusted_networks = trusted_networks - ["127.0.0.1", "::1"]
 
 template "/etc/spamassassin/local.cf" do
   source "local.cf.erb"
index f64873d..32426df 100644 (file)
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-define :squid_fragment, :action => [ :create ], :variables => {} do
+define :squid_fragment, :action => [:create], :variables => {} do
   name = params[:name]
   site_action = params[:action]
 
index 4855a19..aedc892 100644 (file)
@@ -42,7 +42,7 @@ end
 
 service "squid" do
   provider Chef::Provider::Service::Upstart
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
   subscribes :reload, "template[/etc/squid/squid.conf]"
   subscribes :restart, "template[/etc/default/squid]"
index ee4b933..c2e62d2 100644 (file)
@@ -29,7 +29,7 @@ cookbook_file "/etc/ssl/certs/rapidssl.pem" do
   backup false
 end
 
-[ "openstreetmap", "tile.openstreetmap", "crm.osmfoundation" ].each do |certificate|
+["openstreetmap", "tile.openstreetmap", "crm.osmfoundation"].each do |certificate|
   if node[:ssl][:certificates].include?(certificate)
     cookbook_file "/etc/ssl/certs/#{certificate}.pem" do
       owner "root"
index c8dc4aa..24d860f 100644 (file)
@@ -131,7 +131,7 @@ template "/etc/init.d/supybot" do
 end
 
 service "supybot" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :restart => true
   subscribes :restart, "template[/etc/supybot/supybot.conf]"
   subscribes :restart, "template[/etc/supybot/channels.conf]"
index 675f8e7..024075e 100644 (file)
@@ -38,7 +38,7 @@ end
 tilecaches = search(:node, "roles:tilecache").sort_by { |n| n[:hostname] }
 
 apache_site "default" do
-  action [ :disable ]
+  action [:disable]
 end
 
 apache_site "tile.openstreetmap.org" do
@@ -62,7 +62,7 @@ end
 package "renderd"
 
 service "renderd" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => false, :restart => true, :reload => false
 end
 
@@ -451,7 +451,7 @@ template "/etc/init.d/replicate" do
 end
 
 service "replicate" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :restart => true
   subscribes :restart, "template[/usr/local/bin/replicate]"
   subscribes :restart, "template[/etc/init.d/replicate]"
index 1aad4b9..ce4819a 100644 (file)
@@ -20,7 +20,7 @@
 require "ipaddr"
 
 certificate = node[:tilecache][:ssl][:certificate]
-node.default[:ssl][:certificates] = node[:ssl][:certificates] | [ certificate ]
+node.default[:ssl][:certificates] = node[:ssl][:certificates] | [certificate]
 
 include_recipe "ssl"
 include_recipe "squid"
@@ -80,7 +80,7 @@ template "/etc/logrotate.d/squid" do
 end
 
 nginx_site "default" do
-  action [ :delete ]
+  action [:delete]
 end
 
 resolvers = node[:networking][:nameservers].map do |resolver|
index 8424642..31770b5 100644 (file)
@@ -41,7 +41,7 @@ end
 package "rsyslog"
 
 service "rsyslog" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :restart => true, :reload => true
 end
 
index 9fa88c5..a072a88 100644 (file)
@@ -19,7 +19,7 @@
 
 require "yaml"
 
-define :rails_port, :action => [ :create, :enable ] do
+define :rails_port, :action => [:create, :enable] do
   name = params[:name]
   ruby_version = params[:ruby] || "1.9.1"
   rails_directory = params[:directory] || "/srv/#{name}"
index df22c12..6f86f20 100644 (file)
@@ -30,7 +30,7 @@ apache_module "proxy"
 apache_module "proxy_fcgi"
 
 apache_site "default" do
-  action [ :disable ]
+  action [:disable]
 end
 
 apache_site "www.openstreetmap.org" do
index 09a3f78..891609f 100644 (file)
@@ -114,7 +114,7 @@ if %w(database_offline api_offline).include?(node[:web][:status])
   end
 else
   service "cgimap" do
-    action [ :enable, :start ]
+    action [:enable, :start]
     supports :restart => true, :reload => true
     subscribes :restart, "execute[cgimap-build]"
     subscribes :restart, "file[/etc/init.d/cgimap]"
index e1adc5d..f0508f8 100644 (file)
@@ -33,7 +33,7 @@ apache_module "lbmethod_bybusyness"
 apache_module "rewrite"
 
 apache_site "default" do
-  action [ :disable ]
+  action [:disable]
 end
 
 apache_site "www.openstreetmap.org" do
index 10463da..3802cfd 100644 (file)
@@ -73,7 +73,7 @@ if %w(database_offline database_readonly gpx_offline).include?(node[:web][:statu
   end
 else
   service "gpx-import" do
-    action [ :enable, :start ]
+    action [:enable, :start]
     supports :restart => true, :reload => true
     subscribes :restart, "execute[gpx-import-build]"
     subscribes :restart, "template[/etc/init.d/gpx-import]"
index 85bb746..23b88ae 100644 (file)
@@ -24,14 +24,14 @@ include_recipe "mediawiki"
 passwords = data_bag_item("wiki", "passwords")
 
 apache_site "default" do
-  action [ :disable ]
+  action [:disable]
 end
 
-mediawiki_site  "wiki.openstreetmap.org" do
-  aliases       "wiki.osm.org", "wiki.openstreetmap.com", "wiki.openstreetmap.net", "wiki.openstreetmap.ca", "wiki.openstreetmap.eu", "wiki.openstreetmap.pro", "wiki.openstreetmaps.org"
-  directory     "/srv/wiki.openstreetmap.org"
+mediawiki_site "wiki.openstreetmap.org" do
+  aliases "wiki.osm.org", "wiki.openstreetmap.com", "wiki.openstreetmap.net", "wiki.openstreetmap.ca", "wiki.openstreetmap.eu", "wiki.openstreetmap.pro", "wiki.openstreetmaps.org"
+  directory "/srv/wiki.openstreetmap.org"
 
-  enable_ssl    TRUE
+  enable_ssl TRUE
 
   database_name "wiki"
   database_username "wiki-user"
@@ -39,10 +39,10 @@ mediawiki_site  "wiki.openstreetmap.org" do
 
   admin_password passwords["admin"]
 
-  logo          "/osm_logo_wiki.png"
+  logo "/osm_logo_wiki.png"
 
   email_contact "webmaster@openstreetmap.org"
-  email_sender  "wiki@openstreetmap.org"
+  email_sender "wiki@openstreetmap.org"
   email_sender_name "OpenStreetMap Wiki"
 
   metanamespace "OpenStreetMap"
@@ -79,7 +79,7 @@ directory "/srv/dump.wiki.openstreetmap.org" do
 end
 
 apache_site "dump.wiki.openstreetmap.org" do
-  template  "apache_wiki_dump.erb"
+  template "apache_wiki_dump.erb"
   directory "/srv/dump.wiki.openstreetmap.org"
   variables :aliases => "dump.wiki.osm.org"
 end
index abc20c3..b165eb2 100644 (file)
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-define :wordpress_plugin, :action => [ :enable ] do
+define :wordpress_plugin, :action => [:enable] do
   name = params[:name]
   site = params[:site]
   site_directory = node[:wordpress][:sites][site][:directory]
index 7e8e161..08aac28 100644 (file)
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-define :wordpress_site, :action => [ :create, :enable ] do
+define :wordpress_site, :action => [:create, :enable] do
   name = params[:name]
   ssl_enabled = params[:ssl_enabled] || false
   ssl_certificate = params[:ssl_certificate]
index efdcbe9..91ca9de 100644 (file)
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-define :wordpress_theme, :action => [ :enable ] do
+define :wordpress_theme, :action => [:enable] do
   name = params[:name]
   site = params[:site]
   site_directory = node[:wordpress][:sites][site][:directory]
index 0020288..cfcff80 100644 (file)
@@ -20,6 +20,6 @@
 package "xinetd"
 
 service "xinetd" do
-  action [ :enable, :start ]
+  action [:enable, :start]
   supports :status => true, :reload => true, :restart => true
 end
index b38d5ce..529d23b 100644 (file)
@@ -11,14 +11,14 @@ default_attributes(
     }
   },
   :apt => {
-    :sources => [ "openstreetmap" ]
+    :sources => ["openstreetmap"]
   },
   :networking => {
     :roles => {
       :internal => { :metric => 200, :zone => "loc" },
       :external => { :metric => 100 }
     },
-    :search => [ "openstreetmap.org" ]
+    :search => ["openstreetmap.org"]
   },
   :sysctl => {
     :panic => {
index cf8e1f6..1c81d09 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at Blix NL"
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.nl.pool.ntp.org", "1.nl.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.nl.pool.ntp.org", "1.nl.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index 02eab6d..ad67234 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at Blix NO"
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.no.pool.ntp.org", "1.no.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.no.pool.ntp.org", "1.no.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index 756e3c1..e91f541 100644 (file)
@@ -8,7 +8,7 @@ default_attributes(
     }
   },
   :networking => {
-    :nameservers => [ "8.8.8.8", "8.8.4.4" ],
+    :nameservers => ["8.8.8.8", "8.8.4.4"],
     :roles => {
       :external => {
         :zone => "bx"
index 34d801d..5e4f446 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at Bluehost"
 
 default_attributes(
   :networking => {
-    :nameservers => [ "8.8.8.8", "8.8.4.4" ],
+    :nameservers => ["8.8.8.8", "8.8.4.4"],
     :roles => {
       :external => {
         :zone => "bh"
@@ -14,7 +14,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "ntp.bluehost.com" ]
+    :servers => ["ntp.bluehost.com"]
   }
 )
 
index 8c65a4b..96dab6a 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at Bytemark"
 
 default_attributes(
   :networking => {
-    :nameservers => [ "8.8.8.8", "8.8.4.4", "2001:4860:4860::8888", "2001:4860:4860::8844" ],
+    :nameservers => ["8.8.8.8", "8.8.4.4", "2001:4860:4860::8888", "2001:4860:4860::8844"],
     :roles => {
       :external => {
         :zone => "bm"
@@ -14,7 +14,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.uk.pool.ntp.org", "1.uk.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.uk.pool.ntp.org", "1.uk.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index 576f33d..8540943 100644 (file)
@@ -9,7 +9,7 @@ default_attributes(
     }
   },
   :munin => {
-    :allow => [ "193.198.233.210" ]
+    :allow => ["193.198.233.210"]
   },
   :networking => {
     :nameservers => [
@@ -28,7 +28,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.hr.pool.ntp.org", "1.hr.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.hr.pool.ntp.org", "1.hr.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index ee6a58f..ec47152 100644 (file)
@@ -3,20 +3,20 @@ description "Role applied to CRM server"
 
 default_attributes(
   :exim => {
-    :local_domains => [ "crm.osmfoundation.org" ],
+    :local_domains => ["crm.osmfoundation.org"],
     :routes => {
       :crm_return => {
         :comment => "return@crm.osmfoundation.org",
-        :domains => [ "crm.osmfoundation.org" ],
-        :local_parts => [ "return" ],
+        :domains => ["crm.osmfoundation.org"],
+        :local_parts => ["return"],
         :maildir => "/var/mail/crm-return",
         :user => "www-data",
         :group => "mail"
       },
       :crm_mail => {
         :comment => "mail@crm.osmfoundation.org",
-        :domains => [ "crm.osmfoundation.org" ],
-        :local_parts => [ "mail" ],
+        :domains => ["crm.osmfoundation.org"],
+        :local_parts => ["mail"],
         :maildir => "/var/mail/crm-mail",
         :user => "www-data",
         :group => "mail"
index db2f4f6..1a23a91 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all the master database server"
 
 default_attributes(
   :postgresql => {
-    :versions => [ "9.1" ],
+    :versions => ["9.1"],
     :settings => {
       :defaults => {
         :wal_level => "hot_standby",
index d5db9a2..6fcb9bd 100644 (file)
@@ -6,7 +6,7 @@ default_attributes(
     :users => {
       :rails => {
         :status => :role,
-        :members => [ :tomh, :grant ]
+        :members => [:tomh, :grant]
       }
     }
   },
@@ -34,7 +34,7 @@ default_attributes(
     "/store/rails" => { :host => "ironbelly", :path => "/store/rails" }
   },
   :postgresql => {
-    :versions => [ "9.1" ],
+    :versions => ["9.1"],
     :settings => {
       :defaults => {
         :listen_addresses => "*",
index 84c3600..0005000 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at Delta Telecom"
 
 default_attributes(
   :networking => {
-    :nameservers => [ "94.20.20.20", "8.8.8.8", "8.8.4.4" ],
+    :nameservers => ["94.20.20.20", "8.8.8.8", "8.8.4.4"],
     :roles => {
       :external => {
         :zone => "dt"
@@ -14,7 +14,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.az.pool.ntp.org", "1.az.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.az.pool.ntp.org", "1.az.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index 84ca09b..75f3e60 100644 (file)
@@ -68,19 +68,19 @@ default_attributes(
       :mhohmann => { :status => :user },
       :ooc => {
         :status => :role,
-        :members => [ :tomh, :blackadder, :timsc, :ollie ]
+        :members => [:tomh, :blackadder, :timsc, :ollie]
       },
       :apis => {
         :status => :role,
-        :members => [ :tomh ]
+        :members => [:tomh]
       },
       :os => {
         :status => :role,
-        :members => [ :tomh, :grant, :ollie ]
+        :members => [:tomh, :grant, :ollie]
       },
       :gpsmid => {
         :status => :role,
-        :members => [ :apmon, :maba ]
+        :members => [:apmon, :maba]
       }
     }
   },
@@ -99,7 +99,7 @@ default_attributes(
       :master => {
         :repository => "git://git.openstreetmap.org/rails.git",
         :revision => "master",
-        :aliases => [ "api06.dev.openstreetmap.org" ]
+        :aliases => ["api06.dev.openstreetmap.org"]
       },
       :routing => {
         :repository => "git://github.com/apmon/openstreetmap-website.git",
@@ -132,7 +132,7 @@ default_attributes(
     }
   },
   :postgresql => {
-    :versions => [ "9.1", "9.3" ],
+    :versions => ["9.1", "9.3"],
     :settings => {
       :defaults => {
         :shared_buffers => "1GB",
index e3731f1..5722f8d 100644 (file)
@@ -16,7 +16,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.de.pool.ntp.org", "1.de.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.de.pool.ntp.org", "1.de.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index 4ab350d..0b02de1 100644 (file)
@@ -29,12 +29,12 @@ default_attributes(
         :uid => "nobody",
         :gid => "nogroup",
         :transfer_logging => false,
-        :exclude => [ ".*" ],
+        :exclude => [".*"],
         :max_connections => 10,
         :ignore_errors => true,
         :ignore_nonreadable => true,
         :timeout => 3600,
-        :refuse_options => [ "checksum" ]
+        :refuse_options => ["checksum"]
       },
       :agri_extra => {
         :comment => "AGRI Extras Archive",
@@ -45,12 +45,12 @@ default_attributes(
         :uid => "nobody",
         :gid => "nogroup",
         :transfer_logging => false,
-        :exclude => [ ".*" ],
+        :exclude => [".*"],
         :max_connections => 10,
         :ignore_errors => true,
         :ignore_nonreadable => true,
         :timeout => 3600,
-        :refuse_options => [ "checksum" ]
+        :refuse_options => ["checksum"]
       }
     }
   }
index c1ca2e9..58f78d3 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at Firefishy"
 
 default_attributes(
   :networking => {
-    :nameservers => [ "8.8.8.8", "8.8.4.4" ],
+    :nameservers => ["8.8.8.8", "8.8.4.4"],
     :roles => {
       :internal => {
         :inet => {
index 3790013..c032d6b 100644 (file)
@@ -9,7 +9,7 @@ default_attributes(
       },
       :forum => {
         :status => :role,
-        :members => [ :lambertus, :grant ]
+        :members => [:lambertus, :grant]
       }
     }
   },
index 0dfab7c..fa96196 100644 (file)
@@ -8,7 +8,7 @@ default_attributes(
     :keepalive => false
   },
   :apt => {
-    :sources => [ "passenger" ]
+    :sources => ["passenger"]
   },
   :memcached => {
     :memory_limit => 400,
index 653d686..b9007b8 100644 (file)
@@ -9,7 +9,7 @@ default_attributes(
     }
   },
   :exim => {
-    :relay_from_hosts => [ "10.0.0.0/8"]
+    :relay_from_hosts => ["10.0.0.0/8"]
   }
 )
 
index 5667094..534ad10 100644 (file)
@@ -18,7 +18,7 @@ default_attributes(
       },
       :git => {
         :status => :role,
-        :members => [ :tomh, :grant, :matt, :lonvia, :yellowbkpk, :translatewiki ]
+        :members => [:tomh, :grant, :matt, :lonvia, :yellowbkpk, :translatewiki]
       }
     }
   },
index 2e26ed8..3209148 100644 (file)
@@ -7,7 +7,7 @@ default_attributes(
       :enf => { :status => :administrator },
       :gpstile => {
         :status => :role,
-        :members => [ :enf, :tomh ]
+        :members => [:enf, :tomh]
       }
     }
   },
index 948001a..8a9e0af 100644 (file)
@@ -21,7 +21,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.de.pool.ntp.org", "1.de.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.de.pool.ntp.org", "1.de.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index 214e0b3..2e63557 100644 (file)
@@ -8,7 +8,7 @@ default_attributes(
     }
   },
   :networking => {
-    :nameservers => [ "146.179.159.177" ],
+    :nameservers => ["146.179.159.177"],
     :roles => {
       :internal => {
         :inet => {
@@ -33,10 +33,10 @@ default_attributes(
 
 override_attributes(
   :networking => {
-    :search => [ "ic.openstreetmap.org", "openstreetmap.org" ]
+    :search => ["ic.openstreetmap.org", "openstreetmap.org"]
   },
   :ntp => {
-    :servers => [ "0.uk.pool.ntp.org", "1.uk.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.uk.pool.ntp.org", "1.uk.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index 2104788..4bcb8fb 100644 (file)
@@ -3,7 +3,7 @@ description "Master role applied to ironbelly"
 
 default_attributes(
   :apt => {
-    :sources => [ "ubuntugis-unstable" ]
+    :sources => ["ubuntugis-unstable"]
   },
   :git => {
     :allowed_nodes => "*:*",
index ec967bb..4f8a3b5 100644 (file)
@@ -19,7 +19,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.uk.pool.ntp.org", "1.uk.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.uk.pool.ntp.org", "1.uk.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index f7bb27a..c19b033 100644 (file)
@@ -8,7 +8,7 @@ default_attributes(
     }
   },
   :networking => {
-    :nameservers => [ "77.95.64.205", "77.95.64.206", "8.8.8.8", "8.8.4.4" ],
+    :nameservers => ["77.95.64.205", "77.95.64.206", "8.8.8.8", "8.8.4.4"],
     :roles => {
       :external => {
         :zone => "ly"
@@ -16,7 +16,7 @@ default_attributes(
     }
   },
   :snmpd => {
-    :clients => [ "77.95.64.0/24", "77.95.70.0/24" ],
+    :clients => ["77.95.64.0/24", "77.95.70.0/24"],
     :community => "lyonix",
     :location => "LYON",
     :contact => "noc@lyonix.net"
@@ -25,7 +25,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.fr.pool.ntp.org", "1.fr.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.fr.pool.ntp.org", "1.fr.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index 79c65d0..280ad06 100644 (file)
@@ -10,29 +10,29 @@ default_attributes(
       "openstreetmap.co.uk",
       "openstreetmap.org.uk"
     ],
-    :daemon_smtp_ports => [ 25, 26 ],
+    :daemon_smtp_ports => [25, 26],
     :smarthost_name => "mail.openstreetmap.org",
     :smarthost_via => false,
-    :dns_blacklists => [ "sbl-xbl.spamhaus.org" ],
+    :dns_blacklists => ["sbl-xbl.spamhaus.org"],
     :routes => {
       :messages => {
         :comment => "messages.openstreetmap.org",
-        :domains => [ "messages.openstreetmap.org" ],
-        :host => [ "spike-01.openstreetmap.org", "spike-02.openstreetmap.org", "spike-03.openstreetmap.org" ]
+        :domains => ["messages.openstreetmap.org"],
+        :host => ["spike-01.openstreetmap.org", "spike-02.openstreetmap.org", "spike-03.openstreetmap.org"]
       },
       :otrs => {
         :comment => "otrs.openstreetmap.org",
-        :domains => [ "otrs.openstreetmap.org" ],
+        :domains => ["otrs.openstreetmap.org"],
         :host => "ridley.ucl.openstreetmap.org"
       },
       :tickets => {
         :comment => "tickets.openstreetmap.org",
-        :domains => [ "tickets.openstreetmap.org" ],
+        :domains => ["tickets.openstreetmap.org"],
         :host => "ridley.ucl.openstreetmap.org"
       },
       :crm => {
         :comment => "crm.osmfoundation.org",
-        :domains => [ "crm.osmfoundation.org" ],
+        :domains => ["crm.osmfoundation.org"],
         :host => "ridley.ucl.openstreetmap.org"
       }
     },
index 8291d68..1e4cd5f 100644 (file)
@@ -9,7 +9,7 @@ default_attributes(
     }
   },
   :networking => {
-    :nameservers => [ "8.8.8.8", "8.8.4.4" ],
+    :nameservers => ["8.8.8.8", "8.8.4.4"],
     :roles => {
       :external => {
         :zone => "nc"
@@ -20,7 +20,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.tw.pool.ntp.org", "1.tw.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.tw.pool.ntp.org", "1.tw.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index dff1acc..bf509dd 100644 (file)
@@ -8,7 +8,7 @@ default_attributes(
       :twain => { :status => :administrator },
       :nominatim => {
         :status => :role,
-        :members => [ :lonvia, :tomh, :twain ]
+        :members => [:lonvia, :tomh, :twain]
       }
     }
   },
index e1b1e55..3be0a2e 100644 (file)
@@ -43,7 +43,7 @@ default_attributes(
     }
   },
   :postgresql => {
-    :versions => [ "9.1" ],
+    :versions => ["9.1"],
     :settings => {
       :defaults => {
         :shared_buffers => "8GB",
@@ -76,7 +76,7 @@ default_attributes(
 
 override_attributes(
   :networking => {
-    :nameservers => [ "8.8.8.8", "8.8.4.4" ]
+    :nameservers => ["8.8.8.8", "8.8.4.4"]
   }
 )
 
index e6aa963..e68f0c5 100644 (file)
@@ -8,7 +8,7 @@ default_attributes(
     }
   },
   :networking => {
-    :nameservers => [ "8.8.8.8", "8.8.4.4" ],
+    :nameservers => ["8.8.8.8", "8.8.4.4"],
     :roles => {
       :external => {
         :zone => "ool",
@@ -23,7 +23,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "ntp.osuosl.org" ]
+    :servers => ["ntp.osuosl.org"]
   }
 )
 
index 2ace4f9..a3d622e 100644 (file)
@@ -8,12 +8,12 @@ default_attributes(
     }
   },
   :exim => {
-    :local_domains => [ "otrs.openstreetmap.org" ],
+    :local_domains => ["otrs.openstreetmap.org"],
     :routes => {
       :otrs_otrs => {
         :comment => "otrs@otrs.openstreetmap.org",
-        :domains => [ "otrs.openstreetmap.org" ],
-        :local_parts => [ "otrs" ],
+        :domains => ["otrs.openstreetmap.org"],
+        :local_parts => ["otrs"],
         :command => "/opt/otrs/bin/otrs.PostMaster.pl",
         :user => "otrs",
         :group => "www-data",
@@ -21,8 +21,8 @@ default_attributes(
       },
       :otrs_data => {
         :comment => "data@otrs.openstreetmap.org",
-        :domains => [ "otrs.openstreetmap.org" ],
-        :local_parts => [ "data" ],
+        :domains => ["otrs.openstreetmap.org"],
+        :local_parts => ["data"],
         :command => "/opt/otrs/bin/otrs.PostMaster.pl -q 'Data Working Group'",
         :user => "otrs",
         :group => "www-data",
@@ -30,8 +30,8 @@ default_attributes(
       },
       :otrs_support => {
         :comment => "support@otrs.openstreetmap.org",
-        :domains => [ "otrs.openstreetmap.org" ],
-        :local_parts => [ "support" ],
+        :domains => ["otrs.openstreetmap.org"],
+        :local_parts => ["support"],
         :command => "/opt/otrs/bin/otrs.PostMaster.pl -q 'Technical Support'",
         :user => "otrs",
         :group => "www-data",
index c2b90c1..4957435 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at OVH CA"
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.ca.pool.ntp.org", "1.ca.pool.ntp.org", "north-america.pool.ntp.org" ]
+    :servers => ["0.ca.pool.ntp.org", "1.ca.pool.ntp.org", "north-america.pool.ntp.org"]
   }
 )
 
index dba20da..0cf210d 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at OVH"
 
 default_attributes(
   :networking => {
-    :nameservers => [ "8.8.4.4", "213.186.33.99", "8.8.8.8" ],
+    :nameservers => ["8.8.4.4", "213.186.33.99", "8.8.8.8"],
     :roles => {
       :external => {
         :zone => "ov"
index 74bddb6..44f44c6 100644 (file)
@@ -8,12 +8,12 @@ default_attributes(
       :ppawel => { :status => :user },
       :owl => {
         :status => :role,
-        :members => [ :yellowbkpk, :ppawel ]
+        :members => [:yellowbkpk, :ppawel]
       }
     },
     :groups => {
       :adm => {
-        :members => [ :yellowbkpk, :ppawel ]
+        :members => [:yellowbkpk, :ppawel]
       }
     }
   },
@@ -21,10 +21,10 @@ default_attributes(
     :mpm => "event"
   },
   :apt => {
-    :sources => [ "brightbox-ruby-ng", "ubuntugis-stable", "ubuntugis-unstable" ]
+    :sources => ["brightbox-ruby-ng", "ubuntugis-stable", "ubuntugis-unstable"]
   },
   :postgresql => {
-    :versions => [ "9.1" ],
+    :versions => ["9.1"],
     :settings => {
       :defaults => {
         :fsync => "off",
index 16da108..cc128c4 100644 (file)
@@ -9,10 +9,10 @@ default_attributes(
     }
   },
   :munin => {
-    :allow => [ "10.64.1.11" ]
+    :allow => ["10.64.1.11"]
   },
   :networking => {
-    :nameservers => [ "10.64.1.42", "194.167.156.13", "10.64.1.3" ],
+    :nameservers => ["10.64.1.42", "194.167.156.13", "10.64.1.3"],
     :roles => {
       :external => {
         :zone => "pa"
@@ -23,7 +23,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "cannelle.paulla.asso.fr" ]
+    :servers => ["cannelle.paulla.asso.fr"]
   }
 )
 
index 791c2e8..d68bdab 100644 (file)
@@ -7,7 +7,7 @@ default_attributes(
       :bretth => { :status => :user },
       :planet => {
         :status => :administrator,
-        :members => [ :bretth ]
+        :members => [:bretth]
       }
     }
   },
@@ -22,12 +22,12 @@ default_attributes(
         :uid => "nobody",
         :gid => "nogroup",
         :transfer_logging => false,
-        :exclude => [ ".*" ],
+        :exclude => [".*"],
         :max_connections => 10,
         :ignore_errors => true,
         :ignore_nonreadable => true,
         :timeout => 3600,
-        :refuse_options => [ "checksum" ]
+        :refuse_options => ["checksum"]
       }
     }
   },
index 127a526..0f5b6c8 100644 (file)
@@ -51,7 +51,7 @@ default_attributes(
     }
   },
   :postgresql => {
-    :versions => [ "9.3" ],
+    :versions => ["9.3"],
     :settings => {
       :defaults => {
         :shared_buffers => "10GB",
index 96319e0..7803ff3 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at prgmr.com"
 
 default_attributes(
   :networking => {
-    :nameservers => [ "8.8.4.4", "65.19.174.2", "65.19.175.2" ],
+    :nameservers => ["8.8.4.4", "65.19.174.2", "65.19.175.2"],
     :roles => {
       :external => {
         :zone => "pr"
@@ -14,7 +14,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.us.pool.ntp.org", "1.us.pool.ntp.org", "2.us.pool.ntp.org" ]
+    :servers => ["0.us.pool.ntp.org", "1.us.pool.ntp.org", "2.us.pool.ntp.org"]
   }
 )
 
index 559cb61..cae7b65 100644 (file)
@@ -19,7 +19,7 @@ default_attributes(
     }
   },
   :postgresql => {
-    :versions => [ "9.3" ],
+    :versions => ["9.3"],
     :settings => {
       :defaults => {
         :shared_buffers => "10GB",
index 9a1093f..f4f45c0 100644 (file)
@@ -19,7 +19,7 @@ default_attributes(
         :category => "Ups",
         :values => {
           :load => {
-            :sum => [ "apcpdu_apc1.load", "apcpdu_apc2.load", "apcpdu_apc3.load" ],
+            :sum => ["apcpdu_apc1.load", "apcpdu_apc2.load", "apcpdu_apc3.load"],
             :label => "Load"
           }
         }
index 2554b86..87daf4d 100644 (file)
@@ -8,7 +8,7 @@ default_attributes(
     }
   },
   :apt => {
-    :sources => [ "brightbox-ruby-ng" ]
+    :sources => ["brightbox-ruby-ng"]
   },
   :db => {
     :cluster => "9.1/main"
index ad8537c..4b0fb23 100644 (file)
@@ -9,7 +9,7 @@ default_attributes(
       },
       :taginfo => {
         :status => :role,
-        :members => [ :jochen, :tomh ]
+        :members => [:jochen, :tomh]
       }
     }
   },
index 49f98d4..c1e09ae 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at Teleservice"
 
 default_attributes(
   :networking => {
-    :nameservers => [ "8.8.8.8", "8.8.4.4" ],
+    :nameservers => ["8.8.8.8", "8.8.4.4"],
     :roles => {
       :external => {
         :zone => "ts"
@@ -14,7 +14,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.se.pool.ntp.org", "1.se.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.se.pool.ntp.org", "1.se.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index 66011b6..3524d1a 100644 (file)
@@ -6,7 +6,7 @@ default_attributes(
     :users => {
       :tile => {
         :status => :role,
-        :members => [ :jburgess, :tomh ]
+        :members => [:jburgess, :tomh]
       }
     }
   },
@@ -23,10 +23,10 @@ default_attributes(
     }
   },
   :apt => {
-    :sources => [ "ubuntugis-stable" ]
+    :sources => ["ubuntugis-stable"]
   },
   :postgresql => {
-    :versions => [ "9.1" ],
+    :versions => ["9.1"],
     :settings => {
       :defaults => {
         :temp_buffers => "32MB",
index db78d4d..4f521d0 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all tile cache servers"
 
 default_attributes(
   :apt => {
-    :sources => [ "nginx" ]
+    :sources => ["nginx"]
   },
   :sysctl => {
     :network_conntrack_time_wait => {
index c611c20..bba547e 100644 (file)
@@ -31,8 +31,8 @@ default_attributes(
 
 override_attributes(
   :networking => {
-    :nameservers => [ "10.0.0.3", "8.8.8.8", "8.8.4.4" ],
-    :search => [ "ucl.openstreetmap.org", "openstreetmap.org" ]
+    :nameservers => ["10.0.0.3", "8.8.8.8", "8.8.4.4"],
+    :search => ["ucl.openstreetmap.org", "openstreetmap.org"]
   }
 )
 
index 71beb15..208185a 100644 (file)
@@ -23,8 +23,8 @@ default_attributes(
 
 override_attributes(
   :networking => {
-    :nameservers => [ "10.0.0.3", "8.8.8.8", "8.8.4.4" ],
-    :search => [ "ucl.openstreetmap.org", "openstreetmap.org" ]
+    :nameservers => ["10.0.0.3", "8.8.8.8", "8.8.4.4"],
+    :search => ["ucl.openstreetmap.org", "openstreetmap.org"]
   }
 )
 
index cb6bfce..842daad 100644 (file)
@@ -3,13 +3,13 @@ description "Role applied to all servers at UCL"
 
 default_attributes(
   :bind => {
-    :forwarders => [ "144.82.100.1", "144.82.100.41" ]
+    :forwarders => ["144.82.100.1", "144.82.100.41"]
   }
 )
 
 override_attributes(
   :ntp => {
-    :servers => [ "ntp1.ucl.ac.uk", "ntp2.ucl.ac.uk" ]
+    :servers => ["ntp1.ucl.ac.uk", "ntp2.ucl.ac.uk"]
   }
 )
 
index deb62f7..eab62f7 100644 (file)
@@ -21,12 +21,12 @@ default_attributes(
     :rails_hard_memory_limit => 1024
   },
   :exim => {
-    :local_domains => [ "messages.openstreetmap.org" ],
-    :trusted_users => [ "rails" ],
+    :local_domains => ["messages.openstreetmap.org"],
+    :trusted_users => ["rails"],
     :routes => {
       :messages => {
         :comment => "messages.openstreetmap.org",
-        :domains => [ "messages.openstreetmap.org" ],
+        :domains => ["messages.openstreetmap.org"],
         :command => "/srv/www.openstreetmap.org/rails/script/deliver-message $local_part",
         :user => "rails",
         :group => "rails",
index 75bab25..69b6a45 100644 (file)
@@ -6,7 +6,7 @@ default_attributes(
     :users => {
       :rails => {
         :status => :role,
-        :members => [ :tomh, :grant ]
+        :members => [:tomh, :grant]
       }
     }
   },
index 523e73c..4a77532 100644 (file)
@@ -8,7 +8,7 @@ default_attributes(
     }
   },
   :exim => {
-    :trusted_users => [ "www-data" ],
+    :trusted_users => ["www-data"],
     :aliases => {
       :root => "grant"
     },
index fddf3a0..2c9c89b 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to all servers at Yandex"
 
 default_attributes(
   :networking => {
-    :nameservers => [ "8.8.8.8", "8.8.4.4" ],
+    :nameservers => ["8.8.8.8", "8.8.4.4"],
     :roles => {
       :external => {
         :zone => "yx"
@@ -14,7 +14,7 @@ default_attributes(
 
 override_attributes(
   :ntp => {
-    :servers => [ "0.ru.pool.ntp.org", "1.ru.pool.ntp.org", "europe.pool.ntp.org" ]
+    :servers => ["0.ru.pool.ntp.org", "1.ru.pool.ntp.org", "europe.pool.ntp.org"]
   }
 )
 
index 9f1c7e7..39c4013 100644 (file)
@@ -72,7 +72,7 @@ default_attributes(
     }
   },
   :postgresql => {
-    :versions => [ "9.1" ],
+    :versions => ["9.1"],
     :settings => {
       :defaults => {
         :shared_buffers => "3GB",