Merge pull request #202 from edx/jarv/fix-github-users

Jarv/fix github users

playbooks/edx-east/edx_sandbox_cn.yml

----
-
-# This playbook is to configuration
-# the official edX sandbox instance
-# sandbox.edx.org
-
-- name: Configure instance(s)
-  hosts: tag_Name_sandbox-cn
-  sudo: True
-  gather_facts: True
-  vars:
-    migrate_db: True
-    mysql5_workaround: True
-  vars_files:
-    - "{{ secure_dir }}/vars/edx_sandbox.yml"
-  roles:
-    - common
-    - nginx
-    - edxlocal
-    - edxapp
-    - rabbitmq
-    - oraclejdk
-    - elasticsearch
-    - { role: 'edxapp', celery_worker: True }
-    - role: rbenv
-      rbenv_user: "{{ forum_user }}"
-      rbenv_user_home: "{{ forum_home }}"
-      rbenv_ruby_version: "{{ forum_ruby_version }}"
-    - forum

playbooks/roles/common/tasks/create_github_users.yml

@@ -18,7 +18,7 @@
 - name: common | create local user for github user
   user: 
     name={{ item.user }} 
-    {% if item.groups %}groups={{ ",".join(item.groups) }}{% endif %}
+    groups={{ ",".join(item.groups) }}
     shell=/bin/bash
   with_items: github_users
   tags:
@@ -42,4 +42,4 @@
   with_items: github_users
   tags:
     - users
-    - update
+    - update
\ No newline at end of file

playbooks/roles/common/tasks/main.yml

 ---
 - include: create_users.yml
 - include: create_github_users.yml
+  when: github_users is defined
 
 - name: common | Add user www-data
   # This user should be created on the system by default

playbooks/roles/edxapp/tasks/deploy.yml

@@ -96,7 +96,7 @@
   # Need to use shell rather than pip so that we can maintain the context of our current working directory; some
   # requirements are pathed relative to the edx-platform repo. Using the pip from inside the virtual environment implicitly
   # installs everything into that virtual environment.
-  shell: cd {{ edx_platform_code_dir }} && {{ venv_dir }}/bin/pip install --use-mirrors -r {{ base_requirements_file }}
+  shell: cd {{ edx_platform_code_dir }} && {{ venv_dir }}/bin/pip install --exists-action w --use-mirrors -r {{ base_requirements_file }}
   tags:
   - lms
   - cms
@@ -117,7 +117,7 @@
   # Need to use shell rather than pip so that we can maintain the context of our current working directory; some
   # requirements are pathed relative to the edx-platform repo. Using the pip from inside the virtual environment implicitly
   # installs everything into that virtual environment.
-  shell: cd {{ edx_platform_code_dir }} && {{ venv_dir }}/bin/pip install --use-mirrors -r {{ item }}
+  shell: cd {{ edx_platform_code_dir }} && {{ venv_dir }}/bin/pip install --exists-action w --use-mirrors -r {{ item }}
   with_items:
   - "{{ repo_requirements_file }}"
   - "{{ github_requirements_file }}"
@@ -134,7 +134,7 @@
   # Need to use shell rather than pip so that we can maintain the context of our current working directory; some
   # requirements are pathed relative to the edx-platform repo. Using the pip from inside the virtual environment implicitly
   # installs everything into that virtual environment.
-  shell: cd {{ edx_platform_code_dir }} && {{ venv_dir }}/bin/pip install --use-mirrors -r {{ item }}
+  shell: cd {{ edx_platform_code_dir }} && {{ venv_dir }}/bin/pip install --exists-action w --use-mirrors -r {{ item }}
   with_items:
   - "{{ sandbox_base_requirements }}"
   - "{{ sandbox_local_requirements }}"