-
Notifications
You must be signed in to change notification settings - Fork 22
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
Allow roles to be more easily added #86
Comments
Ansible tasks are code. I'd rather that they were in a VCS than a datastore. I'm not convinced this would make things easier enough to warrant the added complexity and deviation from normal deployment mechanisms. |
Nah, they're YAML files 😛 In my opinion this is putting config in code - which is fine for things that don't change often, which is not true in this case. |
^ but I would also advocate for keeping changes in version control. |
Originally we said that roles etc should be in a different (possibly multiple) git repos. I agree with @philwills that it's still code, but it seems to make sense to me to decouple the amigo app from the roles. |
I.e. extract them from the Amigo code so a deploy is not required to update the availble roles.
Some suggestions:
The text was updated successfully, but these errors were encountered: