Failure to find node facts is now a failure.

It was previously just a warning, but the node
is essentially non-functional without the facts,
so it makes more sense for it to be a warning.

Signed-off-by: Luke Kanies <luke@madstop.com>

1 files changed, 8 insertions, 4 deletions

diff --git a/lib/puppet/node.rb b/lib/puppet/node.rb
index 9650562d4..74bf8902d 100644
--- a/lib/puppet/node.rb
+++ b/lib/puppet/node.rb
@@ -67,10 +67,14 @@ class Puppet::Node
 
     # Merge the node facts with parameters from the node source.
     def fact_merge
-        if facts = Puppet::Node::Facts.find(name)
-            merge(facts.values)
-        else
-            Puppet.warning "Could not find facts for %s; you probably have a discrepancy between the node and fact names" % name
+        begin
+            if facts = Puppet::Node::Facts.find(name)
+                merge(facts.values)
+            end
+        rescue => detail
+            error = Puppet::Error.new("Could not retrieve facts for %s: %s" % [name, detail])
+            error.set_backtrace(detail.backtrace)
+            raise error
         end
     end