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

Added sensu::manage_services to optionally disable internal service management. #113

Merged
merged 2 commits into from
Dec 1, 2013

Conversation

vmadman
Copy link
Contributor

@vmadman vmadman commented Nov 14, 2013

This commit adds an extra variable to the sensu class, $manage_services (bool, default=true). When set to FALSE the module will not attempt to create or modify services.

This feature will allow external service management, as with DaemonTools and SupervisorD.

@jamtur01
Copy link
Contributor

Hi - thanks for your contribution! Can you review the test failures please and fix? Thanks! 👍

@vmadman
Copy link
Contributor Author

vmadman commented Nov 18, 2013

Alright, fixed that. I'm still rather new to Puppet and definitely not a Ruby developer.. so I apologize for pushing shotty code. I was not able to get rspec-puppet working on my windows machine, or dev vm (Vagrant/CentOS 6.4) so I had to take a shot in the dark. If you know of any articles that might point me in the right direction on how to get setup for puppet testing I would appreciate it.

jamtur01 added a commit that referenced this pull request Dec 1, 2013
Added sensu::manage_services to optionally disable internal service management.
@jamtur01 jamtur01 merged commit c138296 into sensu:master Dec 1, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants