search

SatelliteQE / ansible-satellite6

Ansible playbooks for Satellite 6 systems' management
GNU General Public License v3.0
4 stars 10 forks source link

Add galaxy_roles also to roles directory. #7

Open kbidarkar opened 6 years ago

kbidarkar commented 6 years ago

@flatfender was suggesting we have both galaxy_roles and roles together.

kbidarkar commented 6 years ago

ansible-galaxy will install roles by default into the first found path in roles_path.

ehelms commented 6 years ago

I think this has the potential to cause developers to commit galaxy roles which is not what you want. I'd suggest keeping the current pattern or installing them into roles/galaxy_roles and ensuring in either case the directory is in .gitignore.

flatfender commented 6 years ago

@ehelms As I understand things, if you pull in a galaxy role in tower, it will put it in the roles directory as opposed the roles/galaxy_roles directory? Seems like we don't need roles/galaxy_roles and that keeping that will cause more confusion. Have you seen this?

ehelms commented 6 years ago

Can you configure Tower where to put galaxy roles? I also would not make using this repository directly more confusing just because of Tower. Not everyone will use Tower with this repository and you can likely configure Tower more directly than the ansible.cfg here.