Move handlers back to tasks

roles/newsreader/handlers/main.yml

@@ -1,62 +1,34 @@
-- name: install npm packages
-  become_user: "{{ app_user }}"
-  command: /usr/bin/npm install
-  args:
-    chdir: "{{ app_dir }}"
-
-- name: build static files
-  become_user: "{{ app_user }}"
-  command: /usr/bin/npm run build:prod
-  args:
-    chdir: "{{ app_dir }}"
-
-- name: run migrations
-  become_user: "{{ app_user }}"
-  django_manage:
-    command: migrate
-    app_path: "{{ app_dir }}/src/"
-    virtualenv: "{{ app_dir }}/.venv"
-    settings: "newsreader.conf.production"
-
-- name: collect static files
-  become_user: "{{ app_user }}"
-  django_manage:
-    command: collectstatic
-    app_path: "{{ app_dir }}/src/"
-    virtualenv: "{{ app_dir }}/.venv"
-    settings: "newsreader.conf.production"
-
 - name: restart gunicorn socket
   systemd:
-    daemon-reload: yes
+    daemon-reload: true
     name: gunicorn.socket
     state: restarted
-    enabled: yes
+    enabled: true
 
 - name: stop gunicorn service
   systemd:
-    daemon-reload: yes
+    daemon-reload: true
     name: gunicorn.service
     state: stopped
-    enabled: no
+    enabled: false
 
 - name: restart pgbouncer
   systemd:
-    daemon-reload: yes
+    daemon-reload: true
     name: pgbouncer
     state: restarted
-    enabled: yes
+    enabled: true
 
 - name: restart celery
   systemd:
-    daemon-reload: yes
+    daemon-reload: true
     name: celery
     state: restarted
-    enabled: yes
+    enabled: true
 
 - name: restart celerybeat
   systemd:
-    daemon-reload: yes
+    daemon-reload: true
     name: celerybeat
     state: restarted
-    enabled: yes
+    enabled: true

roles/newsreader/tasks/main.yml

@@ -163,6 +163,9 @@
       }
 
 - name: ensure pgbouncer is restarted
-  service: name=pgbouncer state=restarted enabled=yes
+  systemd:
+    name: pgbouncer
+    state: restarted
+    enabled: true
 
 - include_tasks: "project.yml"

roles/newsreader/tasks/project.yml

@@ -20,13 +20,36 @@
     repo: "https://git.fudiggity.nl/sonny/newsreader.git"
     dest: "{{ app_dir }}"
     version: "{{ app_branch }}"
-    update: yes
+    update: true
-    force: yes
+    force: true
-  notify:
+
-    - install npm packages
+- name: install npm packages
-    - build static files
+  become_user: "{{ app_user }}"
-    - run migrations
+  command: /usr/bin/npm install
-    - collect static files
+  args:
+    chdir: "{{ app_dir }}"
+
+- name: build static files
+  become_user: "{{ app_user }}"
+  command: /usr/bin/npm run build:prod
+  args:
+    chdir: "{{ app_dir }}"
+
+- name: run migrations
+  become_user: "{{ app_user }}"
+  django_manage:
+    command: migrate
+    app_path: "{{ app_dir }}/src/"
+    virtualenv: "{{ app_dir }}/.venv"
+    settings: "newsreader.conf.production"
+
+- name: collect static files
+  become_user: "{{ app_user }}"
+  django_manage:
+    command: collectstatic
+    app_path: "{{ app_dir }}/src/"
+    virtualenv: "{{ app_dir }}/.venv"
+    settings: "newsreader.conf.production"
 
 - name: include poetry tasks
   include_role: