Skip to content

Commit

Permalink
Fix notify to use the uppercase handler names
Browse files Browse the repository at this point in the history
  • Loading branch information
kyl191 committed Dec 22, 2024
1 parent 28ebf3c commit b6f508c
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 7 deletions.
2 changes: 1 addition & 1 deletion tasks/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
group: root
mode: "0644"
notify:
- restart openvpn
- Restart openvpn

- name: Copy up script if defined
ansible.builtin.copy:
Expand Down
2 changes: 1 addition & 1 deletion tasks/firewalld.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@
# Workaround ansible issue: https://github.com/ansible/ansible/pull/21693
# immediate: true
notify:
- restart firewalld
- Restart firewalld

# workaround for --permanent not working on non-NetworkManager managed ifaces
# https://bugzilla.redhat.com/show_bug.cgi?id=1112742
Expand Down
3 changes: 2 additions & 1 deletion tasks/iptables.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,8 @@
action: insert
comment: "Perform NAT readdressing"
when: openvpn_masquerade_not_snat
notify: "save iptables"
notify:
- Save iptables

- name: Save existing iptables rule before start iptables service
ansible.builtin.shell: "{{ iptables_save_command }}" # noqa command-instead-of-shell
Expand Down
4 changes: 2 additions & 2 deletions tasks/selinux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@
register: semodule_loaded
changed_when: "openvpn_selinux_module not in semodule_loaded.stdout"
notify:
- build and install policy
- Build and install policy

- name: SELinux - copy type enforcement file
ansible.builtin.template:
src: "selinux_module.te.j2"
dest: /var/lib/selinux/{{ openvpn_selinux_module }}.te
mode: "0644"
notify:
- build and install policy
- Build and install policy
4 changes: 2 additions & 2 deletions tasks/ufw.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@
COMMIT
when: not openvpn_masquerade_not_snat
notify:
- restart ufw
- Restart ufw

- name: Setup nat table rules with MASQUERADE - ufw
ansible.builtin.blockinfile:
Expand All @@ -58,4 +58,4 @@
COMMIT
when: openvpn_masquerade_not_snat
notify:
- restart ufw
- Restart ufw

0 comments on commit b6f508c

Please sign in to comment.