make the lint gods happy

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

diff --git a/modules/bacula/manifests/storage.pp b/modules/bacula/manifests/storage.pp
index 7b1c1a8341a79ced3333f645a5fb1095758b8dde..5a0132d31352dd15ab81839c17f5a682e32657ea 100644 (file)
--- a/modules/bacula/manifests/storage.pp
+++ b/modules/bacula/manifests/storage.pp
@@ -15,7 +15,7 @@ class bacula::storage inherits bacula {
        # jobs
        exec { 'bacula-sd reload':
                path        => '/usr/bin:/usr/sbin:/bin:/sbin',
-               command     => 'true',
+               command     => '/bin/true',
                refreshonly => true,
        }
 

diff --git a/modules/munin/manifests/init.pp b/modules/munin/manifests/init.pp
index 8258f9b13248eab8ee0c2ae91971f0ffa141f42b..9684a2944424c25ab09ebcf5734217c0e0cf1a73 100644 (file)
--- a/modules/munin/manifests/init.pp
+++ b/modules/munin/manifests/init.pp
@@ -62,12 +62,12 @@ class munin {
 
        @@munin::master-per-node {
                $::fqdn:
-                       ipaddress => $::ipaddress,
+                       ipaddress   => $::ipaddress,
                        munin_async => $::munin_async,
                        ;
        }
 
-       if $munin_async and $munin_async == 'true' {
+       if $::munin_async and str2bool($::munin_async) == true {
                file { '/etc/ssh/userkeys/munin-async':
                        source => 'puppet:///modules/munin/munin-async-authkeys',
                }

diff --git a/modules/roles/manifests/static_base.pp b/modules/roles/manifests/static_base.pp
index 267c8ba61f64e47550df972dad06642e369c066b..e062a9e5191425891bc86ef57dda1c0ad139d07a 100644 (file)
--- a/modules/roles/manifests/static_base.pp
+++ b/modules/roles/manifests/static_base.pp
@@ -1,17 +1,12 @@
 class roles::static_base {
-    if ! $::staticsync_key {
-        exec { 'create-staticsync-key':
-            command => '/bin/su - staticsync -c \'mkdir -p -m 02700 .ssh && ssh-keygen -C "`whoami`@`hostname` (`date +%Y-%m-%d`)" -P "" -f .ssh/id_rsa -q\'',
-            onlyif  => '/usr/bin/getent passwd staticsync > /dev/null && ! [ -e /home/staticsync/.ssh/id_rsa ]'
-        }
-    }
+       if ! $::staticsync_key {
+               exec { 'create-staticsync-key':
+                       command => '/bin/su - staticsync -c \'mkdir -p -m 02700 .ssh && ssh-keygen -C "`whoami`@`hostname` (`date +%Y-%m-%d`)" -P "" -f .ssh/id_rsa -q\'',
+                       onlyif  => '/usr/bin/getent passwd staticsync > /dev/null && ! [ -e /home/staticsync/.ssh/id_rsa ]'
+               }
+       }
 
-    file {
-        '/etc/static-components.conf':
-            source  => "puppet:///modules/roles/static-mirroring/static-components.conf",
-            ;
-    }
+       file { '/etc/static-components.conf':
+               source => 'puppet:///modules/roles/static-mirroring/static-components.conf',
+       }
 }
-# vim:set et:
-# vim:set sts=4 ts=4:
-# vim:set shiftwidth=4:

diff --git a/modules/roles/manifests/static_master.pp b/modules/roles/manifests/static_master.pp
index df11d6a4231bccbbd87c70f157d3c1eb336cf840..0816754df828fb73a4793aeb51cfd1c7e48a578b 100644 (file)
--- a/modules/roles/manifests/static_master.pp
+++ b/modules/roles/manifests/static_master.pp
@@ -1,26 +1,23 @@
-class roles::static_master inherits roles::static_base {
-    file {
-        '/etc/ssh/userkeys/staticsync':
-            content => template('roles/static-mirroring/static-master-authorized_keys.erb'),
-            ;
-        '/usr/local/bin/static-master-run':
-            source  => "puppet:///modules/roles/static-mirroring/static-master-run",
-            mode => 555,
-            ;
-        '/usr/local/bin/static-master-ssh-wrap':
-            source  => "puppet:///modules/roles/static-mirroring/static-master-ssh-wrap",
-            mode => 555,
-            ;
-        '/usr/local/bin/static-master-update-component':
-            source  => "puppet:///modules/roles/static-mirroring/static-master-update-component",
-            mode => 555,
-            ;
+class roles::static_master {
 
-        '/etc/static-clients.conf':
-            content => template('roles/static-mirroring/static-clients.conf.erb'),
-            ;
-    }
+       include roles::static_base
+
+       file { '/etc/ssh/userkeys/staticsync':
+               content => template('roles/static-mirroring/static-master-authorized_keys.erb'),
+       }
+       file { '/usr/local/bin/static-master-run':
+               source => 'puppet:///modules/roles/static-mirroring/static-master-run',
+               mode   => '0555',
+       }
+       file {'/usr/local/bin/static-master-ssh-wrap':
+               source => 'puppet:///modules/roles/static-mirroring/static-master-ssh-wrap',
+               mode   => '0555',
+       }
+       file { '/usr/local/bin/static-master-update-component':
+               source => 'puppet:///modules/roles/static-mirroring/static-master-update-component',
+               mode   => '0555',
+       }
+       file { '/etc/static-clients.conf':
+               content => template('roles/static-mirroring/static-clients.conf.erb'),
+       }
 }
-# vim:set et:
-# vim:set sts=4 ts=4:
-# vim:set shiftwidth=4:

diff --git a/modules/roles/manifests/static_mirror.pp b/modules/roles/manifests/static_mirror.pp
index 60aa07434a6493011fb865e1dca55133399f55b8..1aa20d773c45254d464822ecfc3857161f8f03f3 100644 (file)
--- a/modules/roles/manifests/static_mirror.pp
+++ b/modules/roles/manifests/static_mirror.pp
@@ -1,41 +1,38 @@
-class roles::static_mirror inherits roles::static_source {
-    include apache2
-
-    package { 'libapache2-mod-macro':
-        ensure => installed,
-    }
-
-    apache2::module { 'macro': require => Package['libapache2-mod-macro']; }
-    apache2::module { 'rewrite': }
-    apache2::module { 'expires': }
-
-    file {
-        '/usr/local/bin/static-mirror-run':
-            source  => "puppet:///modules/roles/static-mirroring/static-mirror-run",
-            mode => 555,
-            ;
-
-        "/srv/static.debian.org":
-            ensure  => directory,
-            owner   => staticsync,
-            group   => staticsync,
-            mode    => '02755'
-            ;
-        "/etc/cron.d/puppet-static-mirror":
-            content => "PATH=/usr/local/bin:/usr/bin:/bin\n@reboot staticsync sleep 60; static-mirror-run --one-stage /srv/static.debian.org bizet.debian.org:-live- > /dev/null\n",
-            ;
-    }
-
-    $vhost_listen = $::hostname ? {
-        klecker => '130.89.148.14:80 [2001:610:1908:b000::148:14]:80',
-        default => '*:80',
-    }
-
-    apache2::site { '010-planet.debian.org':
-        site   => 'planet.debian.org',
-        content => template('roles/static-mirroring/vhost/planet.debian.org.erb'),
-    }
+class roles::static_mirror {
+
+       include roles::static_source
+
+       package { 'libapache2-mod-macro':
+               ensure => installed,
+       }
+
+       apache2::module { 'macro': require => Package['libapache2-mod-macro']; }
+       apache2::module { 'rewrite': }
+       apache2::module { 'expires': }
+
+       file { '/usr/local/bin/static-mirror-run':
+               source => 'puppet:///modules/roles/static-mirroring/static-mirror-run',
+               mode   => '0555',
+       }
+
+       file { '/srv/static.debian.org':
+               ensure => directory,
+               owner  => staticsync,
+               group  => staticsync,
+               mode   => '02755'
+       }
+
+       file { '/etc/cron.d/puppet-static-mirror':
+                       content => "PATH=/usr/local/bin:/usr/bin:/bin\n@reboot staticsync sleep 60; static-mirror-run --one-stage /srv/static.debian.org bizet.debian.org:-live- > /dev/null\n",
+       }
+
+       $vhost_listen = $::hostname ? {
+               klecker => '130.89.148.14:80 [2001:610:1908:b000::148:14]:80',
+               default => '*:80',
+       }
+
+       apache2::site { '010-planet.debian.org':
+               site    => 'planet.debian.org',
+               content => template('roles/static-mirroring/vhost/planet.debian.org.erb'),
+       }
 }
-# vim:set et:
-# vim:set sts=4 ts=4:
-# vim:set shiftwidth=4:

diff --git a/modules/roles/manifests/static_source.pp b/modules/roles/manifests/static_source.pp
index e44624b99198fa4145c7b06f2d44139d48cd6440..821a6cf0445ce7619c73937590e856828014b2e9 100644 (file)
--- a/modules/roles/manifests/static_source.pp
+++ b/modules/roles/manifests/static_source.pp
@@ -1,14 +1,10 @@
-class roles::static_source inherits roles::static_base {
-    file {
-        '/etc/ssh/userkeys/staticsync':
-            content => template('roles/static-mirroring/static-mirror-authorized_keys.erb'),
-            ;
-        '/usr/local/bin/static-mirror-ssh-wrap':
-            source  => "puppet:///modules/roles/static-mirroring/static-mirror-ssh-wrap",
-            mode => 555,
-            ;
-    }
+class roles::static_source {
+       include roles::static_base
+       file { '/etc/ssh/userkeys/staticsync':
+               content => template('roles/static-mirroring/static-mirror-authorized_keys.erb'),
+       }
+       file { '/usr/local/bin/static-mirror-ssh-wrap':
+               source => 'puppet:///modules/roles/static-mirroring/static-mirror-ssh-wrap',
+               mode   => '0555',
+       }
 }
-# vim:set et:
-# vim:set sts=4 ts=4:
-# vim:set shiftwidth=4: