From 5582d6e724511c7536c2c4030fd69f1424f3d5fa Mon Sep 17 00:00:00 2001
From: Jeremy Lecour <jlecour@evolix.fr>
Date: Tue, 9 Mar 2021 18:32:35 +0100
Subject: [PATCH] redis: socket/pid directories have the correct permissions

---
 CHANGELOG.md                    |  2 +-
 redis/tasks/instance-server.yml | 18 +++++++++++++++---
 2 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 7fb0d5ba..9c8cf5a9 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -36,7 +36,7 @@ The **patch** part changes incrementally at each release.
 ### Fixed
 
 * bind9: added log files to apparmor definition so bind can run
-
+* redis: socket/pid directories have the correct permissions
 
 ### Removed
 
diff --git a/redis/tasks/instance-server.yml b/redis/tasks/instance-server.yml
index 502adc8b..1b491d9a 100644
--- a/redis/tasks/instance-server.yml
+++ b/redis/tasks/instance-server.yml
@@ -70,7 +70,21 @@
   tags:
     - redis
 
-- name: "Instance '{{ redis_instance_name }}' other directories are present"
+- name: "Instance '{{ redis_instance_name }}' socket/pid directories are present"
+  file:
+    dest: "{{ item }}"
+    mode: "0755"
+    owner: "redis-{{ redis_instance_name }}"
+    group: "redis-{{ redis_instance_name }}"
+    follow: yes
+    state: directory
+  with_items:
+    - "{{ redis_pid_dir }}"
+    - "{{ redis_socket_dir }}"
+  tags:
+    - redis
+
+- name: "Instance '{{ redis_instance_name }}' data/log directories are present"
   file:
     dest: "{{ item }}"
     mode: "0750"
@@ -79,8 +93,6 @@
     follow: yes
     state: directory
   with_items:
-    - "{{ redis_pid_dir }}"
-    - "{{ redis_socket_dir }}"
     - "{{ redis_data_dir }}"
     - "{{ redis_log_dir }}"
   tags: