Merge pull request DavidWittman#144 from splitice/master

Two minor fixes

Author: DavidWittman

tasks/install.yml

@@ -24,18 +24,21 @@
     path: /etc/redis
     state: directory
 
+- name: check if redis user exists (ignore errors)
+  command: id {{ redis_user }}
+  ignore_errors: yes
+  changed_when: false
+  register: user_exists
+
 - name: add redis user
   user:
     name: "{{ redis_user }}"
     comment: "Redis"
     home: "{{ redis_install_dir }}"
     shell: /bin/false
     system: yes
+  when: user_exists|failed
 
-- name: add redis group
-  group:
-    name: "{{ redis_group }}"
-    state: present
 
 - name: create /var/run/redis
   file:

templates/redis.conf.j2

@@ -24,9 +24,9 @@ databases {{ redis_databases }}
 {% for save in redis_save -%}
 save {{ save }}
 {% endfor -%}
-stop-writes-on-bgsave-error {{ redis_stop_writes_on_bgsave_error }}
-rdbcompression {{ redis_rdbcompression }}
-rdbchecksum {{ redis_rdbchecksum }}
+stop-writes-on-bgsave-error {{ redis_stop_writes_on_bgsave_error|string }}
+rdbcompression {{ redis_rdbcompression|string }}
+rdbchecksum {{ redis_rdbchecksum|string }}
 dbfilename dump.rdb
 
 # Replication
@@ -68,7 +68,7 @@ maxmemory-policy {{ redis_maxmemory_policy }}
 # Append Only Mode
 appendonly {{ redis_appendonly }}
 appendfilename "{{ redis_appendfilename }}"
-appendfsync {{ redis_appendfsync }}
+appendfsync {{ redis_appendfsync|string }}
 no-appendfsync-on-rewrite {{ redis_no_appendfsync_on_rewrite }}
 auto-aof-rewrite-percentage {{ redis_auto_aof_rewrite_percentage }}
 auto-aof-rewrite-min-size {{ redis_auto_aof_rewrite_min_size }}