Merge pull request #370 from edx/jarv/fix-shell-commands

Jarv/fix shell commands

playbooks/roles/discern/tasks/deploy.yml

@@ -56,13 +56,15 @@
     - deploy
 
 - name: discern | install ease python package
-  shell: command="{{discern_venv_dir}}/bin/activate; cd {{ discern_ease_code_dir }}; python setup.py install"
+  shell: >
+    {{ discern_venv_dir }}/bin/activate; cd {{ discern_ease_code_dir }}; python setup.py install
   tags:
     - deploy
 
 #Needed for the ease package to work
 - name: discern | install nltk data using rendered shell script
-  shell: command="{{ discern_venv_dir }}/bin/python -m nltk.downloader -d {{ discern_nltk_data_dir }} all"
+  shell: >
+    {{ discern_venv_dir }}/bin/python -m nltk.downloader -d {{ discern_nltk_data_dir }} all
   sudo_user: "{{ discern_user }}"
   tags:
     - deploy
@@ -71,7 +73,7 @@
 #support virtualenvs as of this comment
 - name: discern | django syncdb migrate and collectstatic for discern
   shell: >
-    command="{{ discern_venv_dir }}/bin/python {{discern_code_dir}}/manage.py {{item}} --noinput --settings={{discern_settings}} --pythonpath={{discern_code_dir}}"
+    {{ discern_venv_dir }}/bin/python {{discern_code_dir}}/manage.py {{item}} --noinput --settings={{discern_settings}} --pythonpath={{discern_code_dir}}
       chdir={{ discern_code_dir }}
   sudo_user: "{{ discern_user }}"
   with_items:
@@ -83,7 +85,7 @@
 #Have this separate from the other three because it doesn't take the noinput flag
 - name: discern | django update_index for discern
   shell: >
-    command="{{ discern_venv_dir}}/bin/python {{discern_code_dir}}/manage.py update_index --settings={{discern_settings}} --pythonpath={{discern_code_dir}}"
+    {{ discern_venv_dir}}/bin/python {{discern_code_dir}}/manage.py update_index --settings={{discern_settings}} --pythonpath={{discern_code_dir}}
       chdir={{ discern_code_dir }}
   sudo_user: "{{ discern_user }}"
   tags:

playbooks/roles/edxapp/tasks/service_variant_config.yml

@@ -55,7 +55,7 @@
 
 - name: edxapp | gather {{ item }} static assets with rake
   shell: >
-    command=SERVICE_VARIANT={{ item }} rake {{ item }}:gather_assets:aws
+    SERVICE_VARIANT={{ item }} rake {{ item }}:gather_assets:aws
     executable=/bin/bash
     chdir={{ edxapp_code_dir }}
   sudo_user: "{{ edxapp_user }}"

playbooks/roles/ora/tasks/ease.yml

@@ -53,7 +53,8 @@
   - deploy
 
 - name: ora | install ease python package
-  shell: command="{{ora_ease_venv_dir}}/bin/activate; cd {{ora_ease_code_dir}}; python setup.py install"
+  shell: >
+    {{ ora_ease_venv_dir }}/bin/activate; cd {{ ora_ease_code_dir }}; python setup.py install
   sudo_user: "{{ ora_user }}"
   tags:
   - deploy