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

move changed config files to master.d and minion.d #12

Closed
aboe76 opened this issue Aug 14, 2013 · 8 comments
Closed

move changed config files to master.d and minion.d #12

aboe76 opened this issue Aug 14, 2013 · 8 comments

Comments

@aboe76
Copy link
Member

aboe76 commented Aug 14, 2013

I would like to propose the following change:

if a config file is changed by pillar data put the file in:

/etc/salt/master.d/master.conf
/etc/salt/minion.d/minion.conf

So not to overwrite the defaults salt package provides ?
If i'm not mistaken salt will prefer the files in master.d and minion.d
because all defaults are commented.

@FrancoTampieri
Copy link

+1
I agree because is more flexible and optimal to automate the installation

@pwaller
Copy link

pwaller commented Jan 15, 2014

👍

3 similar comments
@elfixit
Copy link

elfixit commented Jan 17, 2014

+1

@giannello
Copy link

+1

@bbinet
Copy link

bbinet commented Jun 13, 2014

+1

@jacksoncage
Copy link
Contributor

+1, shouldn't be to hard to fix, let's see if I can get a pull request in.

@nmadhok
Copy link
Member

nmadhok commented Sep 8, 2014

@jacksoncage Did you have a pull request ready for this yet?

jacksoncage added a commit to jacksoncage/salt-formula that referenced this issue Sep 8, 2014
@jacksoncage
Copy link
Contributor

Had forgot about this, fixed in our environment but not send back yet.

nmadhok added a commit that referenced this issue Sep 8, 2014
#12 Moved master and minion config file to .d folder
@nmadhok nmadhok closed this as completed Sep 8, 2014
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

8 participants