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 slave defaults management to be optional #619

Closed
madAndroid opened this issue Aug 4, 2016 · 3 comments
Closed

Allow slave defaults management to be optional #619

madAndroid opened this issue Aug 4, 2016 · 3 comments
Labels
enhancement New feature or request

Comments

@madAndroid
Copy link
Contributor

We set a number of config options in the slave defaults file that don't follow the same structure as the current template - would you be open to allowing the management of the defaults file to be configurable? This would mean that we can use our own template, and manage the file outside of the scope of the slave manifest

madAndroid added a commit to madAndroid/puppet-jenkins that referenced this issue Aug 4, 2016
madAndroid added a commit to madAndroid/puppet-jenkins that referenced this issue Aug 4, 2016
@madAndroid
Copy link
Contributor Author

Alternatively, it may make more sense to make the location of the template configurable.

@jhoblitt
Copy link
Member

I think allow alternative templates would be pretty reasonable.

@jhoblitt jhoblitt added the enhancement New feature or request label Aug 17, 2016
@madAndroid
Copy link
Contributor Author

This is no longer required, the config options we were setting have now been exposed in the main template.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants