Skip to content
This repository has been archived by the owner on Jul 17, 2023. It is now read-only.

Add galaxy_roles also to roles directory. #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kbidarkar
Copy link
Collaborator

@kbidarkar kbidarkar commented Jul 25, 2018

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

@kbidarkar kbidarkar requested a review from ehelms July 25, 2018 07:18
@kbidarkar
Copy link
Collaborator Author

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

@ehelms
Copy link

ehelms commented Jul 25, 2018

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
Copy link

@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
Copy link

ehelms commented Jul 31, 2018

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.

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.

3 participants