Fix new rubocop warnings
authorTom Hughes <tom@compton.nu>
Tue, 11 Jul 2017 19:33:05 +0000 (20:33 +0100)
committerTom Hughes <tom@compton.nu>
Tue, 11 Jul 2017 19:33:05 +0000 (20:33 +0100)
48 files changed:
.rubocop.yml
cookbooks/apache/recipes/default.rb
cookbooks/dev/recipes/default.rb
cookbooks/hardware/attributes/default.rb
cookbooks/hardware/recipes/default.rb
cookbooks/imagery/recipes/gb_os_sv.rb
cookbooks/imagery/resources/site.rb
cookbooks/kibana/recipes/default.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/mysql/recipes/default.rb
cookbooks/networking/recipes/default.rb
cookbooks/nominatim/recipes/default.rb
cookbooks/ntp/metadata.rb
cookbooks/ntp/recipes/default.rb
cookbooks/piwik/attributes/default.rb
cookbooks/planet/files/default/replication-bin/replicate-changesets [changed mode: 0644->0755]
cookbooks/planet/recipes/dump.rb
cookbooks/postgresql/providers/munin.rb
cookbooks/squid/recipes/default.rb
cookbooks/ssl/recipes/default.rb
cookbooks/systemd/resources/service.rb
cookbooks/taginfo/recipes/default.rb
cookbooks/tile/recipes/default.rb
cookbooks/web/recipes/cgimap.rb
cookbooks/web/recipes/gpx.rb
cookbooks/web/recipes/rails.rb
hooks/pre-commit
roles/bytemark.rb
roles/dulcy.rb
roles/grisu.rb
roles/ic.rb
roles/katla.rb
roles/ladon.rb
roles/paulla.rb
roles/pummelzacken.rb
roles/spike-04.rb
roles/spike-05.rb
roles/teleservice.rb
roles/thorn-04.rb
roles/thorn-05.rb
roles/tyan-s7010.rb
roles/web.rb

index 23c3f82..0798540 100644 (file)
@@ -1,12 +1,18 @@
 inherit_from: .rubocop_todo.yml
 
-Style/AlignParameters:
+AllCops:
+  TargetRubyVersion: 2.3
+
+Layout/AlignParameters:
   Exclude:
     - '**/metadata.rb'
 
-Style/ExtraSpacing:
+Layout/ExtraSpacing:
   AllowForAlignment: true
 
+Layout/IndentHeredoc:
+  EnforcedStyle: squiggly
+
 Style/FileName:
   Exclude:
     - 'cookbooks/trac/files/default/trac-authenticate'
@@ -15,8 +21,14 @@ Style/FileName:
     - 'hooks/*'
     - 'roles/*.rb'
 
+Style/FrozenStringLiteralComment:
+  EnforcedStyle: never
+
 Style/HashSyntax:
   EnforcedStyle: hash_rockets
 
 Style/StringLiterals:
   EnforcedStyle: double_quotes
+
+Style/SymbolArray:
+  EnforcedStyle: brackets
index 72af914..9b6c6ef 100644 (file)
@@ -22,7 +22,7 @@ include_recipe "ssl"
 package "apache2"
 package "libwww-perl"
 
-%w(event itk prefork worker).each do |mpm|
+%w[event itk prefork worker].each do |mpm|
   if mpm == node[:apache][:mpm]
     apache_module "mpm_#{mpm}" do
       action [:enable]
index 59fd786..80b8058 100644 (file)
@@ -113,7 +113,7 @@ search(:accounts, "*:*").each do |account|
   name = account["id"]
   details = node[:accounts][:users][name] || {}
 
-  next unless %w(user administrator).include?(details[:status])
+  next unless %w[user administrator].include?(details[:status])
 
   user_home = details[:home] || account["home"] || "#{node[:accounts][:home]}/#{name}"
 
index ce7aae8..393dd03 100644 (file)
@@ -1,10 +1,10 @@
 default[:hardware][:modules] = if node[:lsb][:release].to_f >= 16.04
