Merge branch 'master' of adellam/mailman into master

tasks/main.yml

@@ -13,7 +13,10 @@
 - name: Create the mailman user
   block:
     - name: mailman user
-      user: name={{ mailman_srv_user }} createhome=yes shell=/bin/bash system=yes comment='Mailman User' generate_ssh_key=yes home={{ mailman_home }}
+      user: name={{ mailman_srv_user }} createhome=no shell=/usr/sbin/nologin system=yes comment='Mailman User' generate_ssh_key=yes home={{ mailman_home }}
+
+    - name: Create the mailman user directory
+      file: dest={{ mailman_home }} state=directory mode=0755 owner={{ mailman_srv_user }} group={{ mailman_srv_user }}
 
   when: ansible_distribution_file_variety == "RedHat"
   tags: [ 'mailman' ]
@@ -74,6 +77,9 @@
         - 'alembic'
         - 'mailman'
 
+    - name: Create the tmpfile entry for the mailman lock directory
+      template: src=tmpfile_mailman.conf.j2 dest=/lib/tmpfiles.d/mailman.conf owner=root group=root mode=0644
+
     - name: Install the mailman startup unit
       template: src=mailman.service.systemd.j2 dest=/lib/systemd/system/mailman.service owner=root group=root mode=0644
       register: mailman_unit_install
@@ -87,7 +93,7 @@
 
     - name: Restart mailman if the configuration changed
       service: name=mailman state=restarted
-      when: mailman_conf_install | bool
+      when: mailman_conf_install is changed
 
   tags: [ 'mailman', 'mailman_conf' ]
 

templates/mailman.cfg.j2

@@ -140,7 +140,7 @@ api_version: 3.1
 admin_user: {{ mailman_api_user }}
 
 # The administrative password.
-admin_pass: '{{ mailman_rest_api_pwd }}'
+admin_pass: {{ mailman_rest_api_pwd }}
 
 # Number of workers to start.
 # http://docs.gunicorn.org/en/stable/settings.html#workers

templates/tmpfile_mailman.conf.j2

@@ -0,0 +1 @@
+d {{ mailman_lock_dir }} 0775 {{ mailman_user }}  {{ mailman_user }}