check url and suite only if ensure => present

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

diff --git a/modules/site/manifests/aptrepo.pp b/modules/site/manifests/aptrepo.pp
index c97fade0ea3f6660273676bf628de545a750b186..52036ed9d3f06c8e859676f1621c482a70408378 100644 (file)
--- 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,6 +45,15 @@ 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'),