Bug #122
closed
External nodes script shouldn't output anything if node not found
Added by Frank Sweetser about 15 years ago.
Updated about 15 years ago.
Description
If I'm reading http://reductivelabs.com/trac/puppet/wiki/ExternalNodes correctly, an external nodes script should produce no output in the case where a node is not found. This patch modifies the external node script to only output the response body if the return HTTP code is 200, and output a brief message to stderr otherwise. It also updates routes.rb to not choke on the puppetmaster node search (host.domain, host, default).
Files
- Status changed from New to Assigned
- Target version set to 0.1-3
I'll apply the patch, but just one minor thing to note, the regexp will allow to query for short names as well, currently we only support querying for fqdn values (e.g. if you query for plain "hostname" it will always return 404)
I figured that it made sense to do that for two reasons - one, to handle the short hostname check that puppetmaster does if it can't find the FQDN, and two, for sites that don't use the hostname to identify the puppet node (I saw at least one place that generates a GUID for each node).
Either way, thanks =)
- Status changed from Assigned to Ready For Testing
- % Done changed from 0 to 100
- Status changed from Ready For Testing to Closed
Also available in: Atom
PDF