Merge pull request #3207 from edx/arbab/locust-rewrite

using proper YAML syntax

playbooks/roles/locust/tasks/main.yml

@@ -30,7 +30,7 @@
 #     roles:
 #       - locust
 
-- name: install system packages
+- name: Install system packages
   apt:
     pkg: "{{ item }}"
     state: "present"
@@ -38,42 +38,44 @@
     cache_valid_time: 3600
   with_items: locust_debian_pkgs
 
-- name: install application requirements
-  pip: >
-    requirements="{{ locust_requirements_base }}/{{ item }}"
-    virtualenv="{{ locust_home }}/venvs/{{ locust_service_name }}"
-    state=present
+- name: Install application requirements
+  pip:
+    requirements: "{{ locust_requirements_base }}/{{ item }}"
+    virtualenv: "{{ locust_home }}/venvs/{{ locust_service_name }}"
+    state: present
   become_user: "{{ locust_user }}"
-  with_items: locust_requirements
+  with_items: "{{ locust_requirements }}"
 
-- name: write supervisord config
-  template: >
-    src=edx/app/supervisor/conf.d.available/locust.conf.j2
-    dest="{{ supervisor_available_dir }}/{{ locust_service_name }}.conf"
-    owner={{ supervisor_user }} group={{ common_web_user }} mode=0644
+- name: Write supervisord config
+  template:
+    src: "edx/app/supervisor/conf.d.available/locust.conf.j2"
+    dest: "{{ supervisor_available_dir }}/{{ locust_service_name }}.conf"
+    owner: "{{ supervisor_user }}"
+    group: "{{ common_web_user }}"
+    mode: "0644"
 
-- name: enable supervisor script
-  file: >
-    src={{ supervisor_available_dir }}/{{ locust_service_name }}.conf
-    dest={{ supervisor_cfg_dir }}/{{ locust_service_name }}.conf
-    state=link
-    force=yes
+- name: Enable supervisor script
+  file:
+    src: "{{ supervisor_available_dir }}/{{ locust_service_name }}.conf"
+    dest: "{{ supervisor_cfg_dir }}/{{ locust_service_name }}.conf"
+    state: link
+    force: yes
   when: not disable_edx_services
 
-- name: update supervisor configuration
+- name: Update supervisor configuration
   shell: "{{ supervisor_ctl }} -c {{ supervisor_cfg }} update"
   when: not disable_edx_services
 
-- name: restart the applicaton
-  supervisorctl: >
-    state=restarted
-    supervisorctl_path={{ supervisor_ctl }}
-    config={{ supervisor_cfg }}
-    name={{ locust_service_name }}
+- name: Restart the applicaton
+  supervisorctl:
+    name: "{{ locust_service_name }}"
+    state: restarted
+    supervisorctl_path: "{{ supervisor_ctl }}"
+    config: "{{ supervisor_cfg }}"
   when: not disable_edx_services
   become_user: "{{ supervisor_service_user }}"
 
-- name: increase file descriptor limit of the system (Session Logout and Login would be required)
+- name: Increase file descriptor limit of the system (Session Logout and Login would be required)
   lineinfile:
     dest: "{{ ulimit_conf_file }}"
     line: "{{ item.domain }}  {{ item.type }}  {{ item.item }}  {{ item.value }}"