s/err/fail/

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

diff --git a/modules/apache2/manifests/config.pp b/modules/apache2/manifests/config.pp
index 5d5170046959a70697872dbbd411a380c4059b80..b67ddbee75e5ccdb864abf8874dfab4ef81f20ad 100644 (file)
--- a/modules/apache2/manifests/config.pp
+++ b/modules/apache2/manifests/config.pp
@@ -2,14 +2,14 @@ define apache2::config($config = undef, $template = undef, $ensure = present) {
 
        include apache2
 
-       if ! ($config or $template) {
-               err ( "No configuration found for ${name}" )
-       }
-
        case $ensure {
-               present: {}
+               present: {
+                       if ! ($config or $template) {
+                               fail ( "No configuration found for ${name}" )
+                       }
+               }
                absent:  {}
-               default: { err ( "Unknown ensure value: '$ensure'" ) }
+               default: { fail ( "Unknown ensure value: '$ensure'" ) }
        }
 
        if $template {

diff --git a/modules/apache2/manifests/module.pp b/modules/apache2/manifests/module.pp
index 3a6922bd31c38a5836931fa624deb4b25cecb5a8..17d4a71a0a921815de238f9694e424e2d13248f4 100644 (file)
--- a/modules/apache2/manifests/module.pp
+++ b/modules/apache2/manifests/module.pp
@@ -12,6 +12,6 @@ define apache2::module ($ensure = present) {
                                notify => Service['apache2']
                        }
                }
-               default: { err ( "Unknown ensure value: '$ensure'" ) }
+               default: { fail ( "Unknown ensure value: '$ensure'" ) }
        }
 }

diff --git a/modules/apache2/manifests/site.pp b/modules/apache2/manifests/site.pp
index 263c6d8a692a82327ca864fd52f11a681e6c32f2..8fa5fdf526097df06ce8a681d4acb282848c0c5c 100644 (file)
--- a/modules/apache2/manifests/site.pp
+++ b/modules/apache2/manifests/site.pp
@@ -7,8 +7,10 @@ define apache2::site (
 
        include apache2
 
-       if ! ($config or $template) {
-               err ( "No configuration found for ${name}" )
+       if $ensure == present {
+               if ! ($config or $template) {
+                       fail ( "No configuration found for ${name}" )
+               }
        }
 
        if $site {
@@ -22,7 +24,7 @@ define apache2::site (
        $link_target = $ensure ? {
                present => $target,
                absent  => absent,
-               default => err ( "Unknown ensure value: '$ensure'" ),
+               default => fail ( "Unknown ensure value: '$ensure'" ),
        }
 
        case $template {

diff --git a/modules/munin/manifests/check.pp b/modules/munin/manifests/check.pp
index d807861fe7df7d3ec441de0d4dd34ced27d49e3f..ace6bb52e08d2583776eae0c5cf056314388cfa9 100644 (file)
--- a/modules/munin/manifests/check.pp
+++ b/modules/munin/manifests/check.pp
@@ -11,7 +11,7 @@ define munin::check($ensure = present, $script = undef) {
        $link_target = $ensure ? {
                present => "/usr/share/munin/plugins/${link}",
                absent  => absent,
-               default => err ( "Unknown ensure value: '$ensure'" ),
+               default => fail ( "Unknown ensure value: '$ensure'" ),
        }
 
        file { "/etc/munin/plugins/${name}":

diff --git a/modules/site/manifests/alternative.pp b/modules/site/manifests/alternative.pp
index 94d088818a71398109adf270bb35e7603a28a394..c207df882d4dda95c76f39be9a38711d3fa00c1c 100644 (file)
--- a/modules/site/manifests/alternative.pp
+++ b/modules/site/manifests/alternative.pp
@@ -12,6 +12,6 @@ define site::alternative ($linkto, $ensure = present) {
                                        unless => "[ $(update-alternatives --query ${name} | grep ^Value | awk '{print \$2}') != ${linkto} ]",
                        }
                }
-               default: { err ( "Unknown ensure value: '$ensure'" ) }
+               default: { fail ( "Unknown ensure value: '$ensure'" ) }
        }
 }

diff --git a/modules/site/manifests/aptrepo.pp b/modules/site/manifests/aptrepo.pp
index d461807af79a14af447f51162da33b2ad959e8b2..79d1a59c6cedee053832d7fa2dd8cb37fa0ff907 100644 (file)
--- a/modules/site/manifests/aptrepo.pp
+++ b/modules/site/manifests/aptrepo.pp
@@ -16,11 +16,13 @@ define site::aptrepo ($key = undef, $template = undef, $config = undef, $ensure
        case $ensure {
                present: {}
                absent:  {}
-               default: { err ( "Unknown ensure value: '$ensure'" ) }
+               default: { fail ( "Unknown ensure value: '$ensure'" ) }
        }
 
-       if ! ($template or $config) {
-               err ( "Can't find configuration for ${name}" )
+       if $ensure == present {
+               if ! ($config or $template) {
+                       fail ( "No configuration found for ${name}" )
+               }
        }
 
        if $template {

diff --git a/modules/site/manifests/linux_module.pp b/modules/site/manifests/linux_module.pp
index 8a81c6c0e2d266b7f204fb53d4be4db71d725fac..36c09dcf36e1c347e08f364833d2e0689b2a977c 100644 (file)
--- a/modules/site/manifests/linux_module.pp
+++ b/modules/site/manifests/linux_module.pp
@@ -13,9 +13,7 @@ define site::linux_module ($ensure = present) {
                                        onlyif => "grep -q -F -x '${name}' /etc/modules",
                                }
                        }
-                       default: {
-                               err("invalid ensure value ${ensure}")
-                       }
+                       default: { fail ("invalid ensure value ${ensure}") }
                }
        }
 }

diff --git a/modules/site/manifests/sysctl.pp b/modules/site/manifests/sysctl.pp
index 9786c8e504f788c9cd0bcd222add599aed900220..ac357a7cbd7d64ffc3bf63a32d42c7abe7cd91bc 100644 (file)
--- a/modules/site/manifests/sysctl.pp
+++ b/modules/site/manifests/sysctl.pp
@@ -3,7 +3,7 @@ define site::sysctl ($key, $value, $ensure = present) {
        case $ensure {
                present: {}
                absent:  {}
-               default: { err ( "Unknown ensure value: '$ensure'" ) }
+               default: { fail ( "Unknown ensure value: '$ensure'" ) }
        }
 
        file {