facter 1.7 has its own fact.d

manifests/common.pp

@@ -5,17 +5,25 @@ class puppet::common ($ensure = 'present') {
       default   => $ensure,
     }
   }
+  $ens_dir = $ensure ? {
+    'present' => 'directory',
+    default   => $ensure,
+  }
+
   $facts_d = '/etc/facter/facts.d'
   File['/etc/facter'] -> File[$facts_d]
 
-  file { ['/etc/facter', $facts_d]:
+  file {
-    ensure => $ensure ? {
+    [
-      'present' => 'directory',
+      '/etc/facter',
-      default   => $ensure,
+      $facts_d]:
-    },
+      ensure => $ens_dir,
       mode   => 0755,
       owner  => root,
-    group  => root,
+      group  => root;
+
+    "${::puppet_vardir}/exported":
+      ensure => $ens_dir;
   }
 
   # add facter variable to custom facts.

manifests/init.pp

@@ -74,7 +74,9 @@ class puppet (
     content => template("puppet/puppetd_run.sh.erb"),
     mode    => 0700,
     alias   => "puppetd_run.sh",
-    ensure  => "${ensure}",
+    ensure  => $ensure;
+    "/var/log/puppet":
+    mode => '0755',
   }
 
   case $::kernel {