Update Firewall role so Ubuntu 16.04 doesn't complain when ufw isn't installed.

Author: geerlingguy

provisioning/requirements.yml

@@ -26,7 +26,7 @@
 - src: geerlingguy.elasticsearch
   version: 2.1.3
 - src: geerlingguy.firewall
-  version: 2.2.0
+  version: 2.2.1
 - src: geerlingguy.git
   version: 1.3.0
 - src: geerlingguy.java

provisioning/roles/geerlingguy.firewall/tasks/disable-other-firewalls.yml

@@ -1,6 +1,8 @@
 ---
-- name: Check firewalld package is installed (on RHEL).
+- name: Check if firewalld package is installed (on RHEL).
   shell: yum list installed firewalld
+  args:
+    warn: no
   register: firewalld_installed
   ignore_errors: true
   changed_when: false
@@ -13,8 +15,10 @@
     enabled: no
   when: ansible_os_family == "RedHat" and firewall_disable_firewalld and firewalld_installed.rc == 0
 
-- name: Check ufw package is installed (on Ubuntu).
-  shell: dpkg -l ufw
+- name: Check if ufw package is installed (on Ubuntu).
+  shell: service ufw status
+  args:
+    warn: no
   register: ufw_installed
   ignore_errors: true
   changed_when: false
@@ -27,8 +31,10 @@
     enabled: no
   when: ansible_distribution == "Ubuntu" and firewall_disable_ufw and ufw_installed.rc == 0
 
-- name: Check ufw package is installed (on Archlinux).
+- name: Check if ufw package is installed (on Archlinux).
   command: pacman -Q ufw
+  args:
+    warn: no
   register: ufw_installed
   ignore_errors: true
   changed_when: false

provisioning/roles/geerlingguy.firewall/templates/firewall.bash.j2

@@ -8,7 +8,7 @@
 #   22: SSH
 #   25: SMTP
 #   80: HTTP
-#   123: DNS
+#   123: NTP
 #   443: HTTPS
 #   2222: SSH alternate
 #   4949: Munin