diff options
Diffstat (limited to 'manifests')
| -rw-r--r-- | manifests/init.pp | 47 |
1 files changed, 19 insertions, 28 deletions
diff --git a/manifests/init.pp b/manifests/init.pp index d1d9fce..d23f4a3 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -58,7 +58,6 @@ class ntp($servers='UNSET', case $::operatingsystem { debian, ubuntu: { - $supported = true $pkg_name = [ 'ntp' ] $svc_name = 'ntp' $config = '/etc/ntp.conf' @@ -73,7 +72,6 @@ class ntp($servers='UNSET', } } centos, redhat, oel, linux, fedora, Amazon: { - $supported = true $pkg_name = [ 'ntp' ] $svc_name = 'ntpd' $config = '/etc/ntp.conf' @@ -87,7 +85,6 @@ class ntp($servers='UNSET', } } freebsd: { - $supported = true $pkg_name = ['.*/net/ntp'] $svc_name = 'ntpd' $config = '/etc/ntp.conf' @@ -102,35 +99,29 @@ class ntp($servers='UNSET', } } default: { - $supported = false - notify { "${module_name}_unsupported": - message => "The ${module_name} module is not supported on ${::operatingsystem}", - } + fail("The ${module_name} module is not supported on ${::operatingsystem}") } } - if ($supported == true) { - - package { 'ntp': - ensure => $package_ensure, - name => $pkg_name, - } + package { 'ntp': + name => $pkg_name, + ensure => $package_ensure, + } - file { $config: - ensure => file, - owner => 0, - group => 0, - mode => '0644', - content => template("${module_name}/${config_tpl}"), - require => Package[$pkg_name], - } + file { $config: + ensure => file, + owner => 0, + group => 0, + mode => '0644', + content => template("${module_name}/${config_tpl}"), + require => Package[$pkg_name], + } - service { 'ntp': - ensure => $ensure, - name => $svc_name, - hasstatus => true, - hasrestart => true, - subscribe => [ Package[$pkg_name], File[$config] ], - } + service { 'ntp': + ensure => $ensure, + name => $svc_name, + hasstatus => true, + hasrestart => true, + subscribe => [ Package[$pkg_name], File[$config] ], } } |