#28 Address Ansible Lint '[208] File permissions not mentioned' for a…

… few other cases
T2L · Aug 20, 2020 · bb6847b · bb6847b
1 parent 492ed89
commit bb6847b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/tasks/cores.yml b/tasks/cores.yml
@@ -6,6 +6,7 @@
     owner: '{{ solr_user }}'
     group: '{{ solr_group }}'
     recurse: true
+    mode: 0755
   loop: '{{ solr_cores }}'
 
 - name: Set default core configuration path

diff --git a/tasks/install.yml b/tasks/install.yml
@@ -29,12 +29,19 @@
   when: solr_needs_install
 
 # Make sure all directories exist
-- name: Make sure all directories exist
+- name: Make sure download directory exists
   file:
-    path: "{{ item }}"
+    path: '{{ solr_download_dir }}'
     state: directory
+    mode: 0777
+  when: solr_needs_install
+
+- name: Make sure install & data directories exist
+  file:
+    path: '{{ item }}'
+    state: directory
+    mode: 0755
   loop:
-    - '{{ solr_download_dir }}'
     - '{{ solr_install_dir }}'
     - '{{ solr_data_dir }}'
   when: solr_needs_install