Merge pull request #95 from actionjack/syslog-support

Added syslog support.

README.md

@@ -351,6 +351,11 @@ replication configuration. Default: False  *Note*: deprecated – use replica se
 Specify extra configuration file parameters (i.e.
 textSearchEnabled=true). Default: None
 
+#####`syslog`
+Sends all logging output to the host’s syslog system rather than to standard
+output or a log file. Default: None
+*Important*: You cannot use syslog with logpath.
+
 #####`slave`
 Set to true to configure the current instance to act as slave instance in a
 replication configuration. Default: false

manifests/server.pp

@@ -51,6 +51,7 @@
   $slowms          = undef,
   $keyfile         = undef,
   $set_parameter   = undef,
+  $syslog          = undef,
 
   # Deprecated parameters
   $master          = undef,

manifests/server/config.pp

@@ -45,12 +45,15 @@
   $directoryperdb  = $mongodb::server::directoryperdb
   $profile         = $mongodb::server::profile
   $set_parameter   = $mongodb::server::set_parameter
+  $syslog          = $mongodb::server::syslog
 
   File {
     owner => $user,
     group => $group,
   }
 
+  if ($logpath and $syslog) { fail('You cannot use syslog with logpath')}
+
   if ($ensure == 'present' or $ensure == true) {
 
     # Exists for future compatibility and clarity.

spec/classes/server_config_spec.rb

@@ -94,4 +94,23 @@
     end
   end
 
+  describe 'when specifying syslog value' do
+    context 'it should be set to true' do
+      let(:pre_condition) { ["class mongodb::server { $config = '/etc/mongod.conf' $dbpath = '/var/lib/mongo' $ensure = present $syslog = true }", "include mongodb::server"]}
+
+      it {
+        should contain_file('/etc/mongod.conf').with_content(/^syslog = true/)
+      }
+    end
+
+    context 'if logpath is also set an error should be raised' do
+      let(:pre_condition) { ["class mongodb::server { $config = '/etc/mongod.conf' $dbpath = '/var/lib/mongo' $ensure = present $syslog = true $logpath ='/var/log/mongo/mongod.log' }", "include mongodb::server"]}
+
+      it {
+        expect { should contain_file('/etc/mongod.conf') }.to raise_error(Puppet::Error, /You cannot use syslog with logpath/)
+      }
+    end
+
+  end
+
 end

templates/mongodb.conf.erb

@@ -1,8 +1,13 @@
 # mongo.conf - generated from Puppet
 
+
+<% if @logpath -%>
 #where to log
 logpath=<%= @logpath %>
+<% if @logappend -%>
 logappend=<%= @logappend %>
+<% end -%>
+<% end -%>
 <% if @bind_ip -%>
 # Set this option to configure the mongod or mongos process to bind to and
 # listen for connections from applications on this address.
@@ -162,3 +167,6 @@ directoryperdb = <%= @directoryperdb %>
 <% if @set_parameter -%>
 setParameter = <%= @set_parameter %>
 <% end -%>
+<% if @syslog -%>
+syslog = <%= @syslog %>
+<% end -%>