Work for SLES 12 compatibility

Author: Helen Campbell

README.markdown

@@ -302,6 +302,10 @@ Tells Puppet whether to manage the NTP service. Valid options: 'true' or 'false'
 
 Tells Puppet what NTP service to manage. Valid options: string. Default value: varies by operating system
 
+####`service_provider`
+
+Tells Puppet which service provider to use for NTP. Valid options: string. Default value: 'undef'
+
 ####`stepout`
 
 Tells puppet to change stepout. Applies only if `tinker` value is 'true'. Valid options: unsigned shortint digit. Default value: undef.

manifests/config.pp

@@ -1,6 +1,13 @@
 # Private Class
 class ntp::config inherits ntp {
 
+  #The servers-netconfig file overrides NTP config on SLES 12, interfering with our configuration.
+  if $::operatingsystem == 'SLES' and $::operatingsystemmajrelease == '12' {
+    file { '/var/run/ntp/servers-netconfig':
+      ensure => 'absent'
+    }
+  }
+
   if $ntp::keys_enable {
     case $ntp::config_dir {
       '/', '/etc', undef: {}

manifests/init.pp

@@ -34,6 +34,7 @@
   $service_ensure    = $ntp::params::service_ensure,
   $service_manage    = $ntp::params::service_manage,
   $service_name      = $ntp::params::service_name,
+  $service_provider  = $ntp::params::service_provider,
   $stepout           = $ntp::params::stepout,
   $tinker            = $ntp::params::tinker,
   $tos               = $ntp::params::tos,

manifests/params.pp

@@ -156,6 +156,8 @@
               $service_name  = 'ntpd'
               $keys_file     = '/etc/ntp.keys'
               $package_name  = $default_package_name
+              #Puppet 3 does not recognise systemd as service provider on SLES 12.
+              $service_provider = 'systemd'
             }
             default: {
               fail("The ${module_name} module is not supported on an ${::operatingsystem} ${::operatingsystemmajrelease} distribution.")

manifests/service.pp

@@ -10,6 +10,7 @@
       ensure     => $ntp::service_ensure,
       enable     => $ntp::service_enable,
       name       => $ntp::service_name,
+      provider   => $ntp::service_provider,
       hasstatus  => true,
       hasrestart => true,
     }