-                                 %w(lp)
+                                 %w[lp]
                                else
-                                 %w(loop lp rtc)
+                                 %w[loop lp rtc]
                                end
 
-default[:hardware][:grub][:cmdline] = %w(nomodeset)
+default[:hardware][:grub][:cmdline] = %w[nomodeset]
 default[:hardware][:sensors] = {}
 
 if node[:dmi] && node[:dmi][:system]
@@ -19,7 +19,7 @@ if node[:dmi] && node[:dmi][:system]
   end
 end
 
-if Chef::Util.compare_versions(node[:kernel][:release], [3, 3]) < 0
+if Chef::Util.compare_versions(node[:kernel][:release], [3, 3]).negative?
   default[:hardware][:modules] |= ["microcode"]
 
   if node[:cpu][:"0"][:vendor_id] == "GenuineIntel"
@@ -28,7 +28,7 @@ if Chef::Util.compare_versions(node[:kernel][:release], [3, 3]) < 0
 end
 
 if node[:kernel] && node[:kernel][:modules]
-  raidmods = node[:kernel][:modules].keys & %w(cciss hpsa mptsas mpt2sas mpt3sas megaraid_mm megaraid_sas aacraid)
+  raidmods = node[:kernel][:modules].keys & %w[cciss hpsa mptsas mpt2sas mpt3sas megaraid_mm megaraid_sas aacraid]
 
   default[:apt][:sources] |= ["hwraid"] unless raidmods.empty?
 end
index 0120827..1d5be55 100644 (file)
@@ -86,7 +86,7 @@ end
 
 # Remove legacy HP G4 support which breaks modern hp-health 10.4
 if manufacturer == "HP"
-  %w(/opt/hp/hp-health/bin/hpasmd /usr/lib/libhpasmintrfc.so.3.0 %/usr/lib/libhpasmintrfc.so.3 /usr/lib/libhpasmintrfc.so).each do |filename|
+  %w[/opt/hp/hp-health/bin/hpasmd /usr/lib/libhpasmintrfc.so.3.0 %/usr/lib/libhpasmintrfc.so.3 /usr/lib/libhpasmintrfc.so].each do |filename|
     file filename do
       action :delete
     end
@@ -246,7 +246,7 @@ node[:block_device].each do |name, attributes|
   end
 end
 
-%w(hpssacli lsiutil sas2ircu megactl megacli arcconf).each do |tools_package|
+%w[hpssacli lsiutil sas2ircu megactl megacli arcconf].each do |tools_package|
   if tools_packages.include?(tools_package)
     package tools_package
   else
@@ -384,7 +384,7 @@ smartd_service = if node[:lsb][:release].to_f >= 16.04
 
 disks = disks.compact
 
-if disks.count > 0
+if disks.count.positive?
   package "smartmontools"
 
   template "/usr/local/bin/smartd-mailer" do
@@ -434,7 +434,7 @@ else
   end
 end
 
-if disks.count > 0
+if disks.count.positive?
   munin_plugin "hddtemp_smartctl" do
     conf "munin.hddtemp.erb"
     conf_variables :disks => disks
index 6dd2e98..4dcf0bd 100644 (file)
@@ -1,4 +1,5 @@
 # coding: utf-8
+
 #
 # Cookbook Name:: imagery
 # Recipe:: gb_os_sv
index 137153a..d6e2071 100644 (file)
@@ -89,9 +89,10 @@ action :create do
   end
 
   base_domains = [name] + Array(aliases)
+  tile_domains = base_domains.flat_map { |d| [d, "a.#{d}", "b.#{d}", "c.#{d}"] }
 
   ssl_certificate new_resource.name do
-    domains base_domains.flat_map { |d| [d, "a.#{d}", "b.#{d}", "c.#{d}"] }
+    domains tile_domains
   end
 
   resolvers = node[:networking][:nameservers].map do |resolver|
index adc3927..9f17df9 100644 (file)
@@ -1,4 +1,5 @@
 # coding: utf-8
+
 #
 # Cookbook Name:: kibana
 # Recipe:: default
