-
Notifications
You must be signed in to change notification settings - Fork 116
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
Consider using requirements.yml and with ansible-galaxy to handle roles #140
Comments
It might be a good idea to at least support the To be honest after using |
Absolutley agree with the update command over using ansible-galaxy directly. I set that function up in demos-update to keep the playbook behavior, while getting more flexibility from the roles and using the locally configured roles_path at the same time. There may be a problem in the future if a |
There's currently an issue here for ansible-galaxy to track repos: ansible/ansible#12353 |
Newer versions of ansible have a nice requirements format, making the debops-update script more flexible.
I made a function here making use of it: https://github.com/umeboshi2/demosthenes/blob/master/demosthenes/scripts/demos_update.py#L103
A potential downside is that roles in version control are tarballed and extracted, rather than cloned into the roles_path.
I took the time to make a simple conversion of the galaxy requirements files in the debops-playbooks repo. debops/debops-playbooks#217
The text was updated successfully, but these errors were encountered: