-
Notifications
You must be signed in to change notification settings - Fork 825
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
Easy config using group_vars instead of role-bound vars #39
Comments
Another nice thing about having a single config file is that it would make merge conflicts less likely when updating sovereign. |
I like it. It does decrease the "modularity" of each of the roles, and it isn't in keeping with Ansible conventions, but hey rules are made to be broken ;) I'd love to see a pull request! |
I agree, it would be really convenient to have one global file to override configuration from all the roles. That file could even be added to |
My pull request above is a proposal using Obviously it needs documentation, but I don't want to document this if it's a wrong way to do this :) |
We could gather all vars in a single file, and even use string replacement to make config as simple as:
https://gist.github.com/lisael/f9af1d96b1323c56ec8c
this allow the user to keep all control over her config but also provides a sensible default config with minimal pain.
The text was updated successfully, but these errors were encountered: