Ensure assets are build before rails is restarted

[chef.git] / cookbooks / web / resources / rails_port.rb

diff --git a/cookbooks/web/resources/rails_port.rb b/cookbooks/web/resources/rails_port.rb
index 8159d87105e283b0c48e9418e66a42104cf64fcb..d275afede935da0d31b7c099382e35e5ed8d262c 100644 (file)
--- a/cookbooks/web/resources/rails_port.rb
+++ b/cookbooks/web/resources/rails_port.rb
@@ -22,6 +22,8 @@ require "yaml"
 resource_name :rails_port
 provides :rails_port
 
 resource_name :rails_port
 provides :rails_port
 

+unified_mode true
+

 default_action :create
 
 property :site, String, :name_property => true
 default_action :create
 
 property :site, String, :name_property => true


@@ -32,6 +34,7 @@ property :group, String
 property :repository, String, :default => "https://git.openstreetmap.org/public/rails.git"
 property :revision, String, :default => "live"
 property :run_migrations, [true, false], :default => false
 property :repository, String, :default => "https://git.openstreetmap.org/public/rails.git"
 property :revision, String, :default => "live"
 property :run_migrations, [true, false], :default => false

+property :build_assets, [true, false], :default => true

 property :email_from, String, :default => "OpenStreetMap <support@openstreetmap.org>"
 property :status, String, :default => "online"
 property :database_host, String
 property :email_from, String, :default => "OpenStreetMap <support@openstreetmap.org>"
 property :status, String, :default => "online"
 property :database_host, String


@@ -72,6 +75,7 @@ property :diary_feed_delay, Integer
 property :storage_configuration, Hash, :default => {}
 property :storage_service, String, :default => "local"
 property :storage_url, String
 property :storage_configuration, Hash, :default => {}
 property :storage_service, String, :default => "local"
 property :storage_url, String

+property :tile_cdn_url, String

 
 action :create do
   package %W[
 
 action :create do
   package %W[


@@ -79,12 +83,12 @@ action :create do
     ruby#{new_resource.ruby}-dev
     imagemagick
     nodejs
     ruby#{new_resource.ruby}-dev
     imagemagick
     nodejs

-    geoip-database

     tzdata
   ]
 
   package %w[
     g++
     tzdata
   ]
 
   package %w[
     g++

+    make

     pkg-config
     libpq-dev
     libsasl2-dev
     pkg-config
     libpq-dev
     libsasl2-dev


@@ -110,7 +114,7 @@ action :create do
 
   gem_package "bundler#{new_resource.ruby}" do
     package_name "bundler"
 
   gem_package "bundler#{new_resource.ruby}" do
     package_name "bundler"

-    version "1.16.2"
+    version "2.1.4"

     gem_binary "gem#{new_resource.ruby}"
     options "--format-executable"
   end
     gem_binary "gem#{new_resource.ruby}"
     options "--format-executable"
   end


@@ -133,11 +137,6 @@ action :create do
     depth 1
     user new_resource.user
     group new_resource.group
     depth 1
     user new_resource.user
     group new_resource.group

-    notifies :run, "execute[#{rails_directory}/Gemfile]"
-    notifies :run, "execute[#{rails_directory}/app/assets/javascripts/i18n]"
-    notifies :run, "execute[#{rails_directory}/public/assets]"
-    notifies :delete, "file[#{rails_directory}/public/export/embed.html]"
-    notifies :restart, "passenger_application[#{rails_directory}]"

   end
 
   declare_resource :directory, "#{rails_directory}/tmp" do
   end
 
   declare_resource :directory, "#{rails_directory}/tmp" do


@@ -161,7 +160,6 @@ action :create do
               :name => new_resource.database_name,
               :username => new_resource.database_username,
               :password => new_resource.database_password
               :name => new_resource.database_name,
               :username => new_resource.database_username,
               :password => new_resource.database_password

-    notifies :restart, "passenger_application[#{rails_directory}]"

   end
 
   application_yml = edit_file "#{rails_directory}/config/example.application.yml" do |line|
   end
 
   application_yml = edit_file "#{rails_directory}/config/example.application.yml" do |line|


@@ -184,7 +182,6 @@ action :create do
 
     line.gsub!(/^( *)#geonames_username:.*$/, "\\1geonames_username: \"openstreetmap\"")
 
 
     line.gsub!(/^( *)#geonames_username:.*$/, "\\1geonames_username: \"openstreetmap\"")
 

-    line.gsub!(/^( *)#geoip_database:.*$/, "\\1geoip_database: \"/usr/share/GeoIP/GeoIPv6.dat\"")

     line.gsub!(/^( *)#maxmind_database:.*$/, "\\1maxmind_database: \"/usr/share/GeoIP/GeoLite2-Country.mmdb\"")
 
     if new_resource.gpx_dir
     line.gsub!(/^( *)#maxmind_database:.*$/, "\\1maxmind_database: \"/usr/share/GeoIP/GeoLite2-Country.mmdb\"")
 
     if new_resource.gpx_dir


@@ -283,7 +280,6 @@ action :create do
     group new_resource.group
     mode "664"
     content application_yml
     group new_resource.group
     mode "664"
     content application_yml

-    notifies :run, "execute[#{rails_directory}/public/assets]"

     only_if { ::File.exist?("#{rails_directory}/config/example.application.yml") }
   end
 
     only_if { ::File.exist?("#{rails_directory}/config/example.application.yml") }
   end
 


@@ -323,14 +319,14 @@ action :create do
     "trace_use_job_queue",
     "diary_feed_delay",
     "storage_service",
     "trace_use_job_queue",
     "diary_feed_delay",
     "storage_service",

-    "storage_url"
-  ).reject { |_k, v| v.nil? }.merge(
+    "storage_url",
+    "tile_cdn_url"
+  ).compact.merge(

     "server_protocol" => "https",
     "server_url" => new_resource.site,
     "support_email" => "support@openstreetmap.org",
     "email_return_path" => "bounces@openstreetmap.org",
     "geonames_username" => "openstreetmap",
     "server_protocol" => "https",
     "server_url" => new_resource.site,
     "support_email" => "support@openstreetmap.org",
     "email_return_path" => "bounces@openstreetmap.org",
     "geonames_username" => "openstreetmap",

-    "geoip_database" => "/usr/share/GeoIP/GeoIPv6.dat",

     "maxmind_database" => "/usr/share/GeoIP/GeoLite2-Country.mmdb"
   )
 
     "maxmind_database" => "/usr/share/GeoIP/GeoLite2-Country.mmdb"
   )
 


@@ -348,7 +344,6 @@ action :create do
     group new_resource.group
     mode "664"
     content YAML.dump(settings)
     group new_resource.group
     mode "664"
     content YAML.dump(settings)

-    notifies :run, "execute[#{rails_directory}/public/assets]"

     only_if { ::File.exist?("#{rails_directory}/config/settings.yml") }
   end
 
     only_if { ::File.exist?("#{rails_directory}/config/settings.yml") }
   end
 


@@ -364,7 +359,6 @@ action :create do
     group new_resource.group
     mode "664"
     content YAML.dump(storage_configuration)
     group new_resource.group
     mode "664"
     content YAML.dump(storage_configuration)

-    notifies :run, "execute[#{rails_directory}/public/assets]"

   end
 
   if new_resource.piwik_configuration
   end
 
   if new_resource.piwik_configuration


@@ -373,12 +367,10 @@ action :create do
       group new_resource.group
       mode "664"
       content YAML.dump(new_resource.piwik_configuration)
       group new_resource.group
       mode "664"
       content YAML.dump(new_resource.piwik_configuration)

-      notifies :run, "execute[#{rails_directory}/public/assets]"

     end
   else
     file "#{rails_directory}/config/piwik.yml" do
       action :delete
     end
   else
     file "#{rails_directory}/config/piwik.yml" do
       action :delete

-      notifies :run, "execute[#{rails_directory}/public/assets]"

     end
   end
 
     end
   end
 


@@ -390,7 +382,7 @@ action :create do
     group "root"
     environment "NOKOGIRI_USE_SYSTEM_LIBRARIES" => "yes"
     subscribes :run, "gem_package[bundler#{new_resource.ruby}]"
     group "root"
     environment "NOKOGIRI_USE_SYSTEM_LIBRARIES" => "yes"
     subscribes :run, "gem_package[bundler#{new_resource.ruby}]"

-    notifies :restart, "passenger_application[#{rails_directory}]"
+    subscribes :run, "git[#{rails_directory}]"

   end
 
   execute "#{rails_directory}/db/migrate" do
   end
 
   execute "#{rails_directory}/db/migrate" do


@@ -400,36 +392,74 @@ action :create do
     user new_resource.user
     group new_resource.group
     subscribes :run, "git[#{rails_directory}]"
     user new_resource.user
     group new_resource.group
     subscribes :run, "git[#{rails_directory}]"

-    notifies :restart, "passenger_application[#{rails_directory}]"

     only_if { new_resource.run_migrations }
   end
 
     only_if { new_resource.run_migrations }
   end
 

+  package "yarnpkg" do
+    only_if { new_resource.build_assets }
+  end
+
+  execute "#{rails_directory}/package.json" do
+    action :nothing
+    command "bundle#{new_resource.ruby} exec rake yarn:install"
+    environment "HOME" => rails_directory,
+                "RAILS_ENV" => "production"
+    cwd rails_directory
+    user new_resource.user
+    group new_resource.group
+    subscribes :run, "git[#{rails_directory}]"
+    only_if { new_resource.build_assets }
+  end
+

   execute "#{rails_directory}/app/assets/javascripts/i18n" do
     action :nothing
     command "bundle#{new_resource.ruby} exec rake i18n:js:export"
   execute "#{rails_directory}/app/assets/javascripts/i18n" do
     action :nothing
     command "bundle#{new_resource.ruby} exec rake i18n:js:export"

-    environment "RAILS_ENV" => "production"
+    environment "HOME" => rails_directory,
+                "RAILS_ENV" => "production"

     cwd rails_directory
     user new_resource.user
     group new_resource.group
     cwd rails_directory
     user new_resource.user
     group new_resource.group

-    notifies :run, "execute[#{rails_directory}/public/assets]"
+    subscribes :run, "git[#{rails_directory}]"
+    only_if { new_resource.build_assets }

   end
 
   execute "#{rails_directory}/public/assets" do
     action :nothing
     command "bundle#{new_resource.ruby} exec rake assets:precompile"
   end
 
   execute "#{rails_directory}/public/assets" do
     action :nothing
     command "bundle#{new_resource.ruby} exec rake assets:precompile"

-    environment "RAILS_ENV" => "production"
+    environment "HOME" => rails_directory,
+                "RAILS_ENV" => "production"

     cwd rails_directory
     user new_resource.user
     group new_resource.group
     cwd rails_directory
     user new_resource.user
     group new_resource.group

-    notifies :restart, "passenger_application[#{rails_directory}]"
+    subscribes :run, "git[#{rails_directory}]"
+    subscribes :run, "file[create:#{rails_directory}/config/application.yml]"
+    subscribes :run, "file[#{rails_directory}/config/settings.local.yml]"
+    subscribes :run, "file[#{rails_directory}/config/storage.yml]"
+    subscribes :run, "file[#{rails_directory}/config/piwik.yml]"
+    subscribes :run, "execute[#{rails_directory}/package.json]"
+    subscribes :run, "execute[#{rails_directory}/app/assets/javascripts/i18n]"
+    only_if { new_resource.build_assets }

   end
 
   file "#{rails_directory}/public/export/embed.html" do
     action :nothing
   end
 
   file "#{rails_directory}/public/export/embed.html" do
     action :nothing

+    subscribes :delete, "git[#{rails_directory}]"
+    subscribes :delete, "file[#{rails_directory}/config/settings.local.yml]"

   end
 
   passenger_application rails_directory do
     action :nothing
   end
 
   passenger_application rails_directory do
     action :nothing

+    subscribes :restart, "git[#{rails_directory}]"
+    subscribes :restart, "file[#{rails_directory}/config/database.yml]"
+    subscribes :restart, "file[create:#{rails_directory}/config/application.yml]"
+    subscribes :restart, "file[#{rails_directory}/config/settings.local.yml]"
+    subscribes :restart, "file[#{rails_directory}/config/storage.yml]"
+    subscribes :restart, "file[#{rails_directory}/config/piwik.yml]"
+    subscribes :restart, "execute[#{rails_directory}/Gemfile]"
+    subscribes :restart, "execute[#{rails_directory}/db/migrate]"
+    subscribes :restart, "execute[#{rails_directory}/package.json]"
+    subscribes :restart, "execute[#{rails_directory}/app/assets/javascripts/i18n]"
+    subscribes :restart, "execute[#{rails_directory}/public/assets]"

     only_if { ::File.exist?("/usr/bin/passenger-config") }
   end
 
     only_if { ::File.exist?("/usr/bin/passenger-config") }
   end