Use default sandboxing for tilelog service

[chef.git] / cookbooks / tile / recipes / default.rb

diff --git a/cookbooks/tile/recipes/default.rb b/cookbooks/tile/recipes/default.rb
index 1bc45e92f92082b6629ace57305104967120bb01..b62c89d1dccaa655ddf4725f3b773bf831e05f6e 100644 (file)
--- a/cookbooks/tile/recipes/default.rb
+++ b/cookbooks/tile/recipes/default.rb
@@ -25,6 +25,7 @@ include_recipe "nodejs"
 include_recipe "postgresql"
 include_recipe "prometheus"
 include_recipe "python"
 include_recipe "postgresql"
 include_recipe "prometheus"
 include_recipe "python"

+include_recipe "ruby"

 include_recipe "tools"
 
 blocks = data_bag_item("tile", "blocks")
 include_recipe "tools"
 
 blocks = data_bag_item("tile", "blocks")


@@ -99,13 +100,9 @@ end
 package "renderd"
 
 systemd_service "renderd" do
 package "renderd"
 
 systemd_service "renderd" do

-  description "Mapnik rendering daemon"
+  dropin "chef"

   after "postgresql.service"
   wants "postgresql.service"
   after "postgresql.service"
   wants "postgresql.service"

-  user "www-data"
-  exec_start "/usr/bin/renderd -f"
-  runtime_directory "renderd"
-  standard_error "null"

   limit_nofile 4096
   private_tmp true
   private_devices true
   limit_nofile 4096
   private_tmp true
   private_devices true


@@ -116,6 +113,10 @@ systemd_service "renderd" do
   restart "on-failure"
 end
 
   restart "on-failure"
 end
 

+systemd_service "renderd" do
+  action :delete
+end
+

 service "renderd" do
   action [:enable, :start]
   subscribes :restart, "systemd_service[renderd]"
 service "renderd" do
   action [:enable, :start]
   subscribes :restart, "systemd_service[renderd]"


@@ -156,6 +157,7 @@ end
 package %w[
   python3-cairo
   python3-mapnik
 package %w[
   python3-cairo
   python3-mapnik

+  python3-pyproj

   python3-setuptools
 ]
 
   python3-setuptools
 ]
 


@@ -163,24 +165,6 @@ python_package "pyotp" do
   python_version "3"
 end
 
   python_version "3"
 end
 

-package %w[
-  fonts-noto-cjk
-  fonts-noto-hinted
-  fonts-noto-unhinted
-  fonts-hanazono
-  ttf-unifont
-]
-
-["NotoSansArabicUI-Regular.ttf", "NotoSansArabicUI-Bold.ttf"].each do |font|
-  remote_file "/usr/share/fonts/truetype/noto/#{font}" do
-    action :create_if_missing
-    source "https://github.com/googlei18n/noto-fonts/raw/master/hinted/#{font}"
-    owner "root"
-    group "root"
-    mode "644"
-  end
-end
-

 directory "/srv/tile.openstreetmap.org/cgi-bin" do
   owner "tile"
   group "tile"
 directory "/srv/tile.openstreetmap.org/cgi-bin" do
   owner "tile"
   group "tile"


@@ -215,7 +199,11 @@ directory "/srv/tile.openstreetmap.org/data" do
   mode "755"
 end
 
   mode "755"
 end
 

-package "mapnik-utils"
+package %w[
+  mapnik-utils
+  tar
+  unzip
+]

 
 node[:tile][:data].each_value do |data|
   url = data[:url]
 
 node[:tile][:data].each_value do |data|
   url = data[:url]


@@ -234,8 +222,6 @@ node[:tile][:data].each_value do |data|
   end
 
   if file =~ /\.tgz$/
   end
 
   if file =~ /\.tgz$/

-    package "tar"
-

     execute file do
       action :nothing
       command "tar -zxf #{file} -C #{directory}"
     execute file do
       action :nothing
       command "tar -zxf #{file} -C #{directory}"


