Skip to content
New issue

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

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

Already on GitHub? # to your account

Allow setting service restart command #143

Merged
merged 1 commit into from
Jul 16, 2019

Conversation

antaflos
Copy link
Contributor

@antaflos antaflos commented May 8, 2019

By default the service resource that manages the BIND service will
restart the service when notified by configuration changes. This can be
disruptive.

This change allows the user to set a custom restart command via
dns::service_restart_command for the service resource, such as
/usr/sbin/service bind9 reload or /bin/systemctl reload bind9.service
or even /usr/sbin/rndc reload, to avoid restarting the whole named
process. By default no restart command is set.

@antaflos
Copy link
Contributor Author

I have rebased the branch on the current master and force-pushed.

By default the service resource that manages the BIND service will
restart the service when notified by configuration changes. This can be
disruptive.

This change allows the user to set a custom restart command via
`dns::service_restart_command` for the service resource, such as
`/usr/sbin/service bind9 reload` or `/bin/systemctl reload bind9.service`
or even `/usr/sbin/rndc reload`, to avoid restarting the whole named
process. By default no restart command is set.
@antaflos antaflos force-pushed the service_restart_command branch from cf3506b to b30a6eb Compare June 5, 2019 10:25
@ekohl ekohl merged commit 6bb26ca into theforeman:master Jul 16, 2019
@ekohl
Copy link
Member

ekohl commented Jul 16, 2019

Thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants