Skip to content

Commit

Permalink
Don't install virtualenv as user.
Browse files Browse the repository at this point in the history
  • Loading branch information
root authored and Federico Ressi committed Jun 7, 2018
1 parent 3281824 commit 5893944
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions infrared/common/roles/shade/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
pip:
name: virtualenv
state: forcereinstall
extra_args: --user

# required to avoid any conflicts in venv with
# different packages versions (especially shade)
Expand All @@ -31,12 +30,11 @@
# do not use shimming for these two special packages
- name: shade virtualenv core deps
pip:
name: "{{ item }}"
name:
- setuptools>=39.0.0
- pip>=10.0.1
virtualenv: "{{ shade_path_venv }}"
virtualenv_command: "{{ ansible_env.HOME }}/.local/bin/virtualenv"
with_items:
- setuptools>=39.0.0
- pip>=10.0.1

- name: shade virtualenv deps
pip:
Expand All @@ -48,4 +46,3 @@
# see https://projects.engineering.redhat.com/browse/RHOSINFRA-1562
# TODO: unpin shade to avoid other conflicts
virtualenv: "{{ shade_path_venv }}"
virtualenv_command: "{{ ansible_env.HOME }}/.local/bin/virtualenv"

0 comments on commit 5893944

Please sign in to comment.