Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reload zones when they are updated #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

crispygoth
Copy link

Simply adds an exec resource to nsd::zone that runs nsd-control reload zonename whenever Puppet updates the file or the nsd::zone resource is a target of a notify/subscribe relationship.

@coveralls
Copy link

Coverage Status

Coverage remained the same at ?% when pulling 6c4c1e7 on crispygoth:reload_zone_when_updated into 8f61a4c on bodgit:master.

@bodgit
Copy link
Owner

bodgit commented Nov 29, 2018

This possibly needs to require the service class being up otherwise I'm guessing it will error if nsd-control can't contact the main nsd process on a clean install.

Alternatively, add whatever exit code nsd-control returns when nsd isn't running to the accepted list of error codes. If nsd isn't yet running then it's going to (re)load all of the zones anyway when it starts.

@bodgit bodgit self-assigned this Nov 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants