From: Peter Palfrader Date: Thu, 2 Sep 2010 14:59:44 +0000 (+0200) Subject: We use "keyinfo" in all kinda of places. Rename it to something more generic X-Git-Url: https://git.donarmstrong.com/?p=dsa-puppet.git;a=commitdiff_plain;h=d7475f90c56697e9589e1386ced135498df68c81 We use "keyinfo" in all kinda of places. Rename it to something more generic --- diff --git a/manifests/site.pp b/manifests/site.pp index 9b00836e..f040683c 100644 --- a/manifests/site.pp +++ b/manifests/site.pp @@ -17,8 +17,7 @@ node default { $localinfo = yamlinfo('*', "/etc/puppet/modules/debian-org/misc/local.yaml") $nodeinfo = nodeinfo($fqdn, "/etc/puppet/modules/debian-org/misc/local.yaml") $hoster = whohosts($nodeinfo, "/etc/puppet/modules/debian-org/misc/hoster.yaml") - $keyinfo = allnodeinfo("sshRSAHostKey", "ipHostNumber") - $mxinfo = allnodeinfo("mXRecord") + $ldapall = allnodeinfo("sshRSAHostKey", "ipHostNumber", "mXRecord") notice("hoster for ${fqdn} is ${hoster}") include munin-node diff --git a/modules/exim/templates/manualroute.erb b/modules/exim/templates/manualroute.erb index 2b84b4ed..07ddf551 100644 --- a/modules/exim/templates/manualroute.erb +++ b/modules/exim/templates/manualroute.erb @@ -19,8 +19,8 @@ when 'spohr.debian.org', 'draghi.debian.org', 'merikanto.debian.org' then end mxregex = Regexp.new('^\d+\s+(.*)\.$') -mxinfo.keys.sort.each do |host| - mxinfo[host]['mXRecord'].each do |mx| +ldapall.keys.sort.each do |host| + ldapall[host]['mXRecord'].each do |mx| mxmatch = mxregex.match(mx) if mxmatches.include?(mxmatch[1]) route = host + ":\t\t" + host diff --git a/modules/ferm/templates/defs.conf.erb b/modules/ferm/templates/defs.conf.erb index c39b03d6..a6842097 100644 --- a/modules/ferm/templates/defs.conf.erb +++ b/modules/ferm/templates/defs.conf.erb @@ -19,7 +19,7 @@ mailrelay = [] localinfo.keys.sort.each do |node| if localinfo[node]['mailrelay'] - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall[node]['ipHostNumber'].each do |ip| next if ip =~ /:/ mailrelay << ip end @@ -33,7 +33,7 @@ mailrelay = [] localinfo.keys.sort.each do |node| if localinfo[node]['mailrelay'] - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall[node]['ipHostNumber'].each do |ip| next if ip =~ /\./ mailrelay << ip end @@ -49,7 +49,7 @@ nagii = [] localinfo.keys.sort.each do |node| if localinfo[node]['nagiosmaster'] or localinfo[node]['extranrpeclient'] - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall[node]['ipHostNumber'].each do |ip| next if ip =~ /:/ nagii << ip end @@ -63,7 +63,7 @@ nagii = [] localinfo.keys.sort.each do |node| if localinfo[node]['nagiosmaster'] or localinfo[node]['extranrpeclient'] - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall[node]['ipHostNumber'].each do |ip| next if ip =~ /\./ nagii << ip end @@ -79,7 +79,7 @@ munins = [] localinfo.keys.sort.each do |node| if localinfo[node]['muninmaster'] - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall[node]['ipHostNumber'].each do |ip| next if ip =~ /:/ munins << ip end @@ -93,7 +93,7 @@ munins = [] localinfo.keys.sort.each do |node| if localinfo[node]['muninmaster'] - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall[node]['ipHostNumber'].each do |ip| next if ip =~ /\./ munins << ip end @@ -109,7 +109,7 @@ dbs = [] localinfo.keys.sort.each do |node| if localinfo[node]['dbmaster'] - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall[node]['ipHostNumber'].each do |ip| next if ip =~ /\./ dbs << ip end @@ -123,7 +123,7 @@ dbs = [] localinfo.keys.sort.each do |node| if localinfo[node]['dbmaster'] - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall[node]['ipHostNumber'].each do |ip| next if ip =~ /:/ dbs << ip end @@ -137,9 +137,9 @@ @def $HOST_DEBIAN_V4 = (<%= dbs = [] - keyinfo.keys.sort.each do |node| - next unless keyinfo[node].has_key?('ipHostNumber') - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall.keys.sort.each do |node| + next unless ldapall[node].has_key?('ipHostNumber') + ldapall[node]['ipHostNumber'].each do |ip| next if ip =~ /:/ dbs << ip end @@ -150,9 +150,9 @@ @def $HOST_DEBIAN_V6 = (<%= dbs = [] - keyinfo.keys.sort.each do |node| - next unless keyinfo[node].has_key?('ipHostNumber') - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall.keys.sort.each do |node| + next unless ldapall[node].has_key?('ipHostNumber') + ldapall[node]['ipHostNumber'].each do |ip| next if ip =~ /\./ dbs << ip end diff --git a/modules/munin-node/templates/munin-node.conf.erb b/modules/munin-node/templates/munin-node.conf.erb index 231b1a25..73eab95e 100644 --- a/modules/munin-node/templates/munin-node.conf.erb +++ b/modules/munin-node/templates/munin-node.conf.erb @@ -42,7 +42,7 @@ ignore_file \.rpm(save|new)$ str = '' localinfo.keys.sort.each do |node| if localinfo[node]['muninmaster'] - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall[node]['ipHostNumber'].each do |ip| str += "allow ^" + ip.split('.').join('\.') + "$\n" end end diff --git a/modules/nagios/templates/inc-debian.org.erb b/modules/nagios/templates/inc-debian.org.erb index bf6c2f95..b3ce3585 100644 --- a/modules/nagios/templates/inc-debian.org.erb +++ b/modules/nagios/templates/inc-debian.org.erb @@ -7,7 +7,7 @@ nagii = [] localinfo.keys.sort.each do |node| if localinfo[node]['nagiosmaster'] or localinfo[node]['extranrpeclient'] - nagii << keyinfo[node]['ipHostNumber'] + nagii << ldapall[node]['ipHostNumber'] end end diff --git a/modules/named/templates/named.conf.options.erb b/modules/named/templates/named.conf.options.erb index 84469dff..8d602783 100644 --- a/modules/named/templates/named.conf.options.erb +++ b/modules/named/templates/named.conf.options.erb @@ -8,7 +8,7 @@ acl Nagios { str = '' localinfo.keys.sort.each do |node| if localinfo[node]['nagiosmaster'] - keyinfo[node]['ipHostNumber'].each do |ip| + ldapall[node]['ipHostNumber'].each do |ip| str += "\t" + ip + "/32;\n" end end diff --git a/modules/portforwarder/templates/authorized_keys.erb b/modules/portforwarder/templates/authorized_keys.erb index 985f367c..d119bf2f 100644 --- a/modules/portforwarder/templates/authorized_keys.erb +++ b/modules/portforwarder/templates/authorized_keys.erb @@ -29,7 +29,7 @@ config.each_pair do |sourcehost, services| if allowed_ports.length > 0 sshkey = getportforwarderkey(sourcehost) - remote_ip = keyinfo[sourcehost]['ipHostNumber'].join(',') + remote_ip = ldapall[sourcehost]['ipHostNumber'].join(',') local_bind = '127.101.%d.%d'%[ (sourcehost.hash / 256 % 256), sourcehost.hash % 256 ] lines << "# from #{sourcehost}" diff --git a/modules/ssh/templates/authorized_keys.erb b/modules/ssh/templates/authorized_keys.erb index ba6f9f6f..71b604cd 100644 --- a/modules/ssh/templates/authorized_keys.erb +++ b/modules/ssh/templates/authorized_keys.erb @@ -36,9 +36,9 @@ ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAgEAuGJnElqbhgLtmJp/de8s42cAwKrkAhFq5u8EAkauEv6B <%= machine_keys = case fqdn when "bartok.debian.org", "beethoven.debian.org" then out = '' - keyinfo.keys.sort.each do |node| - out += '# ' + keyinfo[node]['hostname'].to_s + ' -command="/usr/lib/da-backup/da-backup-ssh-wrap ' + keyinfo[node]['hostname'].to_s + '",no-port-forwarding,no-X11-forwarding,no-agent-forwarding,from="' + keyinfo[node]['ipHostNumber'].join(',') + '" ' + keyinfo[node]['sshRSAHostKey'].to_s + ' + ldapall.keys.sort.each do |node| + out += '# ' + ldapall[node]['hostname'].to_s + ' +command="/usr/lib/da-backup/da-backup-ssh-wrap ' + ldapall[node]['hostname'].to_s + '",no-port-forwarding,no-X11-forwarding,no-agent-forwarding,from="' + keyinfo[node]['ipHostNumber'].join(',') + '" ' + keyinfo[node]['sshRSAHostKey'].to_s + ' ' end