Another stab to get multi-python support for virtualenv correctly

Author: miohtama

defaults/main.yml

@@ -10,7 +10,8 @@ sentry_https_url: no
 sentry_secret_key: 1LsmGR1DIyCJ5n2bRG5IVOFHdzEPkTKlW0RzxZVe9S0vc
 sentry_extensions: []                                     # List of sentry-extensions
 
-sentry_virtualenv_command: virtualenv2.7                  # In the case of multiple Python installations
+# Python configuration
+sentry_python: python2.7                                  # In the case of multiple Python  installations
                                                           # Pick one for Sentry using specific virtualenv command
 
 sentry_ssl_certificate:                                   # SSL certificate file - also turns on HTTPS on Nginx

tasks/sentry.yml

@@ -8,10 +8,18 @@
 
 - name: Install Dependencies
   apt: name={{item}}
-  with_items: [libxml2-dev, libxslt1-dev, zlib1g-dev,libffi-dev, python-virtualenv, python-dev]
+  with_items: [libxml2-dev, libxslt1-dev, zlib1g-dev, libffi-dev]
+
+# Try to create virtualenv only if it doesn't exist.
+# This way we can rerun the playbook without errors.
+# true && hack at the start of the line, otherwise one gets YAML syntax errors
+- name: Create virtualenv for Sentry using chosen Python
+  shell: true && [[ ! -d {{sentry_home}}/env ]] || virtualenv --python={{sentry_python}} {{sentry_home}}/env
+  sudo: yes
+  sudo_user: "{{sentry_user}}"
 
 - name: Install Sentry
-  pip: name=sentry executable={{sentry_home}}/env/bin/pip version={{sentry_version}} virtualenv_command="{{sentry_virtualenv_command}}" virtualenv="{{sentry_home}}"/env
+  pip: name=sentry executable={{sentry_home}}/env/bin/pip version={{sentry_version}}  virtualenv="{{sentry_home}}"/env
   sudo: yes
   sudo_user: "{{sentry_user}}"