@@ -243,8 +229,6 @@ node[:tile][:data].each_value do |data|
       group "tile"
     end
   elsif file =~ /\.tar\.bz2$/
       group "tile"
     end
   elsif file =~ /\.tar\.bz2$/

-    package "tar"
-

     execute file do
       action :nothing
       command "tar -jxf #{file} -C #{directory}"
     execute file do
       action :nothing
       command "tar -jxf #{file} -C #{directory}"


@@ -252,8 +236,6 @@ node[:tile][:data].each_value do |data|
       group "tile"
     end
   elsif file =~ /\.zip$/
       group "tile"
     end
   elsif file =~ /\.zip$/

-    package "unzip"
-

     execute file do
       action :nothing
       command "unzip -qq -o #{file} -d #{directory}"
     execute file do
       action :nothing
       command "unzip -qq -o #{file} -d #{directory}"


@@ -293,15 +275,19 @@ nodejs_package "carto"
 
 systemd_service "update-lowzoom@" do
   description "Low zoom tile update service for %i layer"
 
 systemd_service "update-lowzoom@" do
   description "Low zoom tile update service for %i layer"

-  conflicts "render-lowzoom.service"

   user "tile"
   user "tile"

+  exec_start_pre "+/bin/systemctl stop render-lowzoom.service"

   exec_start "/bin/bash /usr/local/bin/update-lowzoom-%i"
   runtime_directory "update-lowzoom-%i"
   private_tmp true
   private_devices true
   private_network true
   exec_start "/bin/bash /usr/local/bin/update-lowzoom-%i"
   runtime_directory "update-lowzoom-%i"
   private_tmp true
   private_devices true
   private_network true

-  protect_system "full"
+  protect_system "strict"

   protect_home true
   protect_home true

+  read_write_paths [
+    "/srv/tile.openstreetmap.org/tiles/%i",
+    "/var/log/tile"
+  ]

   no_new_privileges true
   restart "on-failure"
 end
   no_new_privileges true
   restart "on-failure"
 end


@@ -337,15 +323,15 @@ node[:tile][:styles].each do |name, details|
 
   details[:tile_directories].each do |directory|
     directory directory[:name] do
 
   details[:tile_directories].each do |directory|
     directory directory[:name] do

-      owner "www-data"
-      group "www-data"
+      owner "_renderd"
+      group "_renderd"

       mode "755"
     end
 
     directory[:min_zoom].upto(directory[:max_zoom]) do |zoom|
       directory "#{directory[:name]}/#{zoom}" do
       mode "755"
     end
 
     directory[:min_zoom].upto(directory[:max_zoom]) do |zoom|
       directory "#{directory[:name]}/#{zoom}" do

-        owner "www-data"
-        group "www-data"
+        owner "_renderd"
+        group "_renderd"

         mode "755"
       end
 
         mode "755"
       end
 


@@ -378,9 +364,20 @@ node[:tile][:styles].each do |name, details|
     group "tile"
   end
 
     group "tile"
   end
 

+  if details[:fonts_script]
+    execute details[:fonts_script] do
+      action :nothing
+      command details[:fonts_script]
+      cwd style_directory
+      user "tile"
+      group "tile"
+      subscribes :run, "git[#{style_directory}]"
+    end
+  end
+

   execute "#{style_directory}/project.mml" do
     action :nothing
   execute "#{style_directory}/project.mml" do
     action :nothing

-    command "carto -a 3.0.0 project.mml > project.xml"
+    command "carto -a 3.0.22 project.mml > project.xml"

     cwd style_directory
     user "tile"
     group "tile"
     cwd style_directory
     user "tile"
     group "tile"


@@ -405,6 +402,11 @@ postgresql_user "tomh" do
   superuser true
 end
 
   superuser true
 end
 

