collect slightly fewer packages

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

diff --git a/modules/bacula/manifests/director.pp b/modules/bacula/manifests/director.pp
index f3d6f68..0afd927 100644 (file)
--- a/modules/bacula/manifests/director.pp
+++ b/modules/bacula/manifests/director.pp
@@ -15,8 +15,7 @@ class bacula::director inherits bacula {
                path        => '/usr/bin:/usr/sbin:/bin:/sbin',
                command     => 'service bacula-director reload',
                refreshonly => true,
-        }
-
+       }
 
        file { '/etc/bacula/conf.d':
                ensure  => directory,

diff --git a/modules/buildd/manifests/init.pp b/modules/buildd/manifests/init.pp
index 691abf6..3a9d508 100644 (file)
--- a/modules/buildd/manifests/init.pp
+++ b/modules/buildd/manifests/init.pp
@@ -1,13 +1,16 @@
 class buildd {
 
        package { 'schroot':
-               ensure => installed
+               ensure => installed,
+               tag    => extra_repo,
        }
        package { 'sbuild':
                ensure => installed,
+               tag    => extra_repo,
        }
        package { 'libsbuild-perl':
                ensure => installed,
+               tag    => extra_repo,
                before => Package['sbuild']
        }
 

diff --git a/modules/debian-org/manifests/init.pp b/modules/debian-org/manifests/init.pp
index 4d7d63f..1c42d5f 100644 (file)
--- a/modules/debian-org/manifests/init.pp
+++ b/modules/debian-org/manifests/init.pp
@@ -20,6 +20,7 @@ class debian-org {
                        'dsa-munin-plugins',
                ]:
                ensure => installed,
+               tag    => extra_repo,
        }
 
        package { [
@@ -65,6 +66,7 @@ class debian-org {
        if getfromhash($site::nodeinfo, 'broken-rtc') {
                package { 'fake-hwclock':
                        ensure => installed,
+                       tag    => extra_repo,
                }
        }
 
@@ -229,7 +231,7 @@ class debian-org {
                path        => '/usr/bin:/usr/sbin:/bin:/sbin',
                refreshonly => true,
        }
-       Exec['apt-get update']->Package<| |>
+       Exec['apt-get update']->Package<| tag == extra_repo |>
 
        exec { 'dpkg-reconfigure tzdata -pcritical -fnoninteractive':
                path        => '/usr/bin:/usr/sbin:/bin:/sbin',

diff --git a/modules/hardware/manifests/raid/megactl.pp b/modules/hardware/manifests/raid/megactl.pp
index 1dc59f6..60d2492 100644 (file)
--- a/modules/hardware/manifests/raid/megactl.pp
+++ b/modules/hardware/manifests/raid/megactl.pp
@@ -2,6 +2,7 @@ class hardware::raid::megactl {
 
        package { 'megactl':
                ensure => installed,
+               tag    => extra_repo,
        }
 
        site::aptrepo { 'debian.restricted':

diff --git a/modules/hardware/manifests/raid/proliant.pp b/modules/hardware/manifests/raid/proliant.pp
index 993e5de..410bb5e 100644 (file)
--- a/modules/hardware/manifests/raid/proliant.pp
+++ b/modules/hardware/manifests/raid/proliant.pp
@@ -8,9 +8,11 @@ class hardware::raid::proliant {
 
        package { 'hpacucli':
                ensure  => installed,
+               tag    => extra_repo,
        }
        package { 'hp-health':
                ensure => installed,
+               tag    => extra_repo,
        }
        package { 'arrayprobe':
                ensure => installed,

diff --git a/modules/nagios/manifests/client.pp b/modules/nagios/manifests/client.pp
index 7c5d6ca..a2c2230 100644 (file)
--- a/modules/nagios/manifests/client.pp
+++ b/modules/nagios/manifests/client.pp
@@ -5,6 +5,7 @@ class nagios::client inherits nagios {
        }
        package { 'dsa-nagios-checks':
                ensure => installed,
+               tag    => extra_repo,
        }
 
        service { 'nagios-nrpe-server':

diff --git a/modules/site/manifests/aptrepo.pp b/modules/site/manifests/aptrepo.pp
index db6179f..e53de75 100644 (file)
--- a/modules/site/manifests/aptrepo.pp
+++ b/modules/site/manifests/aptrepo.pp
@@ -55,8 +55,8 @@ define site::aptrepo (
        }
 
        file { "/etc/apt/sources.list.d/${name}.list":
-                       ensure  => $ensure,
-                       content => template('site/aptrepo.erb'),
-                       notify  => Exec['apt-get update'],
+               ensure  => $ensure,
+               content => template('site/aptrepo.erb'),
+               notify  => Exec['apt-get update'],
        }
 }