Skip to content
This repository has been archived by the owner on Mar 23, 2019. It is now read-only.

Fixes #658 by adding ~/.ansible/roles to conductor ANSIBLE_ROLES_PATH #672

Closed

Conversation

marcusianlevine
Copy link
Contributor

@marcusianlevine marcusianlevine commented Jul 30, 2017

ISSUE TYPE
  • Bugfix Pull Request
SUMMARY

With a debian:jessie conductor base from the latest develop commit, Ansible Galaxy is installing roles into a directory that ansible-container does not anticipate, and therefore builds with Galaxy roles in requirements.yml fail because the roles have been installed to, in my case /root/.ansible/roles, which is more generally ~/.ansible/roles — but AC only expects roles to appear in /src/roles or /etc/ansible/roles, or else the mounted volume provided by --roles-path in the CLI.

This pull request simply adds ~/.ansible/roles to the set of roles paths passed to the conductor.

See my comments on #658 for error output and description of the problem.

@chouseknecht
Copy link
Contributor

@marcusianlevine

Thanks for using Ansible Container. We appreciate your time and work on this.

There was actually a bug in the devel branch of Ansble, which as fixed with PR #27524. Roles are now once again installed to /etc/ansible/roles.

In light of the above, I'm closing this.

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.

2 participants