Skip to content
This repository was archived by the owner on Dec 18, 2020. It is now read-only.

Updates to make it clearer#3

Open
dpshelio wants to merge 5 commits intomdavezac:masterfrom
dpshelio:updates
Open

Updates to make it clearer#3
dpshelio wants to merge 5 commits intomdavezac:masterfrom
dpshelio:updates

Conversation

@dpshelio
Copy link

Few changes I've been doing while trying to understand how this works...

Also removed some imports that were not used and DRY-ed the
server_hierarchy function.
Also remove the re-set of `__opts__['file_client']` as it was already
setup on `minion()`.
pkg was set for master config as brew, here will choose the correct one
for the different linux distributions.
'user: {user}\n'
'sudo_user: {sudo_user}\n'
'pkg: brew\n'
'pkg: {pkg}\n'
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would have imagine salt is smart enough to don't need this...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant