Added possibility to define package state

manifests/base.pp

@@ -3,9 +3,16 @@
 # Examples
 #
 #   include logrotate::base
-class logrotate::base {
+class logrotate::base (
+  $compress      = true,
+  $compresscmd   = undef,
+  $compressext   = undef,
+  $dateext       = false,
+  $package_state = 'present',
+  $rotate        = 4
+){
   package { 'logrotate':
-    ensure => latest,
+    ensure => $package_state,
   }
 
   File {
@@ -18,7 +25,7 @@
     '/etc/logrotate.conf':
       ensure  => file,
       mode    => '0444',
-      source  => 'puppet:///modules/logrotate/etc/logrotate.conf';
+      content => template('logrotate/etc/logrotate.conf.erb'),;
     '/etc/logrotate.d':
       ensure  => directory,
       mode    => '0755';

spec/classes/base_spec.rb

@@ -2,7 +2,7 @@
 
 describe 'logrotate::base' do
   it do
-    should contain_package('logrotate').with_ensure('latest')
+    should contain_package('logrotate').with_ensure('present')
 
     should contain_file('/etc/logrotate.conf').with({
       'ensure'  => 'file',

templates/etc/logrotate.conf.erb

@@ -0,0 +1,33 @@
+# THIS FILE IS AUTOMATICALLY DISTRIBUTED BY PUPPET.  ANY CHANGES WILL BE
+# OVERWRITTEN.
+
+# Default values
+# rotate log files weekly
+weekly
+
+# keep X weeks worth of backlogs
+rotate <%= @rotate %>
+
+# create new (empty) log files after rotating old ones
+create
+
+<% if @compress -%>
+compress
+<%- end -%>
+
+<% if @compresscmd -%>
+compresscmd <%= @compresscmd %>
+<%- end -%>
+
+<% if @compressext -%>
+compressext <%= @compressext %>
+<%- end -%>
+
+<% if @dateext -%>
+# use date as a suffix of the rotated file
+dateext
+<%- end -%>
+
+
+# packages drop log rotation information into this directory
+include /etc/logrotate.d