X-Git-Url: https://git.openstreetmap.org/chef.git/blobdiff_plain/b09d9f97fea24b4a1f56a97724d13752d6340996..34ff9fd8c98de023c344dada734e3e9897d132fe:/cookbooks/dev/recipes/default.rb

diff --git a/cookbooks/dev/recipes/default.rb b/cookbooks/dev/recipes/default.rb
index ef8e2dc77..5186e1109 100644
--- a/cookbooks/dev/recipes/default.rb
+++ b/cookbooks/dev/recipes/default.rb
@@ -24,61 +24,78 @@ include_recipe "apache"
 include_recipe "passenger"
 include_recipe "git"
 include_recipe "mysql"
+include_recipe "nodejs"
 include_recipe "postgresql"
-
-package "php-apc"
-package "php-db"
-package "php-cgiwrap"
-package "php-pear"
-
-package "php5-cgi"
-package "php5-cli"
-package "php5-curl"
-package "php5-fpm"
-package "php5-imagick"
-package "php5-mcrypt"
-package "php5-mysql"
-package "php5-pgsql"
-package "php5-sqlite"
-
-package "python"
-package "python-argparse"
-package "python-beautifulsoup"
-package "python-cheetah"
-package "python-dateutil"
-package "python-magic"
-package "python-psycopg2"
-
-easy_install_package "geojson"
+include_recipe "python"
+
+package %w[
+  php
+  php-cgi
+  php-cli
+  php-curl
+  php-db
+  php-fpm
+  php-imagick
+  php-mcrypt
+  php-mysql
+  php-pear
+  php-pgsql
+  php-sqlite3
+]
+
+package %w[
+  pngcrush
+  pngquant
+]
+
+package %w[
+  python
+  python-argparse
+  python-beautifulsoup
+  python-cheetah
+  python-dateutil
+  python-magic
+  python-psycopg2
+  python-gdal
+]
+
+nodejs_package "svgo"
+
+python_package "geojson"
 
 apache_module "env"
 apache_module "expires"
-apache_module "fastcgi-handler"
+apache_module "headers"
+apache_module "proxy"
+apache_module "proxy_fcgi"
 apache_module "rewrite"
+apache_module "suexec"
+apache_module "userdir"
 apache_module "wsgi"
 
+package "apache2-suexec-pristine"
+
 gem_package "sqlite3"
 
 gem_package "rails" do
   version "3.0.9"
 end
 
-service "php5-fpm" do
-  action [ :enable, :start ]
-  supports :status => true, :restart => true, :reload => true
+service "php7.0-fpm" do
+  action [:enable, :start]
 end
 
-template "/etc/php5/fpm/pool.d/default.conf" do
+template "/etc/php/7.0/fpm/pool.d/default.conf" do
   source "fpm-default.conf.erb"
   owner "root"
   group "root"
-  mode 0644
-  notifies :reload, "service[php5-fpm]"
+  mode 0o644
+  notifies :reload, "service[php7.0-fpm]"
 end
 
-file "/etc/php5/fpm/pool.d/www.conf" do
+file "/etc/php/7.0/fpm/pool.d/www.conf" do
   action :delete
-  notifies :reload, "service[php5-fpm]"
+  notifies :reload, "service[php7.0-fpm]"
 end
 
 package "phppgadmin"
@@ -87,10 +104,10 @@ template "/etc/phppgadmin/config.inc.php" do
   source "phppgadmin.conf.erb"
   owner "root"
   group "root"
-  mode 0644
+  mode 0o644
 end
 
-link "/etc/apache2/conf.d/phppgadmin" do
+file "/etc/apache2/conf.d/phppgadmin" do
   action :delete
 end
 
@@ -102,91 +119,107 @@ search(:accounts, "*:*").each do |account|
   name = account["id"]
   details = node[:accounts][:users][name] || {}
 
-  if ["user","administrator"].include?(details[:status])
-    user_home = details[:home] || account["home"] || "#{node[:accounts][:home]}/#{name.to_s}"
-
-    if File.directory?("#{user_home}/public_html")
-      template "/etc/php5/fpm/pool.d/#{name}.conf" do
-        source "fpm.conf.erb"
-        owner "root"
-        group "root"
-        mode 0644
-        variables :user => name
-        notifies :reload, "service[php5-fpm]"
-      end
+  next unless %w[user administrator].include?(details[:status])
 
-      apache_site "#{name}.dev.openstreetmap.org" do
-        template "apache.user.erb"
-        directory "#{user_home}/public_html"
-        variables :user => name
-      end
-    end
+  user_home = details[:home] || account["home"] || "#{node[:accounts][:home]}/#{name}"
+
+  next unless File.directory?("#{user_home}/public_html")
+
+  port = 7000 + account["uid"].to_i
+
+  template "/etc/php/7.0/fpm/pool.d/#{name}.conf" do
+    source "fpm.conf.erb"
+    owner "root"
+    group "root"
+    mode 0o644
+    variables :user => name, :port => port
+    notifies :reload, "service[php7.0-fpm]"
+  end
+
+  apache_site "#{name}.dev.openstreetmap.org" do
+    template "apache.user.erb"
+    directory "#{user_home}/public_html"
+    variables :user => name, :port => port
+  end
+
+  template "/etc/sudoers.d/#{name}" do
+    source "sudoers.user.erb"
+    owner "root"
+    group "root"
+    mode 0o440
+    variables :user => name
   end
 end
 
