Updated all templates to use instance variables

resolver/manifests/init.pp

@@ -21,11 +21,13 @@
 #               nameservers => ['192.168.1.100', '192.168.1.101', '192.168.1.102'],
 #       }
 #
-class resolver($domainname = "", $searchpath = "", $nameservers ) {
-        file { "/etc/resolv.conf":
-                owner   => root,
-                group   => root,
-                mode    => 644,
-                content => template("resolver/resolv.conf.erb"),
-        }
+class resolver($domainname = "", $searchpath = [], $nameservers) {
+
+    file { "/etc/resolv.conf":
+        owner   => "root",
+        group   => "root",
+        mode    => "0644",
+        content => template("resolver/resolv.conf.erb"),
+    }
+
 }

resolver/templates/resolv.conf.erb

@@ -1,6 +1,9 @@
-<% if !domainname.empty? %>domain <%= domainname %>
+<% unless @domainname.empty? -%>
+domain <%= @domainname %>
 <% end -%>
-<% if !searchpath.empty? %>search <%= searchpath.join(" ") %>
+<% unless @searchpath.empty? -%>
+search <%= @searchpath.join(" ") %>
 <% end -%>
-<% nameservers.each do |ns| %>nameserver <%= ns %>
+<% @nameservers.each do |ns| -%>
+nameserver <%= ns %>
 <% end -%>