Extract logic from template to recipe.

diff --git a/cookbooks/web/recipes/cgimap.rb b/cookbooks/web/recipes/cgimap.rb
index 32b6365ca46acd813e4293bcdcdedc750d06bc2b..fc1e1f47ca6d2932501c4c05b04c5e7488349cad 100644 (file)
--- a/cookbooks/web/recipes/cgimap.rb
+++ b/cookbooks/web/recipes/cgimap.rb
@@ -24,12 +24,29 @@ db_passwords = data_bag_item("db", "passwords")
 
 package "openstreetmap-cgimap-bin"
 
+if node[:web][:readonly_database_host]
+  database_host = node[:web][:readonly_database_host]
+  database_readonly = true
+else
+  database_host = node[:web][:database_host]
+  database_readonly = node[:web][:status] == "database_readonly"
+end
+
+memcached_servers = node[:web][:memcached_servers] || []
+
 template "/etc/init.d/cgimap" do
   owner "root"
   group "root"
   mode 0o755
   source "cgimap.init.erb"
-  variables :db_password => db_passwords["rails"]
+  variables {
+    :db_password => db_passwords["rails"],
+    :pid_directory => node[:web][:pid_directory],
+    :log_directory => node[:web][:log_directory],
+    :database_host => database_host,
+    :database_readonly => database_readonly,
+    :memcached_servers => memcached_servers
+  }
 end
 
 if %w(database_offline api_offline).include?(node[:web][:status])

diff --git a/cookbooks/web/templates/default/cgimap.init.erb b/cookbooks/web/templates/default/cgimap.init.erb
index a2b25d2d9507300b0579242631cd21fc140b6463..888e10cff321d1d29cd23163698fa68cb3488d14 100644 (file)
--- a/cookbooks/web/templates/default/cgimap.init.erb
+++ b/cookbooks/web/templates/default/cgimap.init.erb
@@ -1,29 +1,14 @@
 #!/bin/bash
 
-<%
-pid_directory = node[:web][:pid_directory]
-log_directory = node[:web][:log_directory]
-
-if node[:web][:readonly_database_host]
-  database_host = node[:web][:readonly_database_host]
-  database_readonly = true
-else
-  database_host = node[:web][:database_host]
-  database_readonly = node[:web][:status] == "database_readonly"
-end
-
-memcached_servers = node[:web][:memcached_servers] || []
-%>
-
-CGIMAP_HOST=<%= database_host %>; export CGIMAP_HOST
+CGIMAP_HOST=<%= @database_host %>; export CGIMAP_HOST
 CGIMAP_DBNAME=openstreetmap; export CGIMAP_DBNAME
 CGIMAP_USERNAME=rails; export CGIMAP_USERNAME
 CGIMAP_PASSWORD=<%= @db_password %>; export CGIMAP_PASSWORD
 
-CGIMAP_PIDFILE=<%= pid_directory + "/cgimap.pid" %>; export CGIMAP_PIDFILE
-CGIMAP_LOGFILE=<%= log_directory + "/cgimap.log" %>; export CGIMAP_LOGFILE
+CGIMAP_PIDFILE=<%= @pid_directory + "/cgimap.pid" %>; export CGIMAP_PIDFILE
+CGIMAP_LOGFILE=<%= @log_directory + "/cgimap.log" %>; export CGIMAP_LOGFILE
 
-CGIMAP_MEMCACHE=<%= memcached_servers.join(",") %>; export CGIMAP_MEMCACHE
+CGIMAP_MEMCACHE=<%= @memcached_servers.join(",") %>; export CGIMAP_MEMCACHE
 CGIMAP_RATELIMIT=204800; export CGIMAP_RATELIMIT
 CGIMAP_MAXDEBT=250; export CGIMAP_MAXDEBT