template breakage

Signed-off-by: Stephen Gran <steve@lobefin.net>

diff --git a/modules/exim/templates/manualroute.erb b/modules/exim/templates/manualroute.erb
index 21a194edaa708b46cea755ad8291c8f7d8ed1899..430a4bf96e870b7b8e2123ec3d45f46f8ea42b08 100644 (file)
--- a/modules/exim/templates/manualroute.erb
+++ b/modules/exim/templates/manualroute.erb
@@ -8,7 +8,7 @@
 ## buildd.debian.org: grieg.debian.org::587 byname
 
 <%=
-mxmatches = [ scope.lookupvar('::fqdn') ]
+mxmatches = [ @fqdn ]
 routes = []
 extraroutes = []
 

diff --git a/modules/ferm/templates/defs.conf.erb b/modules/ferm/templates/defs.conf.erb
index aafed399a3f55bd617540308aad413c0302248ed..67e3457db2e41c330770f304f6f8e3d608626500 100644 (file)
--- a/modules/ferm/templates/defs.conf.erb
+++ b/modules/ferm/templates/defs.conf.erb
@@ -21,17 +21,15 @@
 
 <%
   rolehost={}
-  localinfo = scope.lookupvar('site::localinfo')
   allnodeinfo = scope.lookupvar('site::allnodeinfo')
+  roles = scope.lookupvar('site::roles')
 
   %w{mailrelay nagiosmaster extranrpeclient muninmaster dbmaster static_mirror static_source static_master}.each do |role|
     rolehost[role] = []
-    localinfo.keys.sort.each do |node|
-      if localinfo[node][role]
+    roles[role].each do |node|
         rolehost[role] << allnodeinfo[node]['ipHostNumber']
-      end
     end
-    rolehost[role].flatten!
+    rolehost[role].flatten!.sort.uniq
   end
 
   dbs = []

diff --git a/modules/munin/templates/munin-node.conf.erb b/modules/munin/templates/munin-node.conf.erb
index d2d00b8cecb3d55666962dc1b06f1d6004ad8acb..7a316a1950883969886cbc03a8fa65b9b52575bf 100644 (file)
--- a/modules/munin/templates/munin-node.conf.erb
+++ b/modules/munin/templates/munin-node.conf.erb
@@ -41,12 +41,11 @@ ignore_file \.rpm(save|new)$
 allow ^127\.0\.0\.1$
 <%=
 str = ''
-scope.lookupvar('site::localinfo').keys.sort.each do |node|
-   if scope.lookupvar('site::localinfo')[node]['muninmaster']
+roles = scope.lookupvar('site::roles')
+roles['muninmaster'].each do |node|
       scope.lookupvar('site::allnodeinfo')[node]['ipHostNumber'].each do |ip|
          str += "allow ^" + ip.split('.').join('\.') + "$\n"
       end
-   end
 end
 str
 -%>

diff --git a/modules/named/templates/named.conf.options.erb b/modules/named/templates/named.conf.options.erb
index 325b7f1fc0dc8eac0b7280e62555416cbf3703d2..9ec33e8b1c24385b892118d1a4b0455aa568a64c 100644 (file)
--- a/modules/named/templates/named.conf.options.erb
+++ b/modules/named/templates/named.conf.options.erb
@@ -5,13 +5,12 @@
 
 acl Nagios {
 <%=
+  roles = scope.lookupvar('site::roles')
   str = ''
-  scope.lookupvar('site::localinfo').keys.sort.each do |node|
-      if scope.lookupvar('site::localinfo')[node]['nagiosmaster']
+  roles['nagiosmaster'].each do |node|
           scope.lookupvar('site::allnodeinfo')[node]['ipHostNumber'].each do |ip|
               str += "\t" + ip + "/32;\n"
           end
-      end
   end
   str-%>
 };

diff --git a/modules/roles/templates/static-mirroring/static-clients.conf.erb b/modules/roles/templates/static-mirroring/static-clients.conf.erb
index edbf313651cabbb82012dfa1da142392671f9180..8e2b872172a365cd968349af7cc7e1cebc267d40 100644 (file)
--- a/modules/roles/templates/static-mirroring/static-clients.conf.erb
+++ b/modules/roles/templates/static-mirroring/static-clients.conf.erb
@@ -4,15 +4,8 @@
 
 <%=
 
-mirrors = []
-scope.lookupvar('site::localinfo').keys.sort.each do |node|
-    if scope.lookupvar('site::localinfo')[node]['static_mirror']
-        mirrors << node
-    end
-end
+scope.lookupvar('site::roles')['static_mirror'].join("\n")
 
-
-mirrors.join("\n")
 # vim:set et:
 # vim:set sts=4 ts=4:
 # vim:set shiftwidth=4:

diff --git a/modules/roles/templates/static-mirroring/staticsync-authorized_keys.erb b/modules/roles/templates/static-mirroring/staticsync-authorized_keys.erb
index 03e91b0630b5429a50eac116486cc77111d6cfe4..eecf4c65fae34c00fe0539666f705947d9ab9250 100644 (file)
--- a/modules/roles/templates/static-mirroring/staticsync-authorized_keys.erb
+++ b/modules/roles/templates/static-mirroring/staticsync-authorized_keys.erb
@@ -13,12 +13,13 @@ def getstaticsynckey(host)
     return key
 end
 
-localinfo = scope.lookupvar('site::localinfo')
 allnodeinfo = scope.lookupvar('site::allnodeinfo')
+roles = scope.lookupvar('site::roles')
 
 callers = []
-localinfo.keys.sort.each do |node|
-    if localinfo[node]['static_mirror'] or localinfo[node]['static_source'] or localinfo[node]['static_master']
+
+%w{static_mirror static_source static_master}.each do |role|
+    roles[role].each do |node|
         key = getstaticsynckey(node)
         callers << { 'node' => node, 'addr' => allnodeinfo[node]['ipHostNumber'], 'key' => key}
     end

diff --git a/modules/roles/templates/weblog_destination-authorized_keys.erb b/modules/roles/templates/weblog_destination-authorized_keys.erb
index fc21537fb28c1c32bcb836ec06a46e8cf876e25e..60867e525dbf1f320d817696b65f431e2394a1f2 100644 (file)
--- a/modules/roles/templates/weblog_destination-authorized_keys.erb
+++ b/modules/roles/templates/weblog_destination-authorized_keys.erb
@@ -13,15 +13,13 @@ def getweblogsynckey(host)
     return key
 end
 
-localinfo = scope.lookupvar('site::localinfo')
 allnodeinfo = scope.lookupvar('site::allnodeinfo')
+roles = scope.lookupvar('site::roles')
 
 mirrors = []
-localinfo.keys.sort.each do |node|
-    if localinfo[node]['weblog_provider']
+roles['weblog_provider'].each do |node|
         key = getweblogsynckey(node)
         mirrors << { 'node' => node, 'addr' => allnodeinfo[node]['ipHostNumber'], 'key' => key}
-    end
 end
 
 lines = []