Make apt::setting notify Exec['apt_update'] by default

[puppet-modules/puppetlabs-apt.git] / manifests / setting.pp

diff --git a/manifests/setting.pp b/manifests/setting.pp
index f6f0bc0c2275821a3656d9e14cb73b30f1202f88..c288b35d7ec00ce545af777de09bfe977026738c 100644 (file)
--- a/manifests/setting.pp
+++ b/manifests/setting.pp
@@ -1,9 +1,10 @@
 define apt::setting (
-  $priority   = 50,
-  $ensure     = file,
-  $source     = undef,
-  $content    = undef,
-  $file_perms = {},
+  $priority      = 50,
+  $ensure        = file,
+  $source        = undef,
+  $content       = undef,
+  $file_perms    = {},
+  $notify_update = true,
 ) {
 
   $_file = merge($::apt::file_defaults, $file_perms)
@@ -17,6 +18,7 @@ define apt::setting (
   }
 
   validate_re($ensure,  ['file', 'present', 'absent'])
+  validate_bool($notify_update)
 
   $title_array = split($title, '-')
   $setting_type = $title_array[0]
@@ -46,6 +48,12 @@ define apt::setting (
   $_path = $::apt::config_files[$setting_type]['path']
   $_ext  = $::apt::config_files[$setting_type]['ext']
 
+  if $notify_update {
+    $_notify = Exec['apt_update']
+  } else {
+    $_notify = undef
+  }
+
   file { "${_path}/${_priority}${base_name}${_ext}":
     ensure  => $ensure,
     owner   => $_file['owner'],
@@ -53,5 +61,12 @@ define apt::setting (
     mode    => $_file['mode'],
     content => $content,
     source  => $source,
+    notify  => $_notify,
+  }
+
+  if $notify_update {
+    anchor { "apt::setting::${name}":
+      require => Class['apt::update']
+    }
   }
 }