From 171b4d3546b2c6bdcbd1495709a6d124d9efee0f Mon Sep 17 00:00:00 2001
From: John Bintz <john@coswellproductions.com>
Date: Mon, 14 May 2012 11:07:28 -0400
Subject: [PATCH] clean up varnish config

---
 .../varnish/manifests/debian.pp               | 29 ++++++++++++++-----
 .../varnish/templates/debian/init-d-prerun    |  4 +--
 2 files changed, 23 insertions(+), 10 deletions(-)

diff --git a/shared/additional-modules/varnish/manifests/debian.pp b/shared/additional-modules/varnish/manifests/debian.pp
index 51d7312..680b73e 100644
--- a/shared/additional-modules/varnish/manifests/debian.pp
+++ b/shared/additional-modules/varnish/manifests/debian.pp
@@ -1,21 +1,34 @@
-class varnish::debian($version, $vcl_template) {
-  $user = 'varnish'
-  $group = 'varnish'
+class varnish::debian($version, $vcl_template, $store_file_mb = 1024) {
+  $varnish_user = 'varnish'
+  $varnish_group = 'varnish'
 
-  user { $user: uid => 27835 }
+  user { $varnish_user: uid => 27835 }
 
-  package { 'libpcre3-dev': ensure => installed }
+  $packages = [ 'libpcre3', 'libpcre3-dev', 'pkg-config' ]
+
+  package { $packages: ensure => installed }
 
   class { 'varnish':
     version => $version,
-    require => Package['libpcre3-dev'],
+    require => Package[$packages],
     vcl_template => $vcl_template
   }
 
-  init_d { 'varnish':
-    init_d_prerun => template('varnish/debian/init-d-prerun'),
+  init_d_bundle { 'varnish':
     init_d_prolog => template('varnish/debian/init-d-prolog'),
+    init_d_prerun => template('varnish/debian/init-d-prerun'),
     require => Class['varnish']
   }
+
+  exec { 'ensure-data-store-ownership':
+    command => "chown -R ${varnish_user}:${varnish_group} ${varnish::data}",
+    path => $base::path,
+    require => Class['varnish']
+  }
+
+  logrotate_d { 'varnishncsa':
+    postrotate => 'service varnish rotate',
+    pattern => "${varnish::log}/access.log"
+  }
 }
 
diff --git a/shared/additional-modules/varnish/templates/debian/init-d-prerun b/shared/additional-modules/varnish/templates/debian/init-d-prerun
index 8d9cb86..611dccd 100644
--- a/shared/additional-modules/varnish/templates/debian/init-d-prerun
+++ b/shared/additional-modules/varnish/templates/debian/init-d-prerun
@@ -4,5 +4,5 @@ ulimit -n ${NFILES:-131072}
 # Maxiumum locked memory size for shared memory log
 ulimit -l ${MEMLOCK:-82000}
 
-USER=<%= user %>
-GROUP=<%= group %>
+USER=<%= varnish_user %>
+GROUP=<%= varnish_group %>