Apparently (and fortunately) exec names are site-wide and not just per module. This...

diff --git a/modules/apt-keys/manifests/init.pp b/modules/apt-keys/manifests/init.pp
index 6ecdca64823978fd703975cb969a16b2eb07ec5f..45e94f990581a221ebde6333f6ad4714813e0d4c 100644 (file)
--- a/modules/apt-keys/manifests/init.pp
+++ b/modules/apt-keys/manifests/init.pp
@@ -3,17 +3,17 @@ class apt-keys {
         "/etc/apt/trusted-keys.d/":
           ensure  => directory,
           purge   => true,
-          notify  => Exec["updatekeys"],
+          notify  => Exec["apt-keys-update"],
           ;
 
         "/etc/apt/trusted-keys.d/db.debian.org.asc":
           source  => "puppet:///apt-keys/db.debian.org.asc",
           mode    => 664,
-          notify  => Exec["updatekeys"],
+          notify  => Exec["apt-keys-update"],
           ;
     }
 
-    exec { "updatekeys":
+    exec { "apt-keys-update":
          command => 'for keyfile in /etc/apt/trusted-keys.d/*; do apt-key add $keyfile; done',
          refreshonly => true
     }

diff --git a/modules/buildd/manifests/init.pp b/modules/buildd/manifests/init.pp
index 825f436b3b32340e7b2dca35662ac8fac7ccefa3..8ca128e767b7e8a91aa20b98deffc2b0b4391a22 100644 (file)
--- a/modules/buildd/manifests/init.pp
+++ b/modules/buildd/manifests/init.pp
@@ -16,7 +16,7 @@ class buildd {
         "/etc/apt/trusted-keys.d/buildd.debian.org.asc":
           source  => "puppet:///buildd/buildd.debian.org.asc",
           mode    => 664,
-          notify  => Exec["updatekeys"],
+          notify  => Exec["apt-keys-update"],
           ;
    }
 }