add pid & socket as options. Make pid & log_file optional. Don't create service files when not managed. Apps don't force reload of emperor as the emperor auto reloads the vassals.

Author: elmerfud

manifests/app.pp

@@ -44,7 +44,6 @@
         group   => $gid,
         mode    => '0644',
         content => template($template),
-        require => Package[$::uwsgi::package_name],
-        notify  => Service[$::uwsgi::service_name]
+        require => Package[$::uwsgi::package_name]
     }
 }

manifests/init.pp

@@ -98,6 +98,8 @@
     $install_python_dev  = $uwsgi::params::install_python_dev,
     $python_pip          = $uwsgi::params::python_pip,
     $python_dev          = $uwsgi::params::python_dev,
+    $pidfile             = $uwsgi::params::pidfile,
+    $socket              = $uwsgi::params::socket,
     $emperor_options     = undef
 ) inherits uwsgi::params {
 
@@ -136,44 +138,49 @@
         require => Package[$package_name]
     }
 
-    if $service_file == undef {
-        $service_file_real = $service_provider ? {
-            redhat  => '/etc/init.d/uwsgi',
-            upstart => '/etc/init/uwsgi.conf',
-            default => '/etc/init/uwsgi.conf',
-        }
-    } else {
-        $service_file_real = $service_file
-    }
+    if $manage_service_file == true {
+      if $service_file == undef {
+          $service_file_real = $service_provider ? {
+              redhat  => '/etc/init.d/uwsgi',
+              upstart => '/etc/init/uwsgi.conf',
+              default => '/etc/init/uwsgi.conf',
+          }
+      } else {
+          $service_file_real = $service_file
+      }
 
-    if $service_file_mode == undef {
-        $service_file_mode_real = $service_provider ? {
-            redhat  => '0555',
-            upstart => '0644',
-            default => '0644',
-        }
-    } else {
-        $service_file_mode_real = $service_file_mode
-    }
+      if $service_file_mode == undef {
+          $service_file_mode_real = $service_provider ? {
+              redhat  => '0555',
+              upstart => '0644',
+              default => '0644',
+          }
+      } else {
+          $service_file_mode_real = $service_file_mode
+      }
 
-    if $service_template == undef {
-        $service_template_real = $service_provider ? {
-            redhat  => 'uwsgi/uwsgi_service-redhat.erb',
-            upstart => 'uwsgi/uwsgi_upstart.conf.erb',
-            default => 'uwsgi/uwsgi_upstart.conf.erb',
-        }
-    } else {
-        $service_template_real = $service_template
-    }
+      if $service_template == undef {
+          $service_template_real = $service_provider ? {
+              redhat  => 'uwsgi/uwsgi_service-redhat.erb',
+              upstart => 'uwsgi/uwsgi_upstart.conf.erb',
+              default => 'uwsgi/uwsgi_upstart.conf.erb',
+          }
+      } else {
+          $service_template_real = $service_template
+      }
 
-    file { $service_file_real:
-        ensure   => $file_ensure,
-        owner    => 'root',
-        group    => 'root',
-        mode     => $service_file_mode_real,
-        replace  => $manage_service_file,
-        content  => template($service_template_real),
-        require  => Package[$package_name]
+      file { $service_file_real:
+          ensure   => $file_ensure,
+          owner    => 'root',
+          group    => 'root',
+          mode     => $service_file_mode_real,
+          replace  => $manage_service_file,
+          content  => template($service_template_real),
+          require  => Package[$package_name]
+      }
+      $required_files = [ $config_file, $service_file_real ]
+    } else {
+      $required_files = $config_file
     }
 
     file { $app_directory:
@@ -192,13 +199,9 @@
         provider   => $service_provider,
         require    => [
             Package[$package_name],
-            File[$config_file],
-            File[$service_file_real]
+            File[$required_files]
             ],
-        subscribe  => [
-            File[$config_file],
-            File[$service_file_real]
-            ]
+        subscribe  => File[$required_files]
     }
 
     # finally, configure any applications necessary

templates/uwsgi.ini.erb

@@ -4,14 +4,18 @@
 # uWSGI main process configuration
 #
 [uwsgi]
+<% if @socket -%>
 socket = <%= @socket %>
+<% end -%>
 pidfile = <%= @pidfile %>
 emperor = <%= @app_directory %>
 emperor-tyrant = true
 master = true
 autoload = true
 log-date = true
+<% if @log_file -%>
 logto = <%= @log_file %>
+<% end -%>
 <%
 if @emperor_options
   @emperor_options.sort.each do |key, value|