index e96af7e..7186626 100644 (file)
@@ -36,7 +36,7 @@ munin_plugin_conf "memcached_multi" do
   template "munin.erb"
 end
 
-%w(bytes commands conns evictions items memory).each do |stat|
+%w[bytes commands conns evictions items memory].each do |stat|
   munin_plugin "memcached_multi_#{stat}" do
     target "memcached_multi_"
   end
index 17bba16..79c3066 100755 (executable)
@@ -12,7 +12,7 @@ def uris_from_status(server)
   rows = (tables[1] / "tr")[1..-1]
   data = rows.collect { |r| (r / "td").collect(&:inner_html) }
   # filter where the PID is numeric, status is 'W' and host matches the server
-  matching_data = data.select { |r| r[1].to_i > 0 && r[3].match(/W/) && r[11].match(server) }
+  matching_data = data.select { |r| r[1].to_i.positive? && r[3].match(/W/) && r[11].match(server) }
   # return only the URI part
   matching_data.collect { |r| r[12] }
 end
index 6be9fc6..f5bbd11 100755 (executable)
@@ -12,7 +12,7 @@ def uri_and_times_from_status(server)
   rows = (tables[1] / "tr")[1..-1]
   data = rows.collect { |r| (r / "td").collect(&:inner_html) }
   # filter where the PID is numeric, status is 'W' and host matches the server
-  matching_data = data.select { |r| r[1].to_i > 0 && r[3].match(/W/) && r[11].match(server) }
+  matching_data = data.select { |r| r[1].to_i.positive? && r[3].match(/W/) && r[11].match(server) }
   # return URI and number of seconds processing for each request
   matching_data.collect { |r| [r[12], r[5].to_i] }
 end
index 1aa9a77..7017c63 100755 (executable)
@@ -6,13 +6,13 @@
 require "English"
 
 def output_config
-  puts <<-END
-graph_args --base 1024 -l 0 --vertical-label bytes --upper-limit 4056231936
-graph_category passenger
-graph_title Passenger memory
+  puts <<~END
+    graph_args --base 1024 -l 0 --vertical-label bytes --upper-limit 4056231936
+    graph_category passenger
+    graph_title Passenger memory
 
-memory.label memory
-END
+    memory.label memory
+  END
   exit 0
 end
 
index 39842a0..b8a06cf 100755 (executable)
@@ -3,18 +3,18 @@
 require "English"
 
 def output_config
-  puts <<-END
-graph_category passenger
-graph_title Passenger processes
-graph_order active inactive
-graph_vlabel processes
-graph_total total
+  puts <<~END
+    graph_category passenger
+    graph_title Passenger processes
+    graph_order active inactive
+    graph_vlabel processes
+    graph_total total
 
-active.label busy servers
-active.draw AREA
-inactive.label idle servers
-inactive.draw STACK
-END
+    active.label busy servers
+    active.draw AREA
+    inactive.label idle servers
+    inactive.draw STACK
+  END
   exit 0
 end
 
index 60be633..8d4159b 100755 (executable)
@@ -3,13 +3,13 @@
 require "English"
 
 def output_config
-  puts <<-END
-graph_category passenger
-graph_title Passenger queues
-graph_vlabel count
+  puts <<~END
+    graph_category passenger
+    graph_title Passenger queues
+    graph_vlabel count
 
-global.label global
-END
+    global.label global
+  END
   exit 0
 end
 
index 1e34fc7..3eeb776 100755 (executable)
@@ -3,17 +3,17 @@
 require "English"
 
 def output_config
-  puts <<-END
-graph_args --base 1000
-graph_category passenger
-graph_title Passenger requests
-graph_vlabel requests / ${graph_period}
+  puts <<~END
+    graph_args --base 1000
+    graph_category passenger
+    graph_title Passenger requests
+    graph_vlabel requests / ${graph_period}
 
-requests.label requests
-requests.type DERIVE
-requests.max 1000000
-requests.min 0
-END
+    requests.label requests
+    requests.type DERIVE
+    requests.max 1000000
+    requests.min 0
+  END
   exit 0
 end
 
