Merge pull request #3139 from edx/arbab/mode-fix

fix the file/template mode while using with_items

playbooks/roles/aws/tasks/main.yml

@@ -108,7 +108,7 @@
 - name: Update the ssh motd on Ubuntu
   file:
     path: "{{ item }}"
-    mode: 0644
+    mode: "0644"
   when: (vagrant_home_dir.stat.exists == false) and (ansible_distribution in common_debian_variants)
   with_items:
     - "/etc/update-motd.d/10-help-text"

playbooks/roles/edx_ansible/tasks/deploy.yml

@@ -28,7 +28,7 @@
     src: "{{ item.src }}"
     owner: "{{ edx_ansible_user }}"
     group: "{{ edx_ansible_user }}"
-    mode: 755
+    mode: "0755"
   with_items:
     - { src: 'update.j2', dest: 'update' }
     - { src: 'show-repo-heads.j2', dest: 'show-repo-heads' }

playbooks/roles/edxapp/tasks/main.yml

@@ -111,7 +111,7 @@
     state: directory
     owner: "{{ common_log_user }}"
     group: "{{ common_log_user }}"
-    mode: 0750
+    mode: "0750"
   with_items: "{{ service_variants_enabled }}"
   tags:
     - install

playbooks/roles/edxapp/tasks/service_variant_config.yml

@@ -137,7 +137,7 @@
       state: "{{ item }}"
       owner: "{{ edxapp_user }}"
       group: "{{ common_web_group }}"
-      mode:  0755
+      mode:  "0755"
   when: celery_worker is not defined and not devstack
   with_items: ['absent', 'directory']
   tags:

playbooks/roles/flower/tasks/main.yml

@@ -28,7 +28,7 @@
     dest: "{{ item.dest }}"
     owner: "{{ FLOWER_USER }}"
     group: "{{ common_web_group }}"
-    mode: 0640
+    mode: "0640"
   notify:
     - restart flower
   with_items:

playbooks/roles/nginx/tasks/main.yml

@@ -113,7 +113,7 @@
     dest: "{{ nginx_sites_available_dir }}/{{ item }}"
     owner: root
     group: "{{ common_web_user }}"
-    mode: 0640
+    mode: "0640"
   with_items: "{{ nginx_sites }}"
   notify: reload nginx
   tags:
@@ -139,7 +139,7 @@
     dest: "{{ nginx_sites_available_dir }}/{{ (item | basename).rstrip('.j2') }}"
     owner: root
     group: "{{ common_web_user }}"
-    mode: 0640
+    mode: "0640"
   with_items: "{{ nginx_extra_sites }}"
   notify: reload nginx
   tags:
@@ -165,7 +165,7 @@
     dest: "{{ nginx_conf_dir }}/{{ (item | basename).rstrip('.j2') }}"
     owner: root
     group: "{{ common_web_user }}"
-    mode: 0640
+    mode: "0640"
   with_items: "{{ nginx_extra_configs }}"
   notify: reload nginx
   tags:
@@ -178,7 +178,7 @@
     dest: "{{ nginx_sites_available_dir }}/{{ item.key }}"
     owner: root
     group: "{{ common_web_user }}"
-    mode: 0640
+    mode: "0640"
   with_dict: "{{ nginx_redirects }}"
   notify: reload nginx
   tags:
@@ -207,7 +207,7 @@
     dest: "{{ nginx_server_static_dir }}/{{ item.file }}"
     owner: root
     group: "{{ common_web_user }}"
-    mode: 0640
+    mode: "0640"
   with_items: "{{ NGINX_SERVER_HTML_FILES }}"
   tags:
     - install

playbooks/roles/user/tasks/main.yml

@@ -124,7 +124,7 @@
   template: 
     src: default.bashrc.j2
     dest: "/home/{{ item.name }}/.bashrc" 
-    mode: 0640
+    mode: "0640"
     owner: "{{ item.name }}"
   when: not (item.type is defined and item.type == 'restricted') and item.get('state', 'present') == 'present'
   with_items: "{{ user_info }}"
@@ -133,7 +133,7 @@
   template:
     src: default.profile.j2
     dest: "/home/{{ item.name }}/.profile" 
-    mode: 0640
+    mode: "0640"
     owner: "{{ item.name }}"
   when: item.get('state', 'present') == 'present'
   with_items: "{{ user_info }}"
@@ -152,7 +152,7 @@
   template:
     src: restricted.bashrc.j2
     dest: "/home/{{ item.name }}/.bashrc" 
-    mode: 0640
+    mode: "0640"
     owner: "{{ item.name }}"
   when: item.type is defined and item.type == 'restricted' and item.get('state', 'present') == 'present'
   with_items: "{{ user_info }}"
@@ -178,7 +178,7 @@
   file:
     path: "/home/{{ item.name }}/bin" 
     state: directory 
-    mode: 0750
+    mode: "0750"
     owner: root 
     group: "{{ item.name }}"
   when: item.type is defined and item.type == 'restricted' and item.get('state', 'present') == 'present'