Merge pull request #75 from EncoreTechnologies/hotfix/ubuntu-puppet-facts

Patching facts

Author: nmaludy

CHANGELOG.md

@@ -4,11 +4,17 @@ All notable changes to this project will be documented in this file.
 
 ## Development
 
-- Bumped module `puppetlabs/puppet_agent` to `< 5.0.0`
+* Fixed issue where puppet facts were not in the expected spot causing
+  puppet_facts plan to fail. We added a conditional to check for the facts
+  in both places
+
+  Contributed by Bradley Bishop (@bishopbm1)
+
+* Bumped module `puppetlabs/puppet_agent` to `< 5.0.0`
 
   Contributed by @fetzerms
 
-- Added module `puppetlabs/reboot` to `>= 3.0.0 < 5.0.0`
+* Added module `puppetlabs/reboot` to `>= 3.0.0 < 5.0.0`
 
   Contributed by @fetzerms
 

plans/puppet_facts.pp

@@ -23,7 +23,14 @@
   #
   # We only want to set the "values" as facts on the node
   $result_set.each |$result| {
-    add_facts($result.target, $result.value['values'])
+    # Debian systems to not have values param they are just in the value return
+    if 'values' in $result.value {
+      $target_fact_values = $result.value['values']
+    } else {
+      $target_fact_values = $result.value
+    }
+
+    add_facts($result.target, $target_fact_values)
   }
   return $result_set
 }