Skip to content

Commit

Permalink
Make sure tune_cpu gets started
Browse files Browse the repository at this point in the history
In the past we could (and did) fail somewhere in the playbook which
would result in not handling this notified handler. This also meant that
we would not copy the file again on the next run, so didn't kick the
handler and also didn't hit the handler on the next run.

The disadvantage here is we might hit other handlers more often if next
roles want to hit the same handler.
  • Loading branch information
vmcj committed Sep 17, 2024
1 parent 215e2f2 commit 2bada2d
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions provision-contest/ansible/roles/judgedaemon/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,10 @@
notify:
- Enable and restart tune_cpu

- name: Enable and restart tune_cpu (flushing as this should always happen even if we fail further)
# We need a daemon-reload, so we loose idempotency if we start the service in the next block
ansible.builtin.meta: flush_handlers

- name: Copy judgedaemon related systemd unit files
copy:
remote_src: true
Expand Down

0 comments on commit 2bada2d

Please sign in to comment.