Add fail2ban role and clean up yaml lint noise

roles/ensure_ansible_prereq/.travis.yml

@@ -1,16 +1,13 @@
 ---
 language: python
-python: "2.7"
+python: '2.7'
 
 # Use the new container infrastructure
 sudo: false
-
 # Install ansible
 addons:
   apt:
-    packages:
-    - python-pip
-
+    packages: [python-pip]
 install:
   # Install ansible
   - pip install ansible
@@ -20,10 +17,8 @@ install:
 
   # Create ansible.cfg with correct roles_path
   - printf '[defaults]\nroles_path=../' >ansible.cfg
-
 script:
   # Basic role syntax check
   - ansible-playbook tests/test.yml -i tests/inventory --syntax-check
-
 notifications:
-  webhooks: https://galaxy.ansible.com/api/v1/notifications/
+  webhooks: https://galaxy.ansible.com/api/v1/notifications/

roles/ensure_ansible_prereq/defaults/main.yml

@@ -1,2 +1,2 @@
 ---
-# defaults file for ensure_ansible_prereq
+...

roles/ensure_ansible_prereq/handlers/main.yml

@@ -1,17 +1,17 @@
 ---
 # handlers file for ensure_ansible_prereq
-- name: 'ensure_ansible_prereq.package_facts'
+- name: ensure_ansible_prereq.package_facts
   ansible.builtin.package_facts:
-- name: 'ensure_ansible_prereq.service_facts'
+- name: ensure_ansible_prereq.service_facts
   ansible.builtin.service_facts:
-- name: 'ensure_ansible_prereq.service_reload'
+- name: ensure_ansible_prereq.service_reload
   when:
     - ansible_facts["system"] == 'Linux'
     - ansible_facts["service_mgr"] == 'systemd'
     - ensure_ansible_prereq is defined
   ansible.builtin.systemd:
     daemon_reload: 'yes'
-- name: 'ensure_ansible_prereq.service_restart'
+- name: ensure_ansible_prereq.service_restart
   when:
     - ansible_facts["system"] == 'Linux'
     - ensure_ansible_prereq is defined
@@ -21,9 +21,7 @@
   ansible.builtin.service:
     enabled: '{{ item.enabled }}'
     name: '{{ item.name }}'
-    state: 'restarted'
+    state: restarted
   loop: '{{ ensure_ansible_prereq.service_list }}'
   loop_control:
     label: '{{ item.name }} will be restarted'
-...
-

roles/ensure_ansible_prereq/meta/main.yml

@@ -1,3 +1,4 @@
+---
 galaxy_info:
   author: Jason Rothstein
   description: Ensure required pre-req are present for ansible management
@@ -15,7 +16,6 @@ galaxy_info:
   # - Apache-2.0
   # - CC-BY-4.0
   license: LGPL-3.0-or-later
-
   min_ansible_version: 2.9
 
   # If this a Container Enabled role, provide the minimum Ansible Container version.
@@ -38,7 +38,6 @@ galaxy_info:
   #   - 1.0
   #   - 7
   #   - 99.99
-
   galaxy_tags: []
     # List tags for your role here, one per line. A tag is a keyword that describes
     # and categorizes the role. Users find roles by searching for tags. Be sure to
@@ -46,8 +45,6 @@ galaxy_info:
     #
     # NOTE: A tag is limited to a single word comprised of alphanumeric characters.
     #       Maximum 20 tags per role.
-
 dependencies: []
   # List your role dependencies here, one per line. Be sure to remove the '[]' above,
   # if you add dependencies to this list.
-  

roles/ensure_ansible_prereq/tasks/Debian-default.yml

@@ -1,9 +1,7 @@
 ---
 # tasks file for ensure_ansible_prereq
-- name: 'repository changes'
+- name: repository changes
   ansible.builtin.lineinfile:
-    path: '/etc/apt/sources.list'
-    state: 'absent'
+    path: /etc/apt/sources.list
+    state: absent
     regexp: 'deb cdrom:'
-...
-

roles/ensure_ansible_prereq/tasks/default.yml

@@ -1,2 +1,2 @@
 ---
-# tasks file for ensure_ansible_prereq
+...

roles/ensure_ansible_prereq/tasks/main.yml

@@ -1,51 +1,56 @@
 ---
 # tasks file for ensure_ansible_prereq
-- name: 'include vendor / version specific tasks'
-  when:
-    - ansible_facts["system"] == 'Linux'
+- name: include vendor / version specific tasks
+  when: ["ansible_facts[\"system\"] == 'Linux'"]
   ansible.builtin.include_tasks:
     file: '{{ lookup("first_found", findme) }}'
   vars:
     findme:
       files:
-        - '{{ ansible_facts["distribution"] }}-{{ ansible_facts["distribution_major_version"] }}-{{ ansible_facts["architecture"] }}.yml'
-        - '{{ ansible_facts["distribution"] }}-{{ ansible_facts["distribution_major_version"] }}-default.yml'
+        - '{{ ansible_facts["distribution"] }}-{{ ansible_facts["distribution_major_version"]
+          }}-{{ ansible_facts["architecture"] }}.yml'
+        - '{{ ansible_facts["distribution"] }}-{{ ansible_facts["distribution_major_version"]
+          }}-default.yml'
         - '{{ ansible_facts["distribution"] }}-default.yml'
-        - '{{ ansible_facts["os_family"] }}-{{ ansible_facts["distribution_major_version"] }}-{{ ansible_facts["architecture"] }}.yml'
-        - '{{ ansible_facts["os_family"] }}-{{ ansible_facts["distribution_major_version"] }}-default.yml'
+        - '{{ ansible_facts["os_family"] }}-{{ ansible_facts["distribution_major_version"]
+          }}-{{ ansible_facts["architecture"] }}.yml'
+        - '{{ ansible_facts["os_family"] }}-{{ ansible_facts["distribution_major_version"]
+          }}-default.yml'
         - '{{ ansible_facts["os_family"] }}-default.yml'
-        - 'default.yml'
-      errors: 'ignore'
-- name: 'include vendor / version specific variables'
-  when:
-    - ansible_facts["system"] == 'Linux'
+        - default.yml
+      errors: ignore
+- name: include vendor / version specific variables
+  when: ["ansible_facts[\"system\"] == 'Linux'"]
   ansible.builtin.include_vars:
     file: '{{ lookup("first_found", findme) }}'
-    name: 'ensure_ansible_prereq'
+    name: ensure_ansible_prereq
   vars:
     findme:
       files:
-        - '{{ ansible_facts["distribution"] }}-{{ ansible_facts["distribution_major_version"] }}-{{ ansible_facts["architecture"] }}.yml'
-        - '{{ ansible_facts["distribution"] }}-{{ ansible_facts["distribution_major_version"] }}-default.yml'
+        - '{{ ansible_facts["distribution"] }}-{{ ansible_facts["distribution_major_version"]
+          }}-{{ ansible_facts["architecture"] }}.yml'
+        - '{{ ansible_facts["distribution"] }}-{{ ansible_facts["distribution_major_version"]
+          }}-default.yml'
         - '{{ ansible_facts["distribution"] }}-default.yml'
-        - '{{ ansible_facts["os_family"] }}-{{ ansible_facts["distribution_major_version"] }}-{{ ansible_facts["architecture"] }}.yml'
-        - '{{ ansible_facts["os_family"] }}-{{ ansible_facts["distribution_major_version"] }}-default.yml'
+        - '{{ ansible_facts["os_family"] }}-{{ ansible_facts["distribution_major_version"]
+          }}-{{ ansible_facts["architecture"] }}.yml'
+        - '{{ ansible_facts["os_family"] }}-{{ ansible_facts["distribution_major_version"]
+          }}-default.yml'
         - '{{ ansible_facts["os_family"] }}-default.yml'
-        - 'default.yml'
-      paths:
-        - '../vars/'
-      errors: 'ignore'
-- name: 'package discovery'
+        - default.yml
+      paths: [../vars/]
+      errors: ignore
+- name: package discovery
   when:
     - ansible_facts["system"] == 'Linux'
     - ansible_facts["packages"] is not defined
   ansible.builtin.package_facts:
-- name: 'service discovery'
+- name: service discovery
   when:
     - ansible_facts["system"] == 'Linux'
     - ansible_facts["services"] is not defined
   ansible.builtin.service_facts:
-- name: 'ensure packages'
+- name: ensure packages
   when:
     - ansible_facts["system"] == 'Linux'
     - ensure_ansible_prereq is defined
@@ -58,9 +63,9 @@
   loop_control:
     label: '{{ item.name }} will be {{ item.state }}'
   notify:
-    - 'ensure_ansible_prereq.package_facts'
-    - 'ensure_ansible_prereq.service_facts'
-- name: 'ensure configurations'
+    - ensure_ansible_prereq.package_facts
+    - ensure_ansible_prereq.service_facts
+- name: ensure configurations
   when:
     - ansible_facts["system"] == 'Linux'
     - ensure_ansible_prereq is defined
@@ -81,11 +86,11 @@
   loop_control:
     label: '{{ item.dest }} will be ensured'
   notify:
-    - 'ensure_ansible_prereq.package_facts'
-    - 'ensure_ansible_prereq.service_facts'
-    - 'ensure_ansible_prereq.service_reload'
-    - 'ensure_ansible_prereq.service_restart'
-- name: 'ensure services'
+    - ensure_ansible_prereq.package_facts
+    - ensure_ansible_prereq.service_facts
+    - ensure_ansible_prereq.service_reload
+    - ensure_ansible_prereq.service_restart
+- name: ensure services
   when:
     - ansible_facts["system"] == 'Linux'
     - ensure_ansible_prereq is defined
@@ -99,8 +104,7 @@
   loop_control:
     label: '{{ item.name }} will be {{ item.state }}'
   notify:
-    - 'ensure_ansible_prereq.package_facts'
-    - 'ensure_ansible_prereq.service_facts'
-- name: 'flush handlers'
-  ansible.builtin.meta: 'flush_handlers'
-...
+    - ensure_ansible_prereq.package_facts
+    - ensure_ansible_prereq.service_facts
+- name: flush handlers
+  ansible.builtin.meta: flush_handlers

roles/ensure_ansible_prereq/tests/test.yml

@@ -1,5 +1,4 @@
 ---
 - hosts: localhost
   remote_user: root
-  roles:
-    - ensure_ansible_prereq
+  roles: [ensure_ansible_prereq]

roles/ensure_ansible_prereq/vars/default.yml

@@ -1,2 +1,2 @@
 ---
-# vars file for ensure_ansible_prereq
+...

roles/ensure_ansible_prereq/vars/main.yml

@@ -1,2 +1,2 @@
 ---
-# vars file for ensure_ansible_prereq
+...