Merge pull request #128 from apenney/disable_monitor

Add support for `disable_monitor` as a new parameter.

Author: Ashley Penney

README.markdown

@@ -120,6 +120,10 @@ Sets the file that ntp configuration is written into.
 
 Determines which template Puppet should use for the ntp configuration.
 
+####`disable_monitor`
+
+Disables monitoring of ntp.
+
 ####`driftfile`
 
 Sets the location of the drift file for ntp.

manifests/init.pp

@@ -2,6 +2,7 @@
   $autoupdate        = $ntp::params::autoupdate,
   $config            = $ntp::params::config,
   $config_template   = $ntp::params::config_template,
+  $disable_monitor   = $ntp::params::disable_monitor,
   $driftfile         = $ntp::params::driftfile,
   $keys_enable       = $ntp::params::keys_enable,
   $keys_file         = $ntp::params::keys_file,
@@ -23,6 +24,7 @@
 
   validate_absolute_path($config)
   validate_string($config_template)
+  validate_bool($disable_monitor)
   validate_absolute_path($driftfile)
   validate_bool($keys_enable)
   validate_re($keys_controlkey, ['^\d+$', ''])

manifests/params.pp

@@ -2,6 +2,7 @@
 
   $autoupdate        = false
   $config_template   = 'ntp/ntp.conf.erb'
+  $disable_monitor   = false
   $keys_enable       = false
   $keys_controlkey   = ''
   $keys_requestkey   = ''

spec/acceptance/disable_monitoring_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper_acceptance'
+
+describe "ntp class with disable_monitor:" do
+  context 'should run successfully' do
+    pp = "class { 'ntp': disable_monitor => true }"
+
+    it 'runs twice' do
+      apply_manifest(pp, :catch_failures => true)
+      apply_manifest(pp, :catch_changes => true)
+    end
+
+    describe file('/etc/ntp.conf') do
+      it { should contain('disable monitor') }
+    end
+  end
+
+  context 'should run successfully' do
+    pp = "class { 'ntp': disable_monitor => false }"
+
+    it 'runs twice' do
+      apply_manifest(pp, :catch_failures => true)
+      apply_manifest(pp, :catch_changes => true)
+    end
+
+    describe file('/etc/ntp.conf') do
+      it { should_not contain('disable monitor') }
+    end
+  end
+
+end

templates/ntp.conf.erb

@@ -6,6 +6,10 @@
 tinker panic 0
 <% end -%>
 
+<% if @disable_monitor == true -%>
+disable monitor
+<% end -%>
+
 <% if @restrict != [] -%>
 # Permit time synchronization with our time source, but do not'
 # permit the source to query or modify the service on this system.'
@@ -40,4 +44,5 @@ requestkey <%= @keys_requestkey %>
 <% if @keys_controlkey != '' -%>
 controlkey <%= @keys_controlkey %>
 <% end -%>
+
 <% end -%>