-if node[:postgresql][:clusters][:"9.1/main"]
+if node[:postgresql][:clusters][:"9.5/main"]
   postgresql_user "apis" do
-    cluster "9.1/main"
+    cluster "9.5/main"
   end
 
-  node[:dev][:rails].each do |name,details|
+  template "/usr/local/bin/cleanup-rails-assets" do
+    cookbook "web"
+    source "cleanup-assets.erb"
+    owner "root"
+    group "root"
+    mode 0o755
+  end
+
+  node[:dev][:rails].each do |name, details|
     database_name = details[:database] || "apis_#{name}"
     site_name = "#{name}.apis.dev.openstreetmap.org"
-    site_aliases = details[:aliases] || []
     rails_directory = "/srv/#{name}.apis.dev.openstreetmap.org"
-    secret_key_base = details[:secret_key_base] || SecureRandom.base64(96)
 
-    node.set[:dev][:rails][name][:secret_key_base] = secret_key_base
-
-    postgresql_database database_name do
-      cluster "9.1/main"
-      owner "apis"
-    end
+    if details[:repository]
+      site_aliases = details[:aliases] || []
+      secret_key_base = details[:secret_key_base] || SecureRandom.base64(96)
 
-    postgresql_extension "#{database_name}_btree_gist" do
-      cluster "9.1/main"
-      database database_name
-      extension "btree_gist"
-    end
+      node.normal[:dev][:rails][name][:secret_key_base] = secret_key_base
 
-    rails_port site_name do
-      ruby node[:passenger][:ruby_version]
-      directory rails_directory
-      user "apis"
-      group "apis"
-      repository details[:repository]
-      revision details[:revision]
-      database_port node[:postgresql][:clusters][:"9.1/main"][:port]
-      database_name database_name
-      database_username "apis"
-      run_migrations true
-    end
+      postgresql_database database_name do
+        cluster "9.5/main"
+        owner "apis"
+      end
 
-    template "#{rails_directory}/config/initializers/setup.rb" do
-      source "rails.setup.rb.erb"
-      owner "apis"
-      group "apis"
-      mode 0644
-      variables :site => site_name
-      notifies :touch, "file[#{rails_directory}/tmp/restart.txt]"
-    end
+      postgresql_extension "#{database_name}_btree_gist" do
+        cluster "9.5/main"
+        database database_name
+        extension "btree_gist"
+      end
 
-    apache_site site_name do
-      template "apache.rails.erb"
-      variables :name => site_name, :aliases => site_aliases, :secret_key_base => secret_key_base
-    end
-  end
+      rails_port site_name do
+        ruby node[:passenger][:ruby_version]
+        directory rails_directory
+        user "apis"
+        group "apis"
+        repository details[:repository]
+        revision details[:revision]
+        database_port node[:postgresql][:clusters][:"9.5/main"][:port]
+        database_name database_name
+        database_username "apis"
+        run_migrations true
+      end
 
-  Dir.glob("/srv/*.apis.dev.openstreetmap.org").each do |rails_directory|
-    name = File.basename(rails_directory, ".apis.dev.openstreetmap.org")
+      template "#{rails_directory}/config/initializers/setup.rb" do
+        source "rails.setup.rb.erb"
+        owner "apis"
+        group "apis"
+        mode 0o644
+        variables :site => site_name
+        notifies :restart, "rails_port[#{site_name}]"
+      end
 
-    unless node[:dev][:rails].include?(name) and node[:dev][:rails][name].include?(:repository)
-      database_config = YAML.load_file("#{rails_directory}/config/database.yml")
-      database_name = database_config["production"]["database"]
-      site_name = "#{name}.apis.dev.openstreetmap.org"
+      ssl_certificate site_name do
+        domains [site_name] + site_aliases
+        notifies :reload, "service[apache2]"
+      end
 
       apache_site site_name do
-        action [ :delete ]
+        template "apache.rails.erb"
+        variables :name => site_name, :aliases => site_aliases, :secret_key_base => secret_key_base
+      end
+    else
+      apache_site site_name do
+        action [:delete]
       end
 
       directory rails_directory do
@@ -194,35 +227,42 @@ if node[:postgresql][:clusters][:"9.1/main"]
         recursive true
       end
 
-      file "/etc/cron.daily/rails-#{name}" do
+      file "/etc/cron.daily/rails-#{site_name.tr('.', '-')}" do
         action :delete
       end
 
       postgresql_database database_name do
         action :drop
-        cluster "9.1/main"
+        cluster "9.5/main"
       end
+
+      node.normal[:dev][:rails].delete(name)
     end
   end
 
   directory "/srv/apis.dev.openstreetmap.org" do
     owner "apis"
     group "apis"
-    mode 0755
+    mode 0o755
   end
 
   template "/srv/apis.dev.openstreetmap.org/index.html" do
     source "apis.html.erb"
     owner "apis"
     group "apis"
-    mode 0644
+    mode 0o644
+  end
+
+  ssl_certificate "apis.dev.openstreetmap.org" do
+    domains "apis.dev.openstreetmap.org"
+    notifies :reload, "service[apache2]"
   end
 
   apache_site "apis.dev.openstreetmap.org" do
     template "apache.apis.erb"
   end
 
-  node[:postgresql][:clusters].each do |name,details|
+  node[:postgresql][:clusters].each_key do |name|
     postgresql_munin name do
       cluster name
       database "ALL"