index 4270812..9c35682 100644 (file)
@@ -42,7 +42,7 @@ end
 package "libdbd-mysql-perl"
 package "libcache-cache-perl"
 
-%w(
+%w[
   commands connections files handler_read handler_tmp handler_transaction
   handler_write innodb_bpool innodb_bpool_act innodb_history_list_length
   innodb_insert_buf innodb_io innodb_io_pend innodb_log innodb_queries
@@ -50,15 +50,15 @@ package "libcache-cache-perl"
   innodb_tnx max_mem mrr myisam_indexes network_traffic performance
   qcache qcache_mem select_types slow sorts table_definitions table_locks
   tmp_tables
-).each do |stat|
+].each do |stat|
   munin_plugin "mysql_#{stat}" do
     target "mysql_"
   end
 end
 
-%w(
+%w[
   bin_relay_log files_tables replication
-).each do |stat|
+].each do |stat|
   munin_plugin "mysql_#{stat}" do
     action :delete
   end
index 16dd482..697f2c3 100644 (file)
@@ -216,7 +216,7 @@ firewall_rule "limit-icmp-echo" do
   rate_limit "s:1/sec:5"
 end
 
-%w(ucl ic bm aws).each do |zone|
+%w[ucl ic bm aws].each do |zone|
   firewall_rule "accept-openvpn-#{zone}" do
     action :accept
     family :inet
index e930ee4..e6d793a 100644 (file)
@@ -274,7 +274,7 @@ template "/etc/init.d/nominatim-update" do
   variables :source_directory => source_directory
 end
 
-%w(backup-nominatim vacuum-db-nominatim).each do |fname|
+%w[backup-nominatim vacuum-db-nominatim].each do |fname|
   template "/usr/local/bin/#{fname}" do
     source "#{fname}.erb"
     owner "root"
index d1906c3..484d0d2 100644 (file)
@@ -8,7 +8,7 @@ version           "0.8.2"
 
 recipe "ntp", "Installs and configures ntp either as a server or client"
 
-%w(ubuntu debian redhat centos fedora).each do |os|
+%w[ubuntu debian redhat centos fedora].each do |os|
   supports os
 end
 
index 29f79d6..cdf3b5a 100644 (file)
@@ -51,7 +51,7 @@ template "/etc/ntp.conf" do
   notifies :restart, "service[ntp]"
 end
 
