variable assignment does not clone string...

[dsa-puppet.git] / modules / resolv / templates / resolv.conf.erb

diff --git a/modules/resolv/templates/resolv.conf.erb b/modules/resolv/templates/resolv.conf.erb
index cb493a9c3c16836f7502182c0cdaf4e7ab2c1962..dc2babed917ba71a2992f86e6cf705373e9429bb 100644 (file)
--- a/modules/resolv/templates/resolv.conf.erb
+++ b/modules/resolv/templates/resolv.conf.erb
@@ -3,39 +3,35 @@
 ### USE: git clone git+ssh://$USER@puppet.debian.org/srv/puppet.debian.org/git/dsa-puppet.git
 ###
 
-<%=
-nameservers = []
+<%
 searchpaths = []
-options = []
-
-if %w{draghi liszt widor spohr}.include?(hostname)
+searchpaths += @sp
+searchpaths << "debian.org" -%>
+search <%= searchpaths.to_a.flatten.join(" ") %>
+<%
+nameservers = []
+if %w{draghi}.include?(hostname)
   nameservers << "127.0.0.1"
 end
+nameservers += @ns
 
-nameservers += nodeinfo['hoster']['nameservers'] if nodeinfo['hoster']['nameservers']
-searchpaths += nodeinfo['hoster']['searchpaths'] if nodeinfo['hoster']['searchpaths']
-options += nodeinfo['hoster']['resolvoptions'] if nodeinfo['hoster']['resolvoptions']
-
-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
-  nameservers.each do |ns|
-    resolvconf += "nameserver " + ns + "\n"
-  end
+if @unbound && @unbound == "true"
+  nameservers = ['127.0.0.1']
 end
 
-options.each do |opt|
-  resolvconf += "options " + opt + "\n"
+if nameservers.empty?
+  #raise Puppet::ParseError, "Something has gone wrong writing resolv.conf.  No nameservers to use!"
+  scope.function_warning(["Something has gone wrong writing resolv.conf.  No nameservers to use - using google's!"])
+  nameservers << '8.8.8.8'
+  nameservers << '8.8.4.4'
 end
 
-resolvconf
-%>
+nameservers.to_a.flatten.each do |nms| -%>
+nameserver <%= nms %>
+<% end -%>
+<%
+options  = []
+options += @opts -%>
+<% options.to_a.flatten.each do |opt| -%>
+options <%= opt %>
+<% end -%>