+postgresql_user "pnorman" do
+  cluster node[:tile][:database][:cluster]
+  superuser true
+end
+

 postgresql_user "tile" do
   cluster node[:tile][:database][:cluster]
 end
 postgresql_user "tile" do
   cluster node[:tile][:database][:cluster]
 end


@@ -413,6 +415,10 @@ postgresql_user "www-data" do
   cluster node[:tile][:database][:cluster]
 end
 
   cluster node[:tile][:database][:cluster]
 end
 

+postgresql_user "_renderd" do
+  cluster node[:tile][:database][:cluster]
+end
+

 postgresql_database "gis" do
   cluster node[:tile][:database][:cluster]
   owner "tile"
 postgresql_database "gis" do
   cluster node[:tile][:database][:cluster]
   owner "tile"


@@ -443,7 +449,7 @@ end
     cluster node[:tile][:database][:cluster]
     database "gis"
     owner "tile"
     cluster node[:tile][:database][:cluster]
     database "gis"
     owner "tile"

-    permissions "tile" => :all, "www-data" => :select
+    permissions "tile" => :all, "www-data" => :select, "_renderd" => :select

   end
 end
 
   end
 end
 


@@ -455,10 +461,20 @@ package %w[
 
 if node[:tile][:database][:external_data_script]
   execute node[:tile][:database][:external_data_script] do
 
 if node[:tile][:database][:external_data_script]
   execute node[:tile][:database][:external_data_script] do

-    command "#{node[:tile][:database][:external_data_script]} -R www-data"
+    command "#{node[:tile][:database][:external_data_script]} -R _renderd"

     cwd "/srv/tile.openstreetmap.org"
     user "tile"
     group "tile"
     cwd "/srv/tile.openstreetmap.org"
     user "tile"
     group "tile"

+    ignore_failure true
+  end
+
+  Array(node[:tile][:database][:external_data_tables]).each do |table|
+    postgresql_table table do
+      cluster node[:tile][:database][:cluster]
+      database "gis"
+      owner "tile"
+      permissions "tile" => :all, "www-data" => :select, "_renderd" => :select
+    end

   end
 end
 
   end
 end
 


@@ -476,7 +492,7 @@ end
 
 file node[:tile][:database][:node_file] do
   owner "tile"
 
 file node[:tile][:database][:node_file] do
   owner "tile"

-  group "www-data"
+  group "_renderd"

   mode "660"
 end
 
   mode "660"
 end
 


@@ -488,15 +504,22 @@ end
 
 package %w[
   osm2pgsql
 
 package %w[
   osm2pgsql

-  ruby

   osmium-tool
   pyosmium
   python3-pyproj
 ]
 
   osmium-tool
   pyosmium
   python3-pyproj
 ]
 

-gem_package "apachelogregex"
-gem_package "file-tail"
-gem_package "lru_redux"
+gem_package "apachelogregex" do
+  gem_binary node[:ruby][:gem]
+end
+
+gem_package "file-tail" do
+  gem_binary node[:ruby][:gem]
+end
+
+gem_package "lru_redux" do
+  gem_binary node[:ruby][:gem]
+end

 
 remote_directory "/usr/local/bin" do
   source "bin"
 
 remote_directory "/usr/local/bin" do
   source "bin"


@@ -521,10 +544,11 @@ systemd_service "tile-ratelimit" do
   user "tile"
   group "adm"
   exec_start "/usr/local/bin/tile-ratelimit"
   user "tile"
   group "adm"
   exec_start "/usr/local/bin/tile-ratelimit"

+  nice 10

   private_tmp true
   private_devices true
   private_network true
   private_tmp true
   private_devices true
   private_network true

-  protect_system "full"
+  protect_system "strict"

   protect_home true
   read_write_paths "/srv/tile.openstreetmap.org/conf"
   no_new_privileges true
   protect_home true
   read_write_paths "/srv/tile.openstreetmap.org/conf"
   no_new_privileges true


@@ -533,7 +557,7 @@ end
 
 service "tile-ratelimit" do
   action [:enable, :start]
 
 service "tile-ratelimit" do
   action [:enable, :start]

-  subscribes :restart, "file[/usr/local/bin/time-ratelimit]"
+  subscribes :restart, "file[/usr/local/bin/tile-ratelimit]"

   subscribes :restart, "systemd_service[tile-ratelimit]"
 end
 
   subscribes :restart, "systemd_service[tile-ratelimit]"
 end
 


@@ -552,7 +576,7 @@ end
 
 directory "/var/lib/replicate/expire-queue" do
   owner "tile"
 
 directory "/var/lib/replicate/expire-queue" do
   owner "tile"

-  group "www-data"
+  group "_renderd"

   mode "775"
 end
 
   mode "775"
 end
 


@@ -567,13 +591,20 @@ end
 systemd_service "expire-tiles" do
   description "Tile dirtying service"
   type "simple"
 systemd_service "expire-tiles" do
   description "Tile dirtying service"
   type "simple"

-  user "www-data"
+  user "_renderd"

   exec_start "/usr/local/bin/expire-tiles"
   exec_start "/usr/local/bin/expire-tiles"

+  nice 10

   standard_output "null"
   private_tmp true
   private_devices true
   standard_output "null"
   private_tmp true
   private_devices true

-  protect_system "full"
+  protect_system "strict"

   protect_home true
   protect_home true

+  read_write_paths [
+    "/store/database/nodes",
+    "/store/tiles/%i",
+    "/var/lib/replicate/expire-queue",
+    "/var/log/tile"
+  ]

   no_new_privileges true
 end
 
   no_new_privileges true
 end
 


@@ -595,8 +626,13 @@ systemd_service "replicate" do
   exec_start "/usr/local/bin/replicate"
   private_tmp true
   private_devices true
   exec_start "/usr/local/bin/replicate"
   private_tmp true
   private_devices true

-  protect_system "full"
+  protect_system "strict"

   protect_home true
   protect_home true

+  read_write_paths [
+    "/store/database/nodes",
+    "/var/lib/replicate",
+    "/var/log/tile"
+  ]

   no_new_privileges true
   restart "on-failure"
 end
   no_new_privileges true
   restart "on-failure"
 end


@@ -629,14 +665,15 @@ systemd_service "render-lowzoom" do
   private_tmp true
   private_devices true
   private_network true
   private_tmp true
   private_devices true
   private_network true

-  protect_system "full"
+  protect_system "strict"

   protect_home true
   protect_home true

+  read_write_paths "/var/log/tile"

   no_new_privileges true
 end
 
 systemd_timer "render-lowzoom" do
   description "Render low zoom tiles"
   no_new_privileges true
 end
 
 systemd_timer "render-lowzoom" do
   description "Render low zoom tiles"

-  on_calendar "Sun *-*~07/1 01:00:00"
+  on_calendar "Fri *-*-* 23:00:00 UTC"

 end
 
 service "render-lowzoom.timer" do
 end
 
 service "render-lowzoom.timer" do


@@ -662,7 +699,7 @@ tile_directories.each do |directory|
 
   cron_d "cleanup-tiles#{label}" do
     minute "0"
 
   cron_d "cleanup-tiles#{label}" do
     minute "0"

-    user "www-data"
+    user "_renderd"

     command "ionice -c 3 /usr/local/bin/cleanup-tiles #{directory}"
     mailto "admins@openstreetmap.org"
   end
     command "ionice -c 3 /usr/local/bin/cleanup-tiles #{directory}"
     mailto "admins@openstreetmap.org"
   end


@@ -681,6 +718,8 @@ munin_plugin "renderd_zoom_time"
 
 munin_plugin "replication_delay"
 
 
 munin_plugin "replication_delay"
 

+package "ruby-webrick"
+

 prometheus_exporter "modtile" do
   port 9494
 end
 prometheus_exporter "modtile" do
   port 9494
 end