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

salt-api support #7

Open
dudemcbacon opened this issue Mar 30, 2015 · 4 comments
Open

salt-api support #7

dudemcbacon opened this issue Mar 30, 2015 · 4 comments

Comments

@dudemcbacon
Copy link

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.

@aboe76
Copy link
Owner

aboe76 commented Mar 30, 2015

No real preference to this, I just keep it as simple as possible

@eperdeme
Copy link

eperdeme commented May 6, 2015

Why can the additional files not go in master.d instead of concat conditionals ?

@eperdeme
Copy link

eperdeme commented May 6, 2015

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.
https://github.com/eperdeme/puppet-salt/

@aboe76
Copy link
Owner

aboe76 commented May 6, 2015

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
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants