Fix Linting (#46)

* Add requirements.yml in root
* Add community.general to requirements
* Add community.mysql to requirements
* Name all import tasks
* Capitalize all task names

defaults/main.yml

@@ -31,7 +31,7 @@ userli_env_vars:
   - name: WKD_FORMAT
     value: "{{ userli_wkd_format }}"
 
-userli_prometheus_textfile_collector_enabled: false
+userli_prometheus_textfile_collector_enabled: False
 userli_prometheus_textfile_collector_path: "/var/lib/prometheus/node-exporter"
 userli_prometheus_textfile_collector_on_calendar: "*-*-* *:*:00"
 

handlers/main.yml

@@ -1,7 +1,6 @@
 ---
 - name: Restart munin-node
   ansible.builtin.service:
-  service:
     name: munin-node
     state: restarted
 
@@ -29,11 +28,11 @@
     state: restarted
 
 - name: Reload systemd daemon
-  systemd:
+  ansible.builtin.systemd:
     daemon_reload: true
 
 - name: Restart userli-collector
-  service:
+  ansible.builtin.service:
     name: "{{ item }}"
     state: restarted
   loop:

molecule/default/prepare.yml

@@ -4,13 +4,12 @@
   become: true
   pre_tasks:
     - name: Run the equivalent of "apt-get update" before installing packages
-      apt:
+      ansible.builtin.apt:
         update_cache: yes
 
     - name: Install cron for testing in docker containers
-      apt:
+      ansible.builtin.apt:
         pkg: cron
-        update_cache: yes
   roles:
     - weareinteractive.apt
     - geerlingguy.apache
@@ -25,7 +24,7 @@
         extra_parameters: |
           SetEnv APP_ENV prod
     apt_packages:
-      - acl  # support for unpriviledged become_user
+      - acl # support for unpriviledged become_user
       - unzip
     php_default_version_debian: "{% if ansible_distribution_release == 'bullseye' %}7.4{% else %}8.2{% endif %}"
     php_packages:

molecule/default/requirements.yml

@@ -1,5 +1,4 @@
 ---
-
 roles:
   - name: weareinteractive.apt
   - name: geerlingguy.apache

requirements.yml

@@ -1,5 +1,4 @@
 ---
-
 collections:
   - community.general
   - community.mysql

tasks/main.yml

@@ -105,5 +105,5 @@
   when: userli_munin | bool
 
 - name: Import metrics tasks
-  import_tasks: metrics.yml
+  ansible.builtin.import_tasks: metrics.yml
   when: userli_prometheus_textfile_collector_enabled | bool