-
Notifications
You must be signed in to change notification settings - Fork 14
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
salt-api support #7
Comments
No real preference to this, I just keep it as simple as possible |
Why can the additional files not go in master.d instead of concat conditionals ? |
Had a quick bash at it - seems to work as a quick proof of concept and it's a lot less messy than the concat option in the folk you have been working on. |
I like it, looks easy enough to be understood. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I was thinking about adding support for provisioning the rest_cherrypy salt-api component. The configuration for the rest_cherrypy component is about twenty additional lines in the /etc/salt/master file. None of the lines are included in the example file for /etc/salt/master.
I could either add support by putting the configuration lines directly into master.erb bounded by some conditional logic or I could use the puppetlabs/concat module to create the two fragments seperately and combine them in the end. I wanted to get your take on it and see if you had any preferences before I started working on the pull request, though.
The text was updated successfully, but these errors were encountered: