Set action to "nothing" explicitly when using passenger_application
authorTom Hughes <tom@compton.nu>
Mon, 24 Sep 2018 11:42:04 +0000 (12:42 +0100)
committerTom Hughes <tom@compton.nu>
Mon, 24 Sep 2018 11:42:04 +0000 (12:42 +0100)
Apparently have "nothing" as the default action for a custom resource
doesn't actually work, even if you define it explicitly.

cookbooks/passenger/resources/application.rb
cookbooks/taginfo/recipes/default.rb
cookbooks/web/resources/rails_port.rb

index 1beb4bd..ce0c0dc 100644 (file)
 # limitations under the License.
 #
 
-default_action :nothing
+default_action :restart
 
 property :application, String, :name_attribute => true
 
-action :nothing do
-end
-
 action :restart do
   execute new_resource.application do
     action :run
index 19c467c..d55add7 100644 (file)
@@ -172,7 +172,9 @@ node[:taginfo][:sites].each do |site|
     variables :name => site_name, :directory => directory
   end
 
-  passenger_application "#{directory}/taginfo/web/public"
+  passenger_application "#{directory}/taginfo/web/public" do
+    action :nothing
+  end
 
   ssl_certificate site_name do
     domains [site_name] + site_aliases
index 067c133..913f0a1 100644 (file)
@@ -346,6 +346,7 @@ action :create do
   end
 
   passenger_application rails_directory do
+    action :nothing
     only_if { ::File.exist?("/usr/bin/passenger-config") }
   end