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.

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

Feature/default playbook path #64

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

astorath
Copy link

I've added auto-discovery of ansible playbook using some default, e.g.:

test/integration/<suite_name>/ansible/converge.yml
test/integration/<suite_name>/converge.yml
test/integration/converge.yml

etc.

@ahelal
Copy link
Owner

ahelal commented Jan 28, 2021

Hi @astorath thank you for the PR. I am a bit out of the ansible game and kitchen.
is the standard now to use a converge.yml ? cause i think it was default.yml

@astorath
Copy link
Author

Hi @astorath thank you for the PR. I am a bit out of the ansible game and kitchen.
is the standard now to use a converge.yml ? cause i think it was default.yml

Well, I've brought this up from molecule. I believe kitchen-ansible do use deafult.yml though.

@ahelal
Copy link
Owner

ahelal commented Jan 29, 2021

@astorath Thanks for the clarification would be possible to also add default.yml in the lookup path ?
And add a couple of sentences in the readme to describe the behaviour ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants