Allow host declaration without IP. #210
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request (PR) description
I often use dhcp to allocate hostnames or to distribute other options (eg routers) where I don't care about the IP address. I tried similar code to the above, but (using hiera) I received a message when I did not include an IP address:
Evaluation Error: Error while evaluating a Resource Statement, Dhcp::Host[LGwebOSTV]: expects a value for parameter 'ip' (file: /etc/puppetlabs/code/environments/production/modules/dhcp/manifests/init.pp, line: 289)
I have worked around this by setting the IP address to 0.0.0.0 and then ignoring it in the template but this seems like a kludge!
This Pull Request (PR) fixes the following issues
Fixes #106