redis: Add redis_log_level parameter and reduce default verbosity.

JGoutin · Aug 26, 2024 · 23225b0 · 23225b0
1 parent 947a21f
commit 23225b0
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/roles/redis/README.md b/roles/redis/README.md
@@ -22,9 +22,10 @@ Security:
 
 ### Optional
 
-| Name         | Default Value | Description                                            |
-|--------------|---------------|--------------------------------------------------------|
-| `redis_user` |               | If specified, allow this Unix user to access to Redis. |
+| Name              | Default Value | Description                                            |
+|-------------------|---------------|--------------------------------------------------------|
+| `redis_log_level` | `"warning"`   | Redis log level in the journal.                        |
+| `redis_user`      |               | If specified, allow this Unix user to access to Redis. |
 
 ## Example Playbook
 

diff --git a/roles/redis/defaults/main.yml b/roles/redis/defaults/main.yml
@@ -0,0 +1,2 @@
+---
+redis_log_level: warning
diff --git a/roles/redis/tasks/main.yml b/roles/redis/tasks/main.yml
@@ -25,7 +25,7 @@
 - name: Ensure Redis is configured
   ansible.builtin.lineinfile:
     path: /etc/redis/redis.conf
-    regexp: '^#?\\s*{{ item.option }}\s'
+    regexp: '^#?\s*{{ item.option }}\s'
     line: '{{ item.option }} {{ item.value }}'
   with_items:
     - option: unixsocket
@@ -38,6 +38,8 @@
       value: "yes"
     - option: logfile
       value: "/dev/null"
+    - option: loglevel
+      value: "{{ redis_log_level }}"
 
 - name: Ensure Redis service parent directory is present
   ansible.builtin.file: