drop a bunch of noops

Signed-off-by: Stephen Gran <steve@lobefin.net>

diff --git a/modules/dacs/manifests/init.pp b/modules/dacs/manifests/init.pp
index 213887a95be592d641f0849b280f2c845d919b72..bd50ca01a824f5477bfad60ef4502d78ab38a4d8 100644 (file)
--- a/modules/dacs/manifests/init.pp
+++ b/modules/dacs/manifests/init.pp
@@ -18,7 +18,6 @@ class dacs {
                owner => root,
                group => www-data,
                mode  => '0640',
-               noop  => true,
        }
 
        file { '/var/log/dacs':

diff --git a/modules/vsftpd/manifests/nolisten.pp b/modules/vsftpd/manifests/nolisten.pp
index 22951f222180748f1be921b56e3c875b2cd0caf6..78fcd008a8b5aed4979a47c599e488745a0ddebe 100644 (file)
--- a/modules/vsftpd/manifests/nolisten.pp
+++ b/modules/vsftpd/manifests/nolisten.pp
@@ -1,32 +1,12 @@
 class vsftpd::nolisten inherits vsftpd {
 
-       $noop = $::hostname ? {
-               bizet     => false,
-               franck    => false,
-               gluck     => false,
-               lobos     => false,
-               kassia    => false,
-               klecker   => false,
-               ravel     => false,
-               saens     => false,
-               santoro   => false,
-               schein    => false,
-               steffani  => false,
-               villa     => false,
-               wieck     => false,
-               morricone => false,
-               default   => true
-       }
-
        Service['vsftpd'] {
                ensure => stopped,
-               noop   => $noop,
        }
 
        Service['vsftpd']->Service['xinetd']
 
        file { '/etc/vsftpd.conf':
-               noop    => $noop,
                content => "listen=NO\n",
                notify  => Service['vsftpd']
        }

diff --git a/modules/vsftpd/manifests/site.pp b/modules/vsftpd/manifests/site.pp
index a2ca6cb29e2c3f91ad643853fa325af83c70897f..d0129e1dfc42cd15f0c176a05ac5ea5180c091d7 100644 (file)
--- a/modules/vsftpd/manifests/site.pp
+++ b/modules/vsftpd/manifests/site.pp
@@ -19,34 +19,14 @@ define vsftpd::site (
 
        $fname = "/etc/vsftpd-${name}.conf"
 
-       $noop = $::hostname ? {
-               bizet     => false,
-               franck    => false,
-               gluck     => false,
-               lobos     => false,
-               kassia    => false,
-               klecker   => false,
-               ravel     => false,
-               saens     => false,
-               santoro   => false,
-               schein    => false,
-               steffani  => false,
-               villa     => false,
-               wieck     => false,
-               morricone => false,
-               default   => true
-       }
-
        if $source {
                file { $fname:
                        ensure => $ensure,
-                       noop   => $noop,
                        source => $source,
                }
        } elsif $content {
                file { $fname:
                        ensure  => $ensure,
-                       noop    => $noop,
                        content => $content,
                }
        } else {

diff --git a/modules/xinetd/manifests/init.pp b/modules/xinetd/manifests/init.pp
index 79e23590d87b6eee9e7b524e767a92627d7fd649..d85ab700eb3dfa6c8369b3f1788796b1bb0b7b9e 100644 (file)
--- a/modules/xinetd/manifests/init.pp
+++ b/modules/xinetd/manifests/init.pp
@@ -1,14 +1,12 @@
 class xinetd {
        package { 'xinetd':
                ensure => installed,
-               noop   => true,
        }
 
        service { 'xinetd':
                ensure    => running,
                hasstatus => false,
                pattern   => '/usr/sbin/xinetd',
-               noop      => true,
                require   => Package['xinetd']
        }
 }

diff --git a/modules/xinetd/manifests/service.pp b/modules/xinetd/manifests/service.pp
index f7c966af6d2506b118600ee7eaa7008e967b9ba1..d8121800529a17f9cb357583ac0495d5939e52ce 100644 (file)
--- a/modules/xinetd/manifests/service.pp
+++ b/modules/xinetd/manifests/service.pp
@@ -18,24 +18,6 @@ define xinetd::service (
 ) {
        include xinetd
 
-       $noop = $::hostname ? {
-               bizet     => false,
-               franck    => false,
-               gluck     => false,
-               lobos     => false,
-               kassia    => false,
-               klecker   => false,
-               ravel     => false,
-               saens     => false,
-               santoro   => false,
-               schein    => false,
-               steffani  => false,
-               villa     => false,
-               wieck     => false,
-               morricone => false,
-               default   => true
-       }
-
        case $ensure {
                present,absent,file: {}
                default: { fail("Invalid ensure for '$name'") }
@@ -50,7 +32,6 @@ define xinetd::service (
 
        file { "/etc/xinetd.d/${name}":
                ensure  => $ensure,
-               noop    => $noop,
                content => template('xinetd/service.erb'),
                notify  => Service['xinetd'],
                require => Package['xinetd'],