]> git.openstreetmap.org Git - chef.git/commitdiff
Disable bogus foodcritic warning
authorTom Hughes <tom@compton.nu>
Fri, 23 Oct 2015 08:06:04 +0000 (09:06 +0100)
committerTom Hughes <tom@compton.nu>
Fri, 23 Oct 2015 08:06:04 +0000 (09:06 +0100)
https://github.com/acrmp/foodcritic/issues/392

cookbooks/logstash/recipes/default.rb
cookbooks/munin/recipes/default.rb
cookbooks/munin/recipes/server.rb

index 55675fa71ee78042aef6b81d8c2757eda4cd798b..b5d03970567372b59a18e38f9e47c497ea0eb48e 100644 (file)
@@ -60,7 +60,7 @@ template "/etc/cron.daily/expire-logstash" do
   mode 0755
 end
 
   mode 0755
 end
 
-forwarders = search(:node, "recipes:logstash\\:\\:forwarder")
+forwarders = search(:node, "recipes:logstash\\:\\:forwarder") # ~FC010
 
 forwarders.each do |forwarder|
   forwarder.interfaces(:role => :external) do |interface|
 
 forwarders.each do |forwarder|
   forwarder.interfaces(:role => :external) do |interface|
index 3bffa27a6ed32869a20a874e46d0a5bcbf79a82e..fb1225c2f25a056e7572b5e936e0f91de60ae8fd 100644 (file)
@@ -29,7 +29,7 @@ service "munin-node" do
   supports :status => true, :restart => true, :reload => true
 end
 
   supports :status => true, :restart => true, :reload => true
 end
 
-servers = search(:node, "recipes:munin\\:\\:server")
+servers = search(:node, "recipes:munin\\:\\:server") # ~FC010
 
 servers.each do |server|
   server.interfaces(:role => :external) do |interface|
 
 servers.each do |server|
   server.interfaces(:role => :external) do |interface|
index 3b71eaf10e42e656b85cc5fe4819100b5e1b95ed..bb5ba243aeeb0973e6ca3882a0f26da5c8adb0b1 100644 (file)
@@ -45,9 +45,9 @@ munin_plugin "rrdcached"
 
 expiry_time = 14 * 86400
 
 
 expiry_time = 14 * 86400
 
-clients = search(:node, "recipes:munin\\:\\:default").sort_by { |n| n[:hostname] }
-frontends = search(:node, "recipes:web\\:\\:frontend").reject { |n| Time.now - Time.at(n[:ohai_time]) > expiry_time }.map { |n| n[:hostname] }.sort
-backends = search(:node, "recipes:web\\:\\:backend").reject { |n| Time.now - Time.at(n[:ohai_time]) > expiry_time }.map { |n| n[:hostname] }.sort
+clients = search(:node, "recipes:munin\\:\\:default").sort_by { |n| n[:hostname] } # ~FC010
+frontends = search(:node, "recipes:web\\:\\:frontend").reject { |n| Time.now - Time.at(n[:ohai_time]) > expiry_time }.map { |n| n[:hostname] }.sort # ~FC010
+backends = search(:node, "recipes:web\\:\\:backend").reject { |n| Time.now - Time.at(n[:ohai_time]) > expiry_time }.map { |n| n[:hostname] }.sort # ~FC010
 tilecaches = search(:node, "roles:tilecache").reject { |n| Time.now - Time.at(n[:ohai_time]) > expiry_time }.sort_by { |n| n[:hostname] }.map do |n|
   { :name => n[:hostname], :interface => n.interfaces(:role => :external).first[:interface] }
 end
 tilecaches = search(:node, "roles:tilecache").reject { |n| Time.now - Time.at(n[:ohai_time]) > expiry_time }.sort_by { |n| n[:hostname] }.map do |n|
   { :name => n[:hostname], :interface => n.interfaces(:role => :external).first[:interface] }
 end