-munin_plugins = %w(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"
index ba7ade3..16089ca 100644 (file)
@@ -1,5 +1,5 @@
 default[:piwik][:version] = "3.0.4"
-default[:piwik][:plugins] = %w(
+default[:piwik][:plugins] = %w[
   Actions API BulkTracking Contents CoreAdminHome CoreConsole CoreHome
   CorePluginsAdmin CoreUpdater CoreVisualizations CustomVariables
   Dashboard DevicesDetection DevicePlugins DoNotTrack Events Feedback Goals
@@ -8,4 +8,4 @@ default[:piwik][:plugins] = %w(
   SegmentEditor SEO SitesManager Transitions UserCountry UserCountryMap
   UserLanguage UsersManager Widgetize VisitFrequency VisitorInterest
   VisitsSummary VisitTime
-)
+]
old mode 100644 (file)
new mode 100755 (executable)
index a5312e1..3cf5429
@@ -100,7 +100,7 @@ class ChangesetBuilder
     xml["comments_count"] = res.num_tuples.to_s
 
     # early return if there aren't any comments
-    return unless res.num_tuples > 0
+    return unless res.num_tuples.positive?
 
     discussion = XML::Node.new("discussion")
     res.each do |row|
@@ -167,7 +167,7 @@ class Replicator
     new_ids = @conn
               .exec("select distinct changeset_id from changeset_comments where created_at >= '#{last_run}' and created_at < '#{@now}' and visible")
               .map { |row| row["changeset_id"].to_i }
-              .select { |c_id| !cs_ids.include?(c_id) }
+              .reject { |c_id| cs_ids.include?(c_id) }
 
     new_ids.each do |id|
       @conn
@@ -277,7 +277,6 @@ class Replicator
         move_tmp_files_into_place!
 
         fl.flock(File::LOCK_UN)
-
       rescue
         STDERR.puts("Error! Couldn't update state.")
         fl.flock(File::LOCK_UN)
index 44f1d6e..0eb5cef 100644 (file)
@@ -20,7 +20,7 @@
 node.default[:incron][:planetdump] = {
   :user => "www-data",
   :path => "/store/backup",
-  :events => %w(IN_CREATE IN_MOVED_TO),
+  :events => %w[IN_CREATE IN_MOVED_TO],
   :command => "/usr/local/bin/planetdump $#"
 }
 
index 986c11f..efa0a8b 100644 (file)
@@ -28,7 +28,7 @@ action :create do
   database = new_resource.database
 
   if cluster
-    %w(cache connections locks querylength scans size transactions tuples).each do |plugin|
+    %w[cache connections locks querylength scans size transactions tuples].each do |plugin|
       munin_plugin "postgres_#{plugin}_#{database}:#{suffix}" do
         target "postgres_#{plugin}_"
         conf "munin.erb"
@@ -45,7 +45,7 @@ end
 action :delete do
   database = new_resource.database
 
-  %w(cache connections locks querylength scans size transactions tuples).each do |plugin|
+  %w[cache connections locks querylength scans size transactions tuples].each do |plugin|
     munin_plugin "postgres_#{plugin}_#{database}:#{suffix}" do
       action :delete
       restart_munin false
index 731cbdd..54e6e4a 100644 (file)
@@ -72,7 +72,7 @@ log "squid-restart" do
   notifies :restart, "service[squid]"
   only_if do
     IO.popen(["squidclient", "--host=127.0.0.1", "--port=80", "mgr:counters"]) do |io|
-      io.each.grep(/^[a-z][a-z_.]+ = -[0-9]+$/).count > 0
+      io.each.grep(/^[a-z][a-z_.]+ = -[0-9]+$/).count.positive?
     end
   end
 end
index 328ba8b..536110d 100644 (file)
@@ -20,7 +20,7 @@
 package "openssl"
 package "ssl-cert"
 
-%w(letsencrypt dhparam).each do |certificate|
+%w[letsencrypt dhparam].each do |certificate|
   cookbook_file "/etc/ssl/certs/#{certificate}.pem" do
     owner "root"
     group "root"
index 664304d..e7779fa 100644 (file)
@@ -25,7 +25,7 @@ property :after, [String, Array]
 property :wants, [String, Array]
 property :type, String,
          :default => "simple",
-         :is => %w(simple forking oneshot dbus notify idle)
+         :is => %w[simple forking oneshot dbus notify idle]
 property :limit_nofile, Integer
 property :environment, Hash, :default => {}
 property :environment_file, [String, Hash]
@@ -40,14 +40,14 @@ property :exec_reload, String
 property :runtime_directory, String
 property :runtime_directory_mode, Integer
 property :standard_input, String,
-         :is => %w(null tty tty-force tty-fail socket)
+         :is => %w[null tty tty-force tty-fail socket]
 property :standard_output, String,
-         :is => %w(inherit null tty journal syslog kmsg journal+console syslog+console kmsg+console socket)
+         :is => %w[inherit null tty journal syslog kmsg journal+console syslog+console kmsg+console socket]
 property :standard_error, String,
-         :is => %w(inherit null tty journal syslog kmsg journal+console syslog+console kmsg+console socket)
+         :is => %w[inherit null tty journal syslog kmsg journal+console syslog+console kmsg+console socket]
 property :success_exit_status, [Integer, String, Array]
 property :restart, String,
-         :is => %w(on-success on-failure on-abnormal on-watchdog on-abort always)
+         :is => %w[on-success on-failure on-abnormal on-watchdog on-abort always]
 property :private_tmp, [TrueClass, FalseClass]
 property :private_devices, [TrueClass, FalseClass]
 property :private_network, [TrueClass, FalseClass]
index 57ccb3d..cd3262c 100644 (file)
@@ -45,7 +45,7 @@ ruby_version = node[:passenger][:ruby_version]
 
 package "ruby#{ruby_version}"
 
-%w(json sqlite3 sinatra sinatra-r18n rack-contrib).each do |gem|
+%w[json sqlite3 sinatra sinatra-r18n rack-contrib].each do |gem|
   gem_package gem do
     gem_binary "gem#{ruby_version}"
   end
@@ -181,7 +181,7 @@ node[:taginfo][:sites].each do |site|
     subscribes :run, "git[#{directory}/osmium-tool]"
   end
 
-  %w(taginfo/web/tmp bin data data/old download sources planet planet/log planet/replication).each do |dir|
+  %w[taginfo/web/tmp bin data data/old download sources planet planet/log planet/replication].each do |dir|
     directory "#{directory}/#{dir}" do
       owner "taginfo"
       group "taginfo"
index 679d2a3..6cdf619 100644 (file)
@@ -376,7 +376,7 @@ postgresql_extension "hstore" do
   database "gis"
 end
 
-%w(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"
@@ -385,7 +385,7 @@ end
   end
 end
 
-%w(geometry_columns planet_osm_line planet_osm_point planet_osm_polygon planet_osm_roads).each do |table|
+%w[geometry_columns planet_osm_line planet_osm_point planet_osm_polygon planet_osm_roads].each do |table|
   postgresql_table table do
     cluster node[:tile][:database][:cluster]
     database "gis"
index 0875804..8303955 100644 (file)
@@ -63,7 +63,7 @@ systemd_service "cgimap" do
   pid_file "#{node[:web][:pid_directory]}/cgimap.pid"
 end
 
-if %w(database_offline api_offline).include?(node[:web][:status])
+if %w[database_offline api_offline].include?(node[:web][:status])
   service "cgimap" do
     action :stop
   end
index f74ac50..59d5bc1 100644 (file)
@@ -79,7 +79,7 @@ systemd_service "gpx-import" do
   pid_file "#{pid_directory}/gpx-import.pid"
 end
 
-if %w(database_offline database_readonly gpx_offline).include?(node[:web][:status])
+if %w[database_offline database_readonly gpx_offline].include?(node[:web][:status])
   service "gpx-import" do
     action :stop
   end
index 725f3b7..14eacc5 100644 (file)
@@ -48,7 +48,7 @@ end
 rails_directory = "#{node[:web][:base_directory]}/rails"
 
 piwik_configuration = data_bag_item("web", "piwik").to_hash.reject do |k, _|
-  %w(chef_type data_bag id).include?(k)
+  %w[chef_type data_bag id].include?(k)
 end
 
 rails_port "www.openstreetmap.org" do
index 7c7655e..cb7bc82 100755 (executable)
@@ -3,7 +3,7 @@
 ok = true
 
 if IO.popen(["git", "ls-files", "--unmerged"]).read.empty?
-  need_stash = !IO.popen(%w(git diff)).read.empty?
+  need_stash = !IO.popen(%w[git diff]).read.empty?
 
   system("git", "stash", "save", "--keep-index", "--quiet") if need_stash
 
index 6d539ff..404d847 100644 (file)
@@ -27,7 +27,7 @@ default_attributes(
     }
   },
   :web => {
-    :backends => %w(rails4 rails5),
+    :backends => %w[rails4 rails5],
     :fileserver => "grisu",
     :readonly_database_host => "katla.bm.openstreetmap.org"
   }
index 29cf0bb..355f98c 100644 (file)
@@ -42,7 +42,7 @@ default_attributes(
   :nominatim => {
     :state => "standalone",
     :enable_backup => false,
-    :dbadmins => %w(lonvia tomh),
+    :dbadmins => %w[lonvia tomh],
     :dbcluster => "9.6/main",
     :flatnode_file => "/ssd/nominatim/nodes.store",
     :logdir => "/ssd/nominatim/log",
index 613f9f2..ebc6241 100644 (file)
@@ -10,7 +10,7 @@ default_attributes(
         :family => :inet,
         :address => "10.0.32.20",
         :bond => {
-          :slaves => %w(em1 em2)
+          :slaves => %w[em1 em2]
         }
       },
       :external_ipv4 => {
index 4041d2b..33e9962 100644 (file)
@@ -30,7 +30,7 @@ default_attributes(
     }
   },
   :web => {
-    :backends => %w(rails1 rails2 rails3),
+    :backends => %w[rails1 rails2 rails3],
     :fileserver => "ironbelly",
     :readonly_database_host => "karm.ic.openstreetmap.org"
   }
index 399c767..4753dd8 100644 (file)
@@ -58,7 +58,7 @@ default_attributes(
         :family => :inet,
         :address => "10.0.32.40",
         :bond => {
-          :slaves => %w(eth0 eth1)
+          :slaves => %w[eth0 eth1]
         }
       }
     }
index f70e0af..75da422 100644 (file)
@@ -15,7 +15,7 @@ default_attributes(
           :mode => "802.3ad",
           :miimon => "100",
           :xmithashpolicy => "layer3+4",
-          :slaves => %w(eth0 eth1)
+          :slaves => %w[eth0 eth1]
         }
       },
       :external_ipv6 => {
index db4252d..3d542df 100644 (file)
@@ -1,4 +1,5 @@
 # coding: utf-8
+
 name "paulla"
 description "Role applied to all servers at PauLLA"
 
index f5a573d..a7ccf2e 100644 (file)
@@ -33,7 +33,7 @@ default_attributes(
   },
   :nominatim => {
     :state => "standalone",
-    :dbadmins => %w(lonvia tomh),
+    :dbadmins => %w[lonvia tomh],
     :dbcluster => "9.5/main",
     :flatnode_file => "/ssd/nominatim/nodes.store",
     :tablespaces => {
index 343a5c5..4936947 100644 (file)
@@ -10,7 +10,7 @@ default_attributes(
         :family => :inet,
         :address => "10.0.32.21",
         :bond => {
-          :slaves => %w(em1 em2)
+          :slaves => %w[em1 em2]
         }
       },
       :external_ipv4 => {
index 6df29d9..7710900 100644 (file)
@@ -10,7 +10,7 @@ default_attributes(
         :family => :inet,
         :address => "10.0.32.22",
         :bond => {
-          :slaves => %w(em1 em2)
+          :slaves => %w[em1 em2]
         }
       },
       :external_ipv4 => {
index 70e0e5b..6bb0b45 100644 (file)
@@ -1,4 +1,5 @@
 # coding: utf-8
+
 name "teleservice"
 description "Role applied to all servers at Teleservice"
 
index 1487ae3..8954a8c 100644 (file)
@@ -10,7 +10,7 @@ default_attributes(
         :family => :inet,
         :address => "10.0.32.41",
         :bond => {
-          :slaves => %w(em1 em2)
+          :slaves => %w[em1 em2]
         }
       }
     }
index e36e220..e0b2091 100644 (file)
@@ -10,7 +10,7 @@ default_attributes(
         :family => :inet,
         :address => "10.0.32.42",
         :bond => {
-          :slaves => %w(em1 em2)
+          :slaves => %w[em1 em2]
         }
       }
     }
index 86ef45b..c6e21bd 100644 (file)
@@ -3,7 +3,7 @@ description "Role applied to machines using the Tyan S7010 motherboard"
 
 default_attributes(
   :hardware => {
-    :modules => %w(i2c_i801 jc42 w83793),
+    :modules => %w[i2c_i801 jc42 w83793],
     :sensors => {
       "jc42-*" => {
         :temps => {
index dce003b..7427f05 100644 (file)
@@ -15,7 +15,7 @@ default_attributes(
   },
   :web => {
     :status => "online",
-    :memcached_servers => %w(rails1.ic rails2.ic rails3.ic)
+    :memcached_servers => %w[rails1.ic rails2.ic rails3.ic]
   }
 )