diff options
| author | Romain Gonçalves <me@rgoncalves.se> | 2022-10-22 16:41:45 +0200 |
|---|---|---|
| committer | Romain Gonçalves <me@rgoncalves.se> | 2022-10-22 16:41:45 +0200 |
| commit | 0df015e9b5c4aefd4245c94ee98021d2af80ed99 (patch) | |
| tree | d9dbf4259500857daf41b627dbc2d116ba92dbc1 /roles/pf | |
| parent | 1cd2d31e0d5f81ce066121de78c0ed042fc3b6aa (diff) | |
| download | rules-0df015e9b5c4aefd4245c94ee98021d2af80ed99.tar.gz | |
lint: pass ansible-lint + yamllint
Diffstat (limited to 'roles/pf')
| -rw-r--r-- | roles/pf/handlers/main.yml | 6 | ||||
| -rw-r--r-- | roles/pf/tasks/main.yml | 4 |
2 files changed, 5 insertions, 5 deletions
diff --git a/roles/pf/handlers/main.yml b/roles/pf/handlers/main.yml index 5d75a1a..8e49812 100644 --- a/roles/pf/handlers/main.yml +++ b/roles/pf/handlers/main.yml @@ -1,12 +1,12 @@ - name: lint pf configuration - command: "pfctl -nf {{ pf_configuration_file }}" + ansible.builtin.command: "pfctl -nf {{ pf_configuration_file }}" - name: enable pf - command: pfctl -e + ansible.builtin.command: pfctl -e register: result failed_when: - result.rc != 0 - "'already enabled' not in result.stderr" - name: restart pf - command: pfctl -f "{{ pf_configuration_file }}" + ansible.builtin.command: pfctl -f "{{ pf_configuration_file }}" diff --git a/roles/pf/tasks/main.yml b/roles/pf/tasks/main.yml index e5b8af8..eb6f078 100644 --- a/roles/pf/tasks/main.yml +++ b/roles/pf/tasks/main.yml @@ -1,5 +1,5 @@ - name: generate pf configuration - template: + ansible.builtin.template: src: pf.conf.j2 dest: "{{ pf_configuration_file }}" owner: 0 @@ -11,7 +11,7 @@ - restart pf - name: test ssh connection on new pf rule - wait_for: + ansible.builtin.wait_for: port: "{{ ansible_port }}" delay: 2 state: started |