Update thin support to use danieldreier/thin module

Works except for two puppet.conf settings ssl_client_header and ssl_client_verify_header need to be disabled manually post-install
Also gives nginx ssl error on first run but still starts

Author: danieldreier

Modulefile

@@ -16,3 +16,4 @@ dependency 'puppetlabs/inifile',          '>= 1.0.0'
 dependency 'puppetlabs/apache',           '>= 0.9.0'
 dependency 'gentoo/portage',              '>= 2.1.0'
 dependency 'ploperations/nginx'
+dependency 'danieldreier/thin'

manifests/server/thin.pp

@@ -4,6 +4,8 @@
   include puppet::server::rack
 
   class { 'puppet::server::standalone': enabled => false }
+  class { '::thin': }
+  class { 'nginx::server': }
 
   $servers = $::processorcount
   nginx::vhost { 'puppetmaster':
@@ -18,10 +20,13 @@
   }
 
   thin::app { 'puppetmaster':
-    user      => 'puppet',
-    group     => 'puppet',
-    rackup    => "${::puppet::params::puppet_confdir}/config.ru",
-    chdir     => $puppet::params::puppet_confdir,
-    subscribe => Concat["${::puppet::params::puppet_confdir}/config.ru"],
+    user       => 'puppet',
+    group      => 'puppet',
+    rackup     => "${::puppet::params::puppet_confdir}/config.ru",
+    chdir      => $puppet::params::puppet_confdir,
+    subscribe  => Concat["${::puppet::params::puppet_confdir}/config.ru"],
+    require    => Class['::thin'],
+    socket     => '/var/run/thin/puppetmaster.sock',
+    force_home => '/etc/puppet',
   }
 }

spec/acceptance/server_spec.rb

@@ -117,4 +117,43 @@ class { "puppet::server":
     end
   end
 
+  context 'running on thin' do
+    after(:all) do
+      cleanup_server()
+    end
+
+    it 'should run with no errors' do
+      pp = <<-EOS
+        class { "puppet::server":
+          servertype   => 'thin',
+          ca           => true,
+        }
+      EOS
+
+      # Run it twice and test for idempotency
+      apply_manifest(pp, :catch_failures => true)
+      expect(apply_manifest(pp, :catch_failures => true).exit_code).to be_zero
+    end
+
+    describe package('thin') do
+      it {
+        should be_installed.by('gem')
+      }
+    end
+
+    describe service('thin-puppetmaster') do
+      it {
+        should be_enabled
+      }
+      it {
+        should be_running
+      }
+    end
+
+    describe port(8140) do
+      it {
+        should be_listening
+      }
+    end
+  end
 end

spec/spec_helper_acceptance.rb

@@ -51,6 +51,9 @@
 mod 'inifile',        :git => 'git://github.com/puppetlabs/puppetlabs-inifile.git'
 mod 'apache',         :git => 'git://github.com/puppetlabs/puppetlabs-apache.git'
 mod 'portage',        :git => 'git://github.com/gentoo/puppet-portage.git'
+mod 'thin',           :git => 'git://github.com/danieldreier/puppet-thin.git'
+
+
   EOS
   on host, "echo \"#{puppetfile}\" > /etc/puppet/Puppetfile"
   on host, "cd /etc/puppet; r10k puppetfile install"

templates/vhost/nginx/thin.conf.erb

@@ -1,7 +1,7 @@
 # nginx configuration for a puppetmaster running on thin
 
 upstream puppetmaster_upstream {
-<% (0...servers.to_i).each do |count| %>
+<% (0...@servers.to_i).each do |count| %>
   server unix:/var/run/thin/puppetmaster.<%= count %>.sock max_fails=0 fail_timeout=0s;
 <% end -%>
 }

tests/thin.pp

@@ -0,0 +1,4 @@
+class { 'puppet::server':
+  servertype   => 'thin',
+  ca           => true,
+}