Merge "Fixing virtualenv issues"

playbooks/roles/bifrost-ironic-install/tasks/pip_install.yml

@@ -18,7 +18,7 @@
     name: "{{ package }}"
     state: "{{ state | default(omit) }}"
     version: "{{ version | default(omit) }}"
-    virtualenv: "{{ bifrost_venv_dir if enable_venv else 'undefined' }}"
+    virtualenv: "{{ bifrost_venv_dir if enable_venv else omit }}"
   when: source_install is not defined or source_install == false
 # NOTE (cinerama): We should be able to use the pip module here and
 # possibly merge these two tasks when

scripts/env-setup.sh

@@ -30,9 +30,7 @@ if [ -x '/usr/bin/apt-get' ]; then
     fi
     if [ -n "${VENV-}" ]; then
         if ! $(virtualenv --version &>/dev/null); then
-            if $(sudo -H apt-get -y install virtualenv); then
-                sudo -H apt-get -y install python-virtualenv
-            fi
+            sudo -H apt-get -y install python-virtualenv
         fi
     fi
 elif [ -x '/usr/bin/yum' ]; then