Fix resolv.conf.erb to work with no nameservers when we have a local recursor

diff --git a/modules/resolv/templates/resolv.conf.erb b/modules/resolv/templates/resolv.conf.erb
index cb493a9c3c16836f7502182c0cdaf4e7ab2c1962..9bc11ba9664e90a63b189e5d328aad840e14ce15 100644 (file)
--- a/modules/resolv/templates/resolv.conf.erb
+++ b/modules/resolv/templates/resolv.conf.erb
@@ -18,16 +18,16 @@ options += nodeinfo['hoster']['resolvoptions'] if nodeinfo['hoster']['resolvopti
 
 searchpaths << "debian.org"
 
-if nameservers.empty?
-  raise Puppet::ParseError, "Something has gone wrong writing resolv.conf: probably included for wrong hoster"
-end
-
 resolvconf  = ''
 resolvconf += "search " + searchpaths.join(" ") + "\n"
 
 if runs_local_resolver
   resolvconf += "nameserver 127.0.0.1\n"
 else
+  if nameservers.empty?
+    raise Puppet::ParseError, "Something has gone wrong writing resolv.conf.  No nameservers to use!"
+  end
+
   nameservers.each do |ns|
     resolvconf += "nameserver " + ns + "\n"
   end