X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=modules%2Fsite%2Fmanifests%2Faptrepo.pp;h=e53de75cd23b3b8c547a219e26d04db6584ebc37;hb=a1d397a274083c5e9dfb9d690593e6d65968b264;hp=c97fade0ea3f6660273676bf628de545a750b186;hpb=9573712862a431d238cf67894f4b422794eaa78f;p=mirror%2Fdsa-puppet.git

diff --git a/modules/site/manifests/aptrepo.pp b/modules/site/manifests/aptrepo.pp
index c97fade0e..e53de75cd 100644
--- a/modules/site/manifests/aptrepo.pp
+++ b/modules/site/manifests/aptrepo.pp
@@ -1,6 +1,6 @@
 define site::aptrepo (
-	$url,
-	$suite,
+	$url='',
+	$suite='',
 	$components=[],
 	$key = undef,
 	$keyid = undef,
@@ -45,9 +45,18 @@ define site::aptrepo (
 		default: { fail ( "Unknown ensure value: '$ensure'" ) }
 	}
 
+	case $ensure {
+		present: {
+			if !($url and $suite) {
+				fail ( "Need both url and suite for $name" )
+			}
+		}
+		default: {}
+	}
+
 	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'],
 	}
 }