import readability

roles/ftp/tasks/main.yml

@@ -33,7 +33,7 @@
       openssl dhparam 
       -outform PEM -2|-5 
       1024|1536|2048|3072|4096|6144|7680|8192 
-      >> /etc/proftpd/dhparams.pem
+      > /etc/proftpd/dhparams.pem
     creates: /etc/proftpd/dhparams.pem
 
 - name: restart and enable proftpd

roles/system/tasks/main.yml

@@ -9,17 +9,26 @@
     - { group: syscom,      gid: 10001 }
     - { group: csc-mirror,  gid: 10014 }
 
-- name: create users 
+- name: create push user
+  # push home should be /
+  user:
+    uid: 1002
+    name: push
+    group: push
+    shell: /bin/sh
+    create_home: no
+
+- name: create other users
+  # syscom and csc-mirror homes are on /users
   user:
-    name: "{{ item.user }}"
-    shell: "{{ item.shell }}"
     uid: "{{ item.uid }}"
+    name: "{{ item.user }}"
     group: "{{ item.user }}"
+    shell: /bin/bash
     create_home: "{{ item.home }}"
   loop:
-    - { user: local, uid: 1000 }
-    - { user: mirror, uid: 1001 }
-    - { user: push, uid: 1002, shell: /bin/sh }
+    - { user: local,      uid: 1000,  home: yes }
+    - { user: mirror,     uid: 1001,  home: yes }
     - { user: syscom,     uid: 10001, home: no }
     - { user: csc-mirror, uid: 10014, home: no }