-
Notifications
You must be signed in to change notification settings - Fork 565
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 ensure parameter to jenkins::slave #150
Conversation
@carlossg could I convince you to add an rspec-puppet example or two to verify that passing a class parameter into the jenkins::slave class gets to the service correctly? |
Added. The travis build is broken from the previous commit. |
Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests |
do you need anything else? |
rebased |
it's failing presumably only because of the pending test? |
puppet-lint error. Previous builds used puppet-lint 0.3.2, this one used 1.0.1
|
lint issues fixed in #167 |
Allows running the slave manually instead of a service, useful in Docker for instance
Add ensure parameter to jenkins::slave
Allows running the slave manually instead of a service, useful in Docker for instance