X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=modules%2Fpuppetmaster%2Flib%2Fpuppet%2Fparser%2Ffunctions%2Fnodeinfo.rb;h=28da4aea48a897e27ba9756ba3b583bb48f07891;hb=e1dde292c2467e94d733eb57d7a23cd999f1fdd8;hp=947caf70ab97c1b58202d68aec65c5946f183d0f;hpb=69c19f570b0c7668d1374600a5a7cedaa81fbe2d;p=dsa-puppet.git diff --git a/modules/puppetmaster/lib/puppet/parser/functions/nodeinfo.rb b/modules/puppetmaster/lib/puppet/parser/functions/nodeinfo.rb index 947caf70..28da4aea 100644 --- a/modules/puppetmaster/lib/puppet/parser/functions/nodeinfo.rb +++ b/modules/puppetmaster/lib/puppet/parser/functions/nodeinfo.rb @@ -1,38 +1,60 @@ module Puppet::Parser::Functions newfunction(:nodeinfo, :type => :rvalue) do |args| - host = args[0] yamlfile = args[1] + begin - require '/var/lib/puppet/lib/puppet/parser/functions/ldapinfo.rb' - require '/var/lib/puppet/lib/puppet/parser/functions/whohosts.rb' + require '/var/lib/puppet/lib/puppet/parser/functions/ldapinfo.rb' + require '/var/lib/puppet/lib/puppet/parser/functions/whohosts.rb' - results = function_yamlinfo(host, yamlfile) - results['ldap'] = function_ldapinfo(host, '*') - unless results['ldap']['ipHostNumber'] - raise Puppet::ParseError, "Host #{host} does not have ipHostNumber values in ldap" - end - results['hoster'] = whohosts(results['ldap']['ipHostNumber'], "/etc/puppet/modules/debian-org/misc/hoster.yaml") - - results['misc'] = {} - fqdn = lookupvar('fqdn') - if fqdn and fqdn == host - v4ips = lookupvar('v4ips') - if v4ips - results['misc']['v4addrs'] = v4ips.split(',') - - # find out if we are behind nat - intersection = results['misc']['v4addrs'] & results['ldap']['ipHostNumber'] - results['misc']['natted'] = intersection.empty? + nodeinfo = function_yamlinfo(host, yamlfile) + nodeinfo['ldap'] = function_ldapinfo(host, '*') + unless nodeinfo['ldap']['ipHostNumber'] + raise Puppet::ParseError, "Host #{host} does not have ipHostNumber values in ldap" + end + nodeinfo['hoster'] = function_whohosts(nodeinfo['ldap']['ipHostNumber'], "/etc/puppet/modules/debian-org/misc/hoster.yaml") + + nodeinfo['misc'] = {} + fqdn = lookupvar('fqdn') + if fqdn and fqdn == host + v4ips = lookupvar('v4ips') + if v4ips + nodeinfo['misc']['v4addrs'] = v4ips.split(',') + + # find out if we are behind nat + intersection = nodeinfo['misc']['v4addrs'] & nodeinfo['ldap']['ipHostNumber'] + nodeinfo['misc']['natted'] = intersection.empty? + end + + v6ips = lookupvar('v6ips') + if v6ips and v6ips != "no" + nodeinfo['misc']['v6addrs'] = v6ips.split(',') + end end - v6ips = lookupvar('v6ips') - if v6ips and v6ips != "no" - results['misc']['v6addrs'] = v6ips.split(',') + if not nodeinfo['hoster']['nameservers'] or nodeinfo['hoster']['nameservers'].empty? + # no nameservers known for this hoster + nodeinfo['misc']['resolver-recursive'] = true + + if nodeinfo['hoster']['allow_dns_query'] + raise Puppet::ParseError, "No nameservers listed for #{nodeinfo['hoster']['name']} yet we should answer somebody's queries? That makes no sense." + end + elsif (nodeinfo['hoster']['nameservers'] & nodeinfo['misc']['v4addrs']).size > 0 or + (nodeinfo['hoster']['nameservers'] & nodeinfo['misc']['v6addrs']).size > 0 + # this host is listed as a nameserver at this location + nodeinfo['misc']['resolver-recursive'] = true + + if not nodeinfo['hoster']['allow_dns_query'] or nodeinfo['hoster']['allow_dns_query'].empty? + raise Puppet::ParseError, "Host #{host} is listed as a nameserver for #{nodeinfo['hoster']['name']} but no allow_dns_query networks are defined for this location" + end + else + nodeinfo['misc']['resolver-recursive'] = false end - end - return(results) + return(nodeinfo) + rescue => e + raise Puppet::ParseError, "Error in nodeinfo for node #{host}, yamlfile #{yamlfile}: #{e.message}\n#{e.backtrace}" + end end end