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 a type of playbook_command and add description more on doc #13187

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

maruloop
Copy link

When ansible was installed into rye or pipenv, I needed to specify playbook_command like the below.

OK:

ansible.playbook_command = ["rye", "run", "ansible-playbook"]

When I specified it as a String type, Vagrant raised an exception.

NG:

ansible.playbook_command = "rye run ansible-playbook"

@hashicorp-cla
Copy link

hashicorp-cla commented May 27, 2023

CLA assistant check
All committers have signed the CLA.

@maruloop maruloop force-pushed the fix_type_playbook_command_on_doc branch from faec2a9 to 31c7672 Compare May 27, 2023 05:05
@maruloop maruloop changed the title Fix a type of playbook_command and add description more Fix a type of playbook_command and add description more on doc May 27, 2023
@chrisroberts
Copy link
Member

@maruloop Do you have the exception information available that Vagrant generated?

@maruloop
Copy link
Author

maruloop commented Aug 3, 2023

@chrisroberts

In the Vagrantfile,

  config.vm.provision "ansible" do |ansible|
    ansible.playbook = "myplaybook.yml"
    ansible.playbook_command = "rye run ansible-playbook"
  end

I got this error.

The Ansible software could not be found! Please verify
that Ansible is correctly installed on your host system.

If you haven't installed Ansible yet, please install Ansible
on your host system. Vagrant can't do this for you in a safe and
automated way.
Please check https://docs.ansible.com for more information.

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

Successfully merging this pull request may close these issues.

3 participants