-
Notifications
You must be signed in to change notification settings - Fork 44
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
Add parameter to manage puppetmaster service status #171
base: production
Are you sure you want to change the base?
Conversation
@rendhalver when do you think a new release is going to be done? This would help/unblock us big time. |
Sounds like a useful addition. |
Thanks a lot, I've got a workaround working by installing dependencies manually and then installing the module from a .tar.gz so no rush on the release if not needed. |
I am going to keep this open till I can make that new release. |
@rendhalver Sorry to bother, are you going to make a release soon? I want to check and try and get hiera5 support as well in this module. |
Thanks for the reminder. |
@@ -29,8 +30,8 @@ | |||
} | |||
|
|||
service { 'puppetserver': | |||
ensure => 'running', | |||
enable => true, | |||
ensure => $service_enable, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will setting this to true work with all versions?
Some may not like it.
Add parameter to manage puppetmaster service status. This way the module can manage if puppetserver should be running or not.
By default will be running but it can be stopped if needed, for example when used on a docker instance.