Revert "simplify site::aptrepo"

This reverts commit 5f7e9b2a528a64883641f38558c028ae4b1967f0.

diff --git a/modules/buildd/manifests/init.pp b/modules/buildd/manifests/init.pp
index ab6b05a2c8231d77543d38eadb2c1b58b75d1400..79065a17c45a28aa501df88b96bc9c3ee90f3c4a 100644 (file)
--- a/modules/buildd/manifests/init.pp
+++ b/modules/buildd/manifests/init.pp
@@ -23,8 +23,8 @@ class buildd {
                ensure => absent,
        }
        site::aptrepo { 'buildd.debian.org':
-               content => template('buildd/etc/apt/sources.list.d/buildd.list.erb'),
-               key     => 'puppet:///modules/buildd/buildd.debian.org.asc',
+               template => 'buildd/etc/apt/sources.list.d/buildd.list.erb',
+               key      => 'puppet:///modules/buildd/buildd.debian.org.asc',
        }
 
        file { '/etc/apt/preferences.d/buildd':

diff --git a/modules/debian-org/manifests/init.pp b/modules/debian-org/manifests/init.pp
index 9da7de5381bba4e5733bd1e49be6e686244b9ff0..da54711e5eebc3d37ee08aa808adea9cf087ccb6 100644 (file)
--- a/modules/debian-org/manifests/init.pp
+++ b/modules/debian-org/manifests/init.pp
@@ -62,13 +62,13 @@ class debian-org {
 
        if $::debarchitecture != 'armhf' {
                site::aptrepo { 'security':
-                       content => template('debian-org/etc/apt/sources.list.d/security.list.erb'),
+                       template => 'debian-org/etc/apt/sources.list.d/security.list.erb',
                }
                site::aptrepo { 'backports.debian.org':
-                       content => template('debian-org/etc/apt/sources.list.d/backports.debian.org.list.erb'),
+                       template => 'debian-org/etc/apt/sources.list.d/backports.debian.org.list.erb',
                }
                site::aptrepo { 'volatile':
-                       content => template('debian-org/etc/apt/sources.list.d/volatile.list.erb'),
+                       template => 'debian-org/etc/apt/sources.list.d/volatile.list.erb',
                }
        }
        site::aptrepo { 'backports.org':
@@ -82,8 +82,8 @@ class debian-org {
        }
 
        site::aptrepo { 'db.debian.org':
-               content => template('debian-org/etc/apt/sources.list.d/debian.org.list.erb'),
-               key     => 'puppet:///modules/debian-org/db.debian.org.asc',
+               template => 'debian-org/etc/apt/sources.list.d/debian.org.list.erb',
+               key      => 'puppet:///modules/debian-org/db.debian.org.asc',
        }
 
        file { '/etc/facter':

diff --git a/modules/debian-org/manifests/proliant.pp b/modules/debian-org/manifests/proliant.pp
index afb6bed529db187ab227c0c75887e0c4eb8fa8d1..04b9cdd39ca4bdce042f29bd96b99e11954fd4e5 100644 (file)
--- a/modules/debian-org/manifests/proliant.pp
+++ b/modules/debian-org/manifests/proliant.pp
@@ -1,7 +1,7 @@
 class debian-org::proliant {
 
        site::aptrepo { 'debian.restricted':
-               content => template('debian-org/etc/apt/sources.list.d/debian.restricted.list.erb'),
+               template => 'debian-org/etc/apt/sources.list.d/debian.restricted.list.erb',
        }
 
        package { 'hpacucli':
@@ -26,3 +26,5 @@ class debian-org::proliant {
                }
        }
 }
+
+

diff --git a/modules/megactl/manifests/init.pp b/modules/megactl/manifests/init.pp
index 7dd32afaab3d0596fbc00392afefb5031cb4c0b4..3c376639a499ff216cd637a546e6cf29810bebeb 100644 (file)
--- a/modules/megactl/manifests/init.pp
+++ b/modules/megactl/manifests/init.pp
@@ -4,6 +4,6 @@ class megactl {
        }
 
        site::aptrepo { 'debian.restricted':
-               content => template('debian-org/etc/apt/sources.list.d/debian.restricted.list.erb'),
+               template => 'debian-org/etc/apt/sources.list.d/debian.restricted.list.erb',
        }
 }

diff --git a/modules/named/manifests/geodns.pp b/modules/named/manifests/geodns.pp
index 7e3ac019ac561224d49cb74ef766147cb040236c..b12593dac8b139cc42c0ae36197ef2a329ecf7fe 100644 (file)
--- a/modules/named/manifests/geodns.pp
+++ b/modules/named/manifests/geodns.pp
@@ -4,7 +4,7 @@ class named::geodns inherits named {
        }
 
        site::aptrepo { 'geoip':
-               content => template('debian-org/etc/apt/sources.list.d/geoip.list.erb'),
+               template => 'debian-org/etc/apt/sources.list.d/geoip.list.erb',
        }
 
        file { '/etc/bind/':

diff --git a/modules/site/manifests/aptrepo.pp b/modules/site/manifests/aptrepo.pp
index ea1b9202da1e3d07a4c0d33cfe36e47af1647142..ae66d6f216b374dbbba7f006e4557d726ea5e334 100644 (file)
--- a/modules/site/manifests/aptrepo.pp
+++ b/modules/site/manifests/aptrepo.pp
@@ -1,4 +1,5 @@
-define site::aptrepo ($key = undef, $keyid = undef, $content = '', $source = '', $ensure = present) {
+define site::aptrepo ($key = undef, $keyid = undef, $template = undef, $config = undef, $ensure = present) {
+
 
        case $ensure {
                present: {
@@ -39,15 +40,22 @@ define site::aptrepo ($key = undef, $keyid = undef, $content = '', $source = '',
        }
 
        if $ensure == present {
-               if ! ($source or $content) {
+               if ! ($config or $template) {
                        fail ( "No configuration found for ${name}" )
                }
        }
 
-       file { "/etc/apt/sources.list.d/${name}.list":
-               ensure  => $ensure,
-               content => $content,
-               source  => $source,
-               notify  => Exec['apt-get update'],
+       if $template {
+               file { "/etc/apt/sources.list.d/${name}.list":
+                       ensure  => $ensure,
+                       content => template($template),
+                       notify => Exec['apt-get update'],
+               }
+       } else {
+               file { "/etc/apt/sources.list.d/${name}.list":
+                       ensure => $ensure,
+                       source => $config,
+                       notify => Exec['apt-get update'],
+               }
        }
 }