Switch to using concat{} instead of lots of file{} magic.

.gitignore

@@ -4,4 +4,5 @@ pkg/
 .librarian/
 .tmp/
 pkg/
+Gemfile.lock
 spec/fixtures/

.nodeset.yml

@@ -29,7 +29,3 @@ sets:
     nodes:
       "main.foo.vm":
         prefab: 'ubuntu-server-12042-x64'
-  'sles-11sp1-x64':
-    nodes:
-      "main.foo.vm":
-        prefab: 'sles-11sp1-x64'

Gemfile

@@ -9,6 +9,6 @@ group :rake do
   gem 'puppet-blacksmith'
   gem 'librarian-puppet-maestrodev'
   gem 'rspec-system-puppet',     :require => false
-  gem 'serverspec',              :require => false
+  gem 'serverspec',   '~> 0.11.0', :require => false
   gem 'rspec-system-serverspec', :require => false
 end

Modulefile

@@ -9,3 +9,4 @@ project_page 'http://github.com/jfryman/puppet-nginx'
 
 dependency 'puppetlabs/stdlib', '>= 0.1.6'
 dependency 'puppetlabs/apt', '>= 1.0.0'
+dependency 'puppetlabs/concat', '>= 1.0.0'

Puppetfile

@@ -2,3 +2,4 @@ forge 'http://forge.puppetlabs.com'
 
 mod 'puppetlabs/stdlib', '>=0.1.6'
 mod 'puppetlabs/apt', '>=1.0.0'
+mod 'puppetlabs/concat', '>=1.0.0'

Puppetfile.lock

@@ -3,9 +3,11 @@ FORGE
   specs:
     puppetlabs/apt (1.2.0)
       puppetlabs/stdlib (>= 2.2.1)
+    puppetlabs/concat (1.0.0)
     puppetlabs/stdlib (4.1.0)
 
 DEPENDENCIES
   puppetlabs/apt (>= 1.0.0)
+  puppetlabs/concat (>= 1.0.0)
   puppetlabs/stdlib (>= 0.1.6)
 

manifests/config.pp

@@ -49,7 +49,6 @@
   }
   if $confd_purge == true {
     File["${nginx::params::nx_conf_dir}/conf.d"] {
-      ignore  => 'vhost_autogen.conf',
       purge   => true,
       recurse => true,
     }
@@ -60,12 +59,19 @@
   }
   if $confd_purge == true {
     File["${nginx::params::nx_conf_dir}/conf.mail.d"] {
-      ignore  => 'vhost_autogen.conf',
       purge   => true,
       recurse => true,
     }
   }
 
+  file { "${nginx::params::nx_conf_dir}/conf.d/vhost_autogen.conf":
+    ensure => absent,
+  }
+
+  file { "${nginx::params::nx_conf_dir}/conf.mail.d/vhost_autogen.conf":
+    ensure => absent,
+  }
+
   file {$nginx::config::nx_run_dir:
     ensure => directory,
   }
@@ -80,6 +86,14 @@
     owner  => $nginx::params::nx_daemon_user,
   }
 
+  file { "${nginx::params::nx_conf_dir}/sites-available":
+    ensure => directory,
+  }
+
+  file { "${nginx::params::nx_conf_dir}/sites-enabled":
+    ensure => directory,
+  }
+
   file { '/etc/nginx/sites-enabled/default':
     ensure => absent,
   }
@@ -94,15 +108,4 @@
     content => template('nginx/conf.d/proxy.conf.erb'),
   }
 
-  file { "${nginx::config::nx_temp_dir}/nginx.d":
-    ensure  => directory,
-    purge   => true,
-    recurse => true,
-  }
-
-  file { "${nginx::config::nx_temp_dir}/nginx.mail.d":
-    ensure  => directory,
-    purge   => true,
-    recurse => true,
-  }
 }

manifests/params.pp

@@ -14,7 +14,6 @@
 #
 # This class file is not called directly
 class nginx::params {
-  $nx_temp_dir                = '/tmp'
   $nx_run_dir                 = '/var/nginx'
 
   $nx_conf_dir                = '/etc/nginx'

manifests/resource/location.pp

@@ -129,6 +129,7 @@
     'absent' => absent,
     default  => file,
   }
+  $config_file = "${nginx::config::nx_conf_dir}/sites-available/${vhost}.conf"
 
   $location_sanitized = regsubst($location, '\/', '_', 'G')
 
@@ -168,18 +169,20 @@
 
   ## Create stubs for vHost File Fragment Pattern
   if ($ssl_only != true) {
-    file {"${nginx::config::nx_temp_dir}/nginx.d/${vhost}-${priority}-${location_sanitized}":
-      ensure  => $ensure_real,
+    concat::fragment { "${vhost}-${priority}-${location_sanitized}":
+      target  => $config_file,
       content => $content_real,
+      order   => $priority,
     }
   }
 
   ## Only create SSL Specific locations if $ssl is true.
   if ($ssl == true) {
     $ssl_priority = $priority + 300
-    file {"${nginx::config::nx_temp_dir}/nginx.d/${vhost}-${ssl_priority}-${location_sanitized}-ssl":
-      ensure  => $ensure_real,
+    concat::fragment {"${vhost}-${ssl_priority}-${location_sanitized}-ssl":
+      target  => $config_file,
       content => $content_real,
+      order   => $ssl_priority,
     }
   }
 

manifests/resource/mailhost.pp

@@ -67,6 +67,13 @@
 
   validate_array($server_name)
 
+  $file_ensure = $ensure ? {
+    'absent' => absent,
+    default  => 'file',
+  }
+
+  $config_file = "${nginx::config::nx_conf_dir}/conf.mail.d/${name}.conf"
+
   # Add IPv6 Logic Check - Nginx service will not start if ipv6 is enabled
   # and support does not exist for it in the kernel.
   if ($ipv6_enable and !$::ipaddress6) {
@@ -80,28 +87,29 @@
     }
   }
 
-  # Use the File Fragment Pattern to construct the configuration files.
-  # Create the base configuration file reference.
+  concat { $config_file:
+    # Waiting on https://github.com/puppetlabs/puppetlabs-concat/pull/39/files
+    #ensure => $file_ensure,
+    owner  => 'root',
+    group  => 'root',
+    mode   => '0644',
+    notify => Class['nginx::service'],
+  }
+
   if ($listen_port != $ssl_port) {
-    file { "${nginx::config::nx_temp_dir}/nginx.mail.d/${name}-001":
-      ensure  => $ensure ? {
-        'absent' => absent,
-        default  => 'file',
-      },
+    concat::fragment { "${name}-header":
+      target  => $config_file,
       content => template('nginx/mailhost/mailhost.erb'),
-      notify  => Class['nginx::service'],
+      order   => '001',
     }
   }
 
   # Create SSL File Stubs if SSL is enabled
   if ($ssl) {
-    file { "${nginx::config::nx_temp_dir}/nginx.mail.d/${name}-700-ssl":
-      ensure  => $ensure ? {
-        'absent' => absent,
-        default  => 'file',
-      },
+    concat::fragment { "${name}-ssl":
+      target  => $config_file,
       content => template('nginx/mailhost/mailhost_ssl.erb'),
-      notify  => Class['nginx::service'],
+      order   => '700',
     }
   }
 }