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

fix configuration of playbook path #14

Merged
merged 1 commit into from Jun 1, 2015
Merged

fix configuration of playbook path #14

merged 1 commit into from Jun 1, 2015

Conversation

chris-rock
Copy link
Member

This PR fixes the test run and configures the playbook path for the ansible provisioner.

@chris-rock chris-rock force-pushed the fix-playbook-path branch 2 times, most recently from cf91e24 to 0ca2172 Compare June 1, 2015 12:29
@rndmh3ro
Copy link
Member

rndmh3ro commented Jun 1, 2015

This seems to be fine, but I don't know the reason behing this PR. Care to explain?

@chris-rock
Copy link
Member Author

I downloaded the latest ansible-playbook gem and it could not find the playbook path in our integration tests anymore.

@rndmh3ro
Copy link
Member

rndmh3ro commented Jun 1, 2015

I cannot reproduce the issue. I did a gem update, deleted the Gemfile.lock and still I could run the tests.
What is the ansible-playbook gem you're talking about?

@chris-rock
Copy link
Member Author

sorry, I was talking about kitchen-ansible. I used the style mentioned in https://github.com/neillturner/kitchen-ansible/blob/master/README.md#testing-multiple-playbooks and this worked on my machine.

rndmh3ro pushed a commit that referenced this pull request Jun 1, 2015
fix configuration of playbook path
@rndmh3ro rndmh3ro merged commit c5e470c into master Jun 1, 2015
@rndmh3ro
Copy link
Member

rndmh3ro commented Jun 1, 2015

In this case it makes sense!

@rndmh3ro rndmh3ro deleted the fix-playbook-path branch June 1, 2015 19:06
rndmh3ro pushed a commit that referenced this pull request Jul 24, 2020
Add meta-information for Ansible Galaxy
rndmh3ro added a commit that referenced this pull request Jul 24, 2020
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.

None yet

2 participants