Skip to content
This repository was archived by the owner on Apr 7, 2025. It is now read-only.

Feature/default playbook path#64

Open
astorath wants to merge 4 commits into
ahelal:masterfrom
astorath:feature/default-playbook-path
Open

Feature/default playbook path#64
astorath wants to merge 4 commits into
ahelal:masterfrom
astorath:feature/default-playbook-path

Conversation

@astorath

Copy link
Copy Markdown

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

ahelal commented Jan 28, 2021

Copy link
Copy Markdown
Owner

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

ahelal commented Jan 29, 2021

Copy link
Copy Markdown
Owner

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