Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Check ANSIBLE_ROLES_PATH before basedir #478

Merged
merged 3 commits into from
Feb 7, 2019
Merged

Check ANSIBLE_ROLES_PATH before basedir #478

merged 3 commits into from
Feb 7, 2019

Commits on Feb 6, 2019

  1. Test: use testproject/roles instead of roles/roles 

    'roles' doesn't seem an adequate toplevel name when testing three
directories depth.

Signed-off-by: Pierre-Louis Bonicoli <pierre-louis.bonicoli@libregerbil.fr>
    @pilou-
    pilou- committed Feb 6, 2019
    Configuration menu
    Copy the full SHA
    649698f View commit details
    Browse the repository at this point in the history

  2. Test: use ansible-lint the way molecule does 

    Signed-off-by: Pierre-Louis Bonicoli <pierre-louis.bonicoli@libregerbil.fr>
    @pilou-
    pilou- committed Feb 6, 2019
    Configuration menu
    Copy the full SHA
    36f40dd View commit details
    Browse the repository at this point in the history

  3. Check DEFAULT_ROLES_PATH before basedir 

    Fix behavior when ANSIBLE_ROLES_PATH is defined, for example:

    cd test/roles/test-role
    ANSIBLE_ROLES_PATH=.. ansible-lint molecule/default/include-import-role.yml

Fix ansible/molecule#1723

Signed-off-by: Pierre-Louis Bonicoli <pierre-louis.bonicoli@libregerbil.fr>
    @pilou-
    pilou- committed Feb 6, 2019
    Configuration menu
    Copy the full SHA
    c80ad2d View commit details
    Browse the repository at this point in the history