Fix linter errors

.ansible-lint

@@ -1,19 +1,25 @@
+# .ansible-lint
+exclude_paths:
+  - .cache/
+  - .github/
+  - molecule/
+
 rulesdir:
-   - ../../rules/
+   - ../../ansible-lint-custom-rules/rules/
 
 enable_list:
   - fqcn-builtins  # opt-in
   - no-log-password  # opt-in
 
 warn_list:
-  - role_vars_start_with_role_name  # All role parameters must be named ^<role_name>_.*
+  - role_vars_start_with_role_name
-  - vars_in_vars_files_have_valid_names  # Variable in vars files must have valid name
+  - vars_in_vars_files_have_valid_names
-  - vars_should_not_be_used  # vars should not be used
+  - vars_should_not_be_used
-  - experimental  # all rules tagged as experimental
+  - experimental
-  - ignore-errors  # Use failed_when and specify error conditions instead of using ignore_errors
+  - ignore-errors
-  - no-handler  # Tasks that run when changed should likely be handlers
+  - no-handler
   - fqcn-builtins
   - no-log-password
 
 use_default_rules: true
-offline: false
+offline: true

.github/workflows/ci.yml

@@ -33,7 +33,7 @@ jobs:
         uses: actions/checkout@v2
         with:
           repository: ansible-middleware/ansible-lint-custom-rules
-          path: ansible_collections/rules/
+          path: ansible_collections/ansible-lint-custom-rules/
 
       - name: Create default collection path
         run: |

molecule/default/molecule.yml

@@ -5,7 +5,8 @@ dependency:
 driver:
   name: docker
 lint: |
-  ansible-lint roles/*
+  ansible-lint --version
+  ansible-lint -vv
 platforms:
   - name: instance
     image: registry.access.redhat.com/ubi8/ubi-init:latest

roles/keycloak/tasks/fastpackages/check.yml

@@ -8,7 +8,7 @@
     changed_when: rpm_info.failed
 
   rescue:
-    - name: "If package {{ package_name }} is missing, add it to the yum install list."
+    - name: "Add {{ package_name }} to the yum install list if missing"
       set_fact:
         packages_to_install: "{{ packages_to_install + [ package_name ] }}"
       when: rpm_info.failed

roles/keycloak/tasks/fastpackages/install.yml

@@ -1,5 +1,6 @@
 ---
-- set_fact:
+- name: Set facts
+  set_fact:
     update_cache: true
     packages_to_install: []
 

roles/keycloak/tasks/install.yml

@@ -81,7 +81,7 @@
     - not keycloak_rhsso_enable
     - not keycloak_offline_install
 
-- name: Performing download from RHN
+- name: Perform download from RHN
   redhat_csp_download:
     url: "{{ keycloak_rhsso_download_url }}"
     dest: "{{ local_path.stat.path }}/{{ keycloak.bundle }}"

roles/keycloak/tasks/main.yml

@@ -38,6 +38,7 @@
           - "-rmaster"
           - "-u{{ keycloak_admin_user }}"
           - "-p{{ keycloak_admin_password }}"
+      changed_when: yes
       become: yes
     - name: "Restart {{ keycloak.service_name }}"
       include_tasks: tasks/restart_keycloak.yml

roles/keycloak/tasks/systemd.yml

@@ -56,7 +56,8 @@
       - keycloak_service_status is defined
       - keycloak_service_status.stdout is defined
 
-- meta: flush_handlers
+- name: Flush handlers
+  meta: flush_handlers
 
 - name: "Wait until Keycloak becomes active {{ keycloak.health_url }}"
   uri: