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 #11048, ansible_local provisioner, fails to install Ansible on an Arch Linux box #11148

Open
wants to merge 1 commit into
base: master
from

Conversation

@skoenen
Copy link

skoenen commented Oct 23, 2019

FIX #11048, add parameter pip_install_cmd to ansible_install.rb for arch guest.

ADD: Handling of pip_install_cmd in ansible_install.rb for arch guest.

…or arch guest.

ADD: Handling of `pip_install_cmd` in `ansible_install.rb` for arch guest.
@hashicorp-cla

This comment has been minimized.

Copy link

hashicorp-cla commented Oct 23, 2019

CLA assistant check
All committers have signed the CLA.

@skoenen skoenen marked this pull request as ready for review Oct 23, 2019
@briancain briancain added this to the 2.2.7 milestone Oct 23, 2019
@gildegoma gildegoma self-assigned this Oct 23, 2019
@gildegoma gildegoma self-requested a review Oct 23, 2019
@gildegoma

This comment has been minimized.

Copy link
Collaborator

gildegoma commented Nov 20, 2019

@skoenen thank you for this pull request. No need to add anything else, as I include your commit in an upcoming pull request, which will cover the missing unit tests issue for all the ansible_local capabilities. I plan to propose this before end of this week...

//cc @briancain @chrisroberts

@gildegoma

This comment has been minimized.

Copy link
Collaborator

gildegoma commented Dec 1, 2019

I had to delay a little bit, but it's coming...

gildegoma added a commit that referenced this pull request Dec 1, 2019
GH-11148: Remove unwanted formatting changes proposed in 9203611.
@gildegoma

This comment has been minimized.

Copy link
Collaborator

gildegoma commented Dec 1, 2019

This change has been integrated into #11227 (though with a rebase, so I am not sure if this pull request will be automatically considered as merged when the new PR is pulled into master...).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.