Fix some correctness issues identified by foodcritic
authorTom Hughes <tom@compton.nu>
Fri, 22 Nov 2013 22:49:01 +0000 (22:49 +0000)
committerTom Hughes <tom@compton.nu>
Fri, 22 Nov 2013 22:49:01 +0000 (22:49 +0000)
cookbooks/dns/metadata.rb
cookbooks/hardware/metadata.rb
cookbooks/munin/metadata.rb
cookbooks/networking/recipes/default.rb
cookbooks/openssh/recipes/default.rb
cookbooks/osqa/metadata.rb
cookbooks/otrs/metadata.rb
cookbooks/otrs/recipes/default.rb
cookbooks/stats/recipes/default.rb

index d2b4f54..41820d8 100644 (file)
@@ -4,3 +4,5 @@ license           "Apache 2.0"
 description       "Configure DNS management"
 long_description  IO.read(File.join(File.dirname(__FILE__), 'README.rdoc'))
 version           "1.0.0"
+depends           "git"
+depends           "apache"
index 8136046..013c604 100644 (file)
@@ -4,3 +4,4 @@ license           "Apache 2.0"
 description       "Configures hardware"
 long_description  IO.read(File.join(File.dirname(__FILE__), 'README.rdoc'))
 version           "1.0.0"
+depends           "git"
index d132b5b..0a25ccb 100644 (file)
@@ -4,4 +4,5 @@ license           "Apache 2.0"
 description       "Installs and configures munin"
 long_description  IO.read(File.join(File.dirname(__FILE__), 'README.rdoc'))
 version           "1.0.0"
+depends           "apache"
 depends           "networking"
index 29ea9ad..9e87e28 100644 (file)
@@ -185,7 +185,7 @@ template "/etc/shorewall/rules" do
   owner "root"
   group "root"
   mode 0644
-  variables :family => "inet", :rules => []
+  variables :rules => []
   notifies :restart, resources(:service => "shorewall")
 end
 
@@ -290,7 +290,7 @@ if not node.interfaces(:family => :inet6).empty?
     owner "root"
     group "root"
     mode 0644
-    variables :family => "inet6", :rules => []
+    variables :rules => []
     notifies :restart, resources(:service => "shorewall6")
   end
 
index 79a2dd1..432da26 100644 (file)
@@ -48,7 +48,9 @@ end.collect do |node|
   }
 
   if node[:keys][:ssh][:host_ecdsa_public]
-    keys[node[:keys][:ssh][:host_ecdsa_type]] = node[:keys][:ssh][:host_ecdsa_public]
+    ecdsa_type = node[:keys][:ssh][:host_ecdsa_type]
+
+    keys[ecdsa_type] = node[:keys][:ssh][:host_ecdsa_public]
   end
 
   Hash[
index da3014e..980b84b 100644 (file)
@@ -6,6 +6,7 @@ long_description  IO.read(File.join(File.dirname(__FILE__), 'README.rdoc'))
 version           "1.0.0"
 depends           "apache"
 depends           "memcached"
+depends           "tools"
 
 attribute "osqa",
   :display_name => "OSQA",
index 5627836..b8781ad 100644 (file)
@@ -6,6 +6,7 @@ long_description  IO.read(File.join(File.dirname(__FILE__), 'README.rdoc'))
 version           "1.0.0"
 depends           "apache"
 depends           "postgresql"
+depends           "tools"
 
 attribute "otrs",
   :display_name => "OTRS",
index c0f9c33..901c0ee 100644 (file)
@@ -18,6 +18,7 @@
 #
 
 include_recipe "tools"
+include_recipe "postgresql"
 include_recipe "apache::ssl"
 
 passwords = data_bag_item("otrs", "passwords")
index b142073..aec5135 100644 (file)
@@ -63,7 +63,7 @@ template "/srv/stats.openstreetmap.org/index.html" do
   source "index.html.erb"
   owner "root"
   group "root"
-  mode 644
+  mode 0644
   variables :sites => node[:stats][:sites]
 end