-
-
Notifications
You must be signed in to change notification settings - Fork 645
Conversation
This fixes an issue that was recently created by the release of a new version of the script here: https://bootstrap.pypa.io/get-pip.py.
Fixes #2138 - tried it out and provisioning works tickety-boo. |
I had a quick play and whilst Python3.6.9 is installed, pip3 doesn't seem to exist on the base box by default. |
I may be misunderstanding what you're saying, but I see the same thing. But my understanding is that |
@@ -111,6 +111,7 @@ Vagrant.configure('2') do |config| | |||
config.vm.provision 'drupalvm', type: provisioner do |ansible| | |||
ansible.compatibility_mode = '2.0' | |||
ansible.playbook = playbook | |||
ansible.install_mode = 'pip3' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahhhh... in my previous readings I was thinking this was changing something inside the VM with the pip install that may happen as a result of Drupal VM's playbooks.
This is changing the Vagrantfile so that installs of ansible in the VM if you need it installed inside will work correctly.
Thanks for catching this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! Provisioning works 👍
Might've misunderstood something here or in #2138 but with dev-master, I'm still seeing the following on vagrant up and vagrant provision:
My local environment:
|
@jasloe - Oops! Sorry about that; it should only apply for |
This fixes an issue that was recently created by the release of a new version of the script here: https://bootstrap.pypa.io/get-pip.py.
TL;DR — they dropped support for Python2 and this now requires pip3. Maybe worth noting I may have had to manually install Python3 via SSH into the box, but I may have also already had it and it was a step I just duplicated. Either way — this fixes issues folks should be having as of